Логика if-else в домашнем задании по языку C

1. Ваш компилятор должен предупредить вас, что вы не передаете переменную типа float *, соответствующую строке формата %f. Вот что говорит gcc -Wall:

1.c: In function ‘main’:
1.c:8:17: warning: format ‘%f’ expects argument of type ‘float *’, but argument 2 has type ‘double’ [-Wformat=]
    8 |         scanf("%f", income);
      |                ~^   ~~~~~~
      |                 |   |
      |                 |   double
      |                 float *

и исправление состоит в том, чтобы изменить его на:

scanf("%f", &income);

2. Всегда проверяйте возвращаемое значение scanf(), иначе вы можете работать с неинициализированными переменными.

3. Вы хотите присвоить значение переменной tax.

4. При тестировании income вы пропускаете значения, равные (нижней или верхней) границе. Вы хотите использовать половину открытого интервала, скажем, [2.5, 5), который вы бы написали как income >= 2.5 && income < 5. Также рассмотрите возможность логического упорядочения условий. Как только вы это сделаете, вы сможете исключить нижнюю границу, поскольку каждое условие выполняется последовательно, как показал вам @Chris:

    double tax;
    if (income < 2.5) 
        tax = 0;
    else if (income < 5.0) 
        tax = 5;
    else if (income < 10.0) 
        tax = 20;
    else 
        tax = 30;

5. Следующий (необязательный) шаг рефакторинга — устранить повторяющиеся условия if-else путем поиска значения налога в массиве, который сопоставляет максимальную границу дохода со значением налога:

#include <values.h>

// ...

    float tax_rates[][2] = {
        {2.5, 0},
        {5, 20},
        {FLT_MAX, 30}
    };
    size_t n = sizeof tax_rates / sizeof *tax_rates;
    float tax = tax_rates[n - 1][1];
    for(size_t i = 0; i < n - 1; i++) {
        if (income < tax_rates[i][0]) {
            tax = tax_rates[i][1];
            break;
        }
    }

Основное преимущество этого подхода заключается в том, что таблица тарифов теперь представляет собой данные, которые могут изменяться независимо от вашего кода. Вы можете прочитать налоговые ставки из текстового файла или загрузить авторитетный источник (IRS, если вы находитесь в США).

6. Наконец, я предлагаю вам исключить подобные printf() утверждения, и вы, вероятно, захотите указать printf(), сколько значащих цифр вы хотите напечатать. Я использовал 2 ниже:

#include <stdio.h>

int main () {
    printf("Kindly note down your income here: ");
    float income;
    int rv = scanf("%f", &income);
    if (rv != 1) {
        printf("scanf failed\n");
        return 1;
    }

    float tax = 0;
    if (income >= 2.5 && income < 5)
        tax = 5;
    else if (income >= 5 && income < 10)
        tax = 20;
    else if (income >= 10)
        tax = 30;

    if (tax)
        printf("You have to pay %.2f tax because your income is %.2f\n", tax, income);
    else
        printf("You have to pay no tax because your income is %.2f\n", income);
}

Хотя Аллан Винд определил источник вашей ошибки, я бы также предложил вам реорганизовать ваше условие, изменив порядок, чтобы уменьшить количество элементов, которые вам нужно проверить, поскольку вы можете предположить, что все предыдущие проверенные условия являются ложными.

    if (income < 2.5) {
        printf("You have to pay no tax because your income is %f\n", income);
    }
    // We now *know* that income >= 2.5
    else if (income < 5.0) {
        printf("You have to pay 5 tax because your income is %f\n", income);
    }
    else if (income < 10.0) {
        printf("You have to pay 20 tax because your income is %f\n", income);
    }
    else {
        printf("You have to pay 30 tax because your income is %f\n", income);
    }

Хотя это не имеет отношения к этому очень простому примеру, использование чисел с плавающей запятой для работы с деньгами сомнительно, но использование чисел с плавающей запятой одинарной точности (по сравнению с double) особенно сомнительно, поскольку деньги могут включать в себя большие числа, которые делают операции с одинарной точностью. числа быстро становятся неточными, что приводит к ошибкам в расчетах. Лишь немногие виды арифметических ошибок расстраивают людей больше, чем те, которые связаны с деньгами.

Поскольку условия являются исчерпывающими (ветвь всегда выполняется), нет необходимости инициализировать tax при ее объявлении.

Изменение предложенного Алланом Виндом кода для этого:

#include <stdio.h>

int main () {
    printf("Kindly note down your income here: ");
    double income;
    int rv = scanf("%lf", &income);
    if (rv != 1) {
        printf("scanf failed\n");
        return 1;
    }

    double tax;
    if (income < 2.5) 
        tax = 0;
    else if (income < 5.0) 
        tax = 5;
    else if (income < 10.0) 
        tax = 20;
    else 
        tax = 30;

    // Or even just:
    // double tax = (income <  2.5) ?  0 :
    //              (income <  5.0) ?  5 :
    //              (income < 10.0) ? 20 : 30;

    if (tax)
        printf("You have to pay %.2f tax because your income is %.2f\n", tax, income);
    else
        printf("You have to pay no tax because your income is %.2f\n",  income);
}