Использование инструкции if вместо значения по умолчанию для корпуса коммутатора

Question

Правильно ли использовать инструкцию if перед открытием switchcase и избежать использования ключевого слова default? Например, мне нужна программа, которая принимает в качестве входного числа месяц и сообщает свое имя. Это код, который использует оператор switchcase:

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

main() { 
    int month; 
    printf("Insert the number of the month and the program will return its name"); 
    scanf("%i", &month); 
    switch (month) { 
     case (1): 
     printf("The month is January"); 
     break; 
     case (2): 
     printf("The month is February"); 
     break; 
     case (3): 
     printf("The month is March"); 
     break; 
     case (4): 
     printf("The month is April"); 
     break; 
     case (5): 
     printf("The month is May"); 
     break; 
     case (6): 
     printf("The month is June"); 
     break; 
     case (7): 
     printf("The month is July"); 
     break; 
     case (8): 
     printf("The month is August"); 
     break; 
     case (9): 
     printf("The month is September"); 
     break; 
     case (10): 
     printf("The month is October"); 
     break; 
     case (11): 
     printf("The month is November"); 
     break; 
     case (12): 
     printf("The month is December"); 
     break; 
     default: 
     printf("not valid"); 
    } 
    system("pause"); 
    return 0; 
} 

Тогда мне было интересно, если я могу поставить условие недействительности в if заявления, а не в default ключевом слове. Мне кажется правильным, поскольку я хочу проверить значение до того, как программа выполнит оператор switchcase. Как вы думаете, было бы правильно? Если я не прошу слишком много, не могли бы вы рассказать мне, почему?

Код с if высказыванием:

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

main() { 
    int month; 
    printf("Insert the number of the month and the program will return its name"); 
    scanf("%i", &month); 
    if (month >= 1 && month <= 12) { 
     switch (month) { 
      case (1): 
      printf("The month is January"); 
      break; 
      case (2): 
      printf("The month is February"); 
      break; 
      case (3): 
      printf("The month is March"); 
      break; 
      case (4): 
      printf("The month is April"); 
      break; 
      case (5): 
      printf("The month is May"); 
      break; 
      case (6): 
      printf("The month is June"); 
      break; 
      case (7): 
      printf("The month is July"); 
      break; 
      case (8): 
      printf("The month is August"); 
      break; 
      case (9): 
      printf("The month is September"); 
      break; 
      case (10): 
      printf("The month is October"); 
      break; 
      case (11): 
      printf("The month is November"); 
      break; 
      case (12): 
      printf("The month is December"); 
      break; 
      default:; 
     } 
    } else { 
     printf("not valid"); 
    } 
    system("pause"); 
    return 0; 
} 

Спасибо и извините за мой английский, но это не мой родной язык. Дайте мне знать, если я не объяснил себя четко.

Answer 1

Оба подхода являются действительными и эквивалентны за исключением некоторых деталей:

• способ определения main полностью устарели. Он не будет компилироваться с компилятором C99 в строгом режиме. Используйте int main(void) или int main(int argc, char *argv[]).
• Проверьте возвращаемое значение scanf(). Если вы набираете то, что не может быть проанализировано как число, scanf() вернет 0 или даже EOF, если вы закроете входной поток, а остальная часть кода будет использовать неинициализированное значение month.
• Не полезно и не идиоматично вставлять в скобки значение в статьях case. Удалить их.
• Безопаснее всегда добавлять оператор break; в конце последнего предложения в инструкции switch, default или нет. Если вы когда-нибудь добавите другое предложение, вы не рискнете его опустить.
• Извлеките пункт default:; из второго кода, это бесполезно и удивительно.

Причина, по которой второй подход может быть более указан, заключается в том, что вы хотите сделать что-то совершенно другое, если вход за пределами допустимого диапазона, например перезапуск операции ввода. if заявление позволит правильно разделить эти ситуации, в то время как только с помощью предложения default может быть не так уместно:

for (;;) { 
    int n, month; 
    printf("Enter a number between 1 and 12: "); 
    n = scanf("%d", &month); 
    if (n == EOF) { 
     printf("Unexpected end of file\n"); 
     exit(1); 
    } 
    if (n != 1) { 
     printf("Invalid input\n"); 
     scanf("%*[^\n]%*c"); /* flush the pending input */ 
     continue; 
    } 
    if (month >= 1 && month <= 12) { 
     switch (month) { 
      case 1: 
      printf("The month is January\n"); 
      break; 
      ... 
      case 12: 
      printf("The month is December\n"); 
      break; 
     } 
     handle_month(month); // perform other tasks 
     break; 
    } else { 
     printf("Invalid month number\n"); 
    } 
} 

Answer 2

Почему бы не быть верным? Это. default просто безопаснее: если вы меняете cases, но не можете изменить if, вы попадете в беду.

Answer 3

Да, вы можете использовать if заявление, как вы описываете. Однако я не вижу никакой пользы от этого.

Answer 4

Оригинал был лучше. Компилятор выполнит почти такую ​​же работу, хотя второй будет немного медленнее из-за предсказания ветвления.

Но основная причина заключается в ремонтопригодности кода. При добавлении или удалении одного значения вам нужно изменить два места с помощью оператора if, а только с switch.

Answer 5

Мой личный взгляд, который может быть не идеальным.

Имея default случай предпочтительнее, поскольку он ловит неучтенные случаи. Люди, похоже, полагаются на него, следовательно, предупреждающий флаг -Wswitch-default в gcc. Итак, иногда вы можете получить предупреждение о компиляторе без default.

В вашем конкретном случае, это нормально, но good practice использовать default.

Answer 6

Фактически, я бы сказал, что ваш код нарушает лучшие практики при использовании операторов switch. Скорее всего, попробуйте следующее:

if (month >= 1 && month <= 12) { 
    char const*const names[] = { 
     "January", 
     "February", 
     ... 
     "November", 
     "December", 
    }; 
    printf("The month is %s\n", names[month -1]); 
} else { 
    printf("not valid\n"); 
} 

Тем не менее, возвращаясь к вашему вопросу ininital, смешиваясь структуры управления потоком, как if/else с switch/caseна тех же данных в обычно плохо. Причина этого в том, что это излишне сложно, поэтому придерживайтесь либо одного оператора switch, как в вашей первоначальной попытке, либо используйте подход, который я набросал выше.