需要如果构建建议

Question

我有一个 for 循环，其中放置了几个 if 语句。这些条件的目的是检查数字的整除性，如果该数字可以被 3 整除，则输出一个字符串。如果该数字可以被 5 整除，则将输出另一个字符串。但是，如果该数字可以同时被 3 和 5 整除，则将在其位置输出完全不同的字符串，而不是其他字符串。

这是我的代码：

for (i = 1; i <= file_int; i++){
     if (i % 3 == 0) {
        printf("Hoppity \n");
    }
    if (i % 5 == 0) {
        printf("Hophop \n");
    }
    if (i % 5 == 0 && i % 3 == 0) {
        printf("Hop \n");
    }   
}

如您所见，最后一个条件不太有效。我应该使用什么类型的控制结构？别的？

多谢。

Answer 1

for (i = 1; i <= file_int; i++){
    if (i % 5 == 0 && i % 3 == 0) {
        printf("Five and three\n");
    } else if (i % 3 == 0) {
        printf("Three\n");
    } else if (i % 5 == 0) {
        printf("Five\n");
    } else {
        printf("None of the conditions passed\n");
    }
}

Answer 2

我会使用 else-ifs 并让我们知道

(i % 5 == 0 && i % 3 == 0) <=> (i % 15 == 0)：

for (i = 1; i <= file_int; i++){
  if (i % 15 == 0)
    printf("Hop \n");
  else if (i % 3 == 0)
    printf("Hoppity \n");
  else if (i % 5 == 0)
    printf("Hophop \n"); 
}

当然，除了 for 循环之外，您也可以不使用任何控制结构就可以逃脱：

const char* values[15] = {"Hop \n", "", "", "Hoppity \n", "", 
                          "Hophop \n", "Hoppity \n", "", "", "Hoppity \n", 
                          "Hophop \n", "", "Hoppity \n", "", ""};
for (int i = 1; i <= 100; i++) 
  printf(values[i % 15]);

对于这个例子来说，这个解决方案有点疯狂，但是它展示了如何以不同的方式做事（并且在编写代码时，在一个函数中永远不会有超过一定数量的分支路径（过度热心的编码约定......））。

Answer 3

只是为了它，而不推荐它，因为它可能更难阅读，因为它滥用从 bool 到 int 的转换：

int msg = (i % 3 == 0) + 2*(i % 5 == 0);
switch ( msg ) {
case 3:
   cout << "Multiple of 3 and 5";
case 2: 
   cout << "Multiple of 5";
case 1:
   cout << "Multiple of 3";
}

可以进一步压缩为

const char* msgs[] = { "", "Multiple 3", "Multiple 5", "Multiple 3 and 5" };
cout << msgs[ (i%3==0) + 2*(i%5==0) ];

：当然，这两种解决方案都违背了问题本身，因为它们不是 if 构造，而是避免在第一种情况下使用 if 和 分支 一般在第二种情况下。

Answer 4

另一种解决方案，更接近您的原始代码。尽管 else 解决方案确实更高效（也更优雅）。

for (i = 1; i <= file_int; i++){
    if (i % 3 == 0 && i % 5 != 0) {
        printf("Hoppity \n");
    }
    if (i % 5 == 0 && i % 3 != 0) {
        printf("Hophop \n");
    }
    if (i % 5 == 0 && i % 3 == 0) {
        printf("Hop \n");
    }   
}