使用 if 语句代替 switch case 的默认值

Question

在打开 switch case 之前使用 if 语句并避免使用 default 关键字是否正确？ 例如，我想要一个程序，它将月份的数字作为输入并告诉你它的名称。这是使用switch case 语句的代码：

#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 关键字中。对我来说这似乎是正确的，因为我想在程序执行 switch case 语句之前验证该值。你怎么看，会是正确的吗？如果我没有问太多，请你告诉我为什么？

带有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 子句中为值加上括号既没用也不惯用。删除它们。
• 在switch 语句的最后一个子句的末尾添加break; 语句更安全，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

实际上，我会说您的代码在使用 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");
}

也就是说，回到你最初的问题，在相同数据上混合流控制结构，如if/else 和switch/case 通常是不好的。这样做的原因是它不必要地复杂，所以要么坚持使用最初尝试中的单个 switch 语句，要么使用我上面概述的方法。

Answer 6

我的个人观点，可能并不完美。

拥有default 案例是首选，因为它可以捕获任何下落不明的案例。人们似乎依赖它，因此，gcc 中的警告标志 -Wswitch-default。 因此，有时您可能会在没有 default 大小写的情况下收到一些编译器警告。

在您的特定情况下，这很好，但使用 default 案例是 good practice。