Программа, которая ничего не возвращает

Question

Я работаю над программой на C, которая может вернуть кредит. yearInt в год интерес loanAmt является общая сумма кредита monthlyPay является ежемесячный платеж numberPay это номер ежемесячного платежа

По некоторым причинам, когда я запускаю программу, там ничего не показывает, даже я типа в отрицательном номер.

Есть ли все-таки исправить это?

#include <stdio.h> 
int main(void) 
{ 
float yearInt = -1; 
int loanAmt = -1; 
float monthlyPay = -1; 
int numberPay = -1; 
int count = 0; 

while (loanAmt<0) 
{ 
    printf("Please enter valid loan value: \n"); 
    scanf("%f", &loanAmt); 
} 

while (yearInt<0) 
{ 
    printf("Please enter valid yearly interest value: \n"); 
    scanf("%f", &yearInt) 
} 

while (monthlyPay<0) 
{ 
    printf("Please enter valid monthly payment value: \n"); 
    scanf("%f", &monthlyPay); 
} 

while (numberPay<0) 
{ 
    printf("Please enter valid number of monthly payments: \n"); 
    scanf("%f", &numberPay); 
} 

if(loanAmt>monthlyPay) 
{ 
    while(count<numberPay) 
    { 
     loanAmt = loanAmt*(1 + (yearInt/12)) - monthlyPay; 
     count += count+1; 
    } 
    printf("The amount of last payment is: %.2f\n", loanAmt); 
else 
    printf("The amount of last payment is: %.2f\n", loanAmt); 
} 
return 0; 
} 

Answer 1

scanf("%f", &loanAmt);   // loanAmt is int 
... 
scanf("%f", &numberPay);  //numberPay is int 

В обоих неправильном аргументе передается %f, следовательно, приводит к UB. Используйте спецификатор %d.

И в этом один } отсутствует -

if(loanAmt>monthlyPay) 
{ 
    while(count<numberPay) 
    { 
    loanAmt = loanAmt*(1 + (yearInt/12)) - monthlyPay; 
    count += count+1; 
    } 
    printf("The amount of last payment is: %.2f\n", loanAmt); //use %d 
    // ADD '}' here 
else // ADD '{' here 
    printf("The amount of last payment is: %.2f\n", loanAmt); //use %d 
} 

Поместите } перед тем else {.

Answer 2

Вы используете неправильный спецификатор формата для чтения loanAmount и numberPay. Вместо "%f" используйте "%d".

scanf("%d", &loanAmt); 

и

scanf("%d", &numberPay); 

Кроме того, всегда проверять возвращаемое значение scanf, чтобы убедиться, что он был в состоянии присвоить данные по всем переменным.

Изменение цикла:

while (loanAmt<0) 
{ 
    printf("Please enter valid loan value: \n"); 
    scanf("%d", &loanAmt); 
} 

в

while (loanAmt<0) 
{ 
    printf("Please enter valid loan value: \n"); 
    if (scanf("%d", &loanAmt) != 1) 
    { 
     // Discard the rest of the line. 
     int c; 
     while ((c = fgetc(stdin)) = '\n' && c != EOF); 
    } 
} 

Это будет еще лучше поставить все проверки в функции и вызов функции из main.

int readInt(char const* prompt) 
{ 
    int val = -1; 

    printf("%s\n", prompt); 
    while (scanf("%f", &val) != 1 || val < 0) 
    { 
     // Discard rest of the line. 
     int c; 
     while ((c = fgetc(stdin)) = '\n' && c != EOF); 

     // If EOF is reached, we have a problem. 
     if (c == EOF) 
     { 
     exit(0); 
     } 
     printf("%s\n", prompt); 
    } 

    return val; 
} 

, а затем используйте:

loanAmount = readInt("Please enter valid loan value: "); 

Добавить аналогичную функцию для чтения float с и называем его для чтения переменных, которые имеют типа float.