Выдача налогового калькулятора

Question

Я сделал эту программу, которая рассчитывает налог для ряда окладов на основе ввода пользователем. Все работает нормально, за исключением того, что я не могу найти способ заставить приложение игнорировать последние 3 printf и просто распечатать сообщение об ошибке, если пользователь вводит отрицательное значение. Я не хочу отображать последние 3 printf, если пользователь вводит отрицательное число, вместо этого я хочу, чтобы приложение просто отображало «Ошибка: вы ввели отрицательное число» или что-то в этом роде. Я использую визуальную студию в качестве моего компилятора, и это приложение C. Помогите плз. А также, пожалуйста, пройдите со мной на мой класс программирования, в котором мы все еще делаем очень простые вещи.

Вот код:

#include <stdio.h> 
#include <stdlib.h> 

int main(void) 
{ 

    float salary, tax = 0; 
    printf("\n\n\tEnter the salary ammount per year: \t\t$"); 
    scanf("%f", &salary); 

    if (salary <= 10000){ 
     tax=0; 
    } 
    else if (salary > 10000 && salary <= 40000){ 
     tax = (salary - 10000)*0.2; 
    } else if (salary > 40000 && salary <= 50000){ 
     tax = 6000 + (salary - 40000)*0.3 ; 
    } else if (salary > 50000 && salary <= 75000){ 
     tax = 9000 + (salary - 50000)*0.4; 
    } else if (salary > 75000.01 && salary <= 100000){ 
     tax = 19000 + (salary - 75000)*0.5; 
    } else if (salary > 100000 && salary >=100000.01){ 
     tax = 31500 + (salary - 100000)*0.6; 
    } 



    printf("\nSalary per year:\t\t $ %.2f\n\n", salary); 
    printf("Tax ammount per year:\t\t $ %.2f\n\n", tax); 
    printf("Salary after tax:\t\t $ %.2f\n", salary-tax); 


    getchar();getchar(); 
} 

Answer 1

Вы уже продемонстрировали, вы знаете, как использовать if заявления, чтобы проверить, относятся ли int в заданном диапазоне. Просто примените эту же логику к выходу. Отрицательные числа: < 0.

Например:

if(salary < 0) { 
    //print error 
} else { 

    if (salary <= 10000) { 
     tax=0; 
    } else if (salary > 10000 && salary <= 40000) { 
     tax = (salary - 10000)*0.2; 
    } else if (salary > 40000 && salary <= 50000) { 
     tax = 6000 + (salary - 40000)*0.3 ; 
    } else if (salary > 50000 && salary <= 75000) { 
     tax = 9000 + (salary - 50000)*0.4; 
    } else if (salary > 75000/*.01*/ && salary <= 100000) { 
     tax = 19000 + (salary - 75000)*0.5; 
    } else /*if (salary > 100000 && salary >=100000.01)*/ { 
     tax = 31500 + (salary - 100000)*0.6; 
    } 

    printf("\nSalary per year:\t\t $ %.2f\n\n", salary); 
    printf("Tax ammount per year:\t\t $ %.2f\n\n", tax); 
    printf("Salary after tax:\t\t $ %.2f\n", salary-tax); 
} 

На последнем else if, вы действительно не нужно if заявление, только else. Хотя быть уверенным (и для ясности), это хорошо, чтобы включить его. ОДНАКО, вам определенно не нужно ** salary >=100000.01)>= будет оценивать только true, если salary > 100000 уже оценен до true. И && будет оцениваться до true, только если обе стороны оценят значение true. Единственный способ быть правдой, а другой неверный - если кто-то вводит нечетную зарплату (100000.005), но даже если эта зарплата будет введена, они все равно должны быть в этой налоговой скобке.

Во всяком случае, ваши printf заявления теперь содержит в else блоке if(salary<0) {}else{} блока, так что они будут печатать только если зарплата была> = 0.

Answer 2

Я думаю, что ваш код подвержены ошибкам, и могу упрощен много:

if (salary <= 10000){ 
    tax=0; 
} else if (salary <= 40000){ 
    tax = (salary - 10000)*0.2; 
} else if (salary <= 50000){ 
    tax = 6000 + (salary - 40000)*0.3 ; 
} else if (salary <= 75000){ 
    tax = 9000 + (salary - 50000)*0.4; 
} else if (salary <= 100000){ 
    tax = 19000 + (salary - 75000)*0.5; 
} else { 
    tax = 31500 + (salary - 100000)*0.6; 
} 

Answer 3

Это не ответ на ваш вопрос, но это всегда лучше, чтобы компьютер делать все работы в расчетах. У вашего кода есть несколько промежуточных результатов, которые вы должны были вычислить вручную. Почему бы не позволить компьютеру это сделать? Он также позволяет вам легко изменять структуру ставок, редактируя ключевые функции, скобки и ставки, и не нужно беспокоиться о промежуточных результатах.

float tax = 0; 
struct { 
    float basis; 
    float rate; 
} *ptr, rateStructure[] = 
// list these from high to low brackets 
{ 
{100000,0.6}, 
{ 75000,0.5}, 
{ 50000,0.4}, 
{ 40000,0.3}, 
{ 10000,0.2}, 
{  0,0.1} 
}; 

ptr = rateStructure; 
do { 
    float bracket = salary - ptr->basis; 
    if (bracket > 0) { 
     tax += bracket * ptr->rate; 
     salary -= bracket; 
    } 
    ptr++; 
} while (salary);