If-construct adviceが必要です

Question

私はいくつかのif文を置いたforループを持っています。これらの条件の目的は、数値の除数をチェックし、その数値が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ソリューションは確かにより効率的（そしてエレガント）です。

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"); 
    } 
} 

Answer 3

私はelse-ifsを使用して

(i % 5 == 0 && i % 3 == 0) <=> (i % 15 == 0)という事実を私たちになるだろう：あなたもまったくfor -loop以外の任意の制御構造を使用せずに逃げることができます。もちろん、

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"); 
} 

：

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]); 

その解決策はわずかですこれは、あなたが物事を別々にする方法を示しています（そして、1つの関数で決まった数の分岐パスを超えることは決してありません。

Answer 4

ちょうどそれのためではなく、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) ]; 

もちろん

、両方のソリューションは、彼らがかの構造ではないので、質問自体に反対しているのではなく、場合の使用を避ける最初のケースではブランチが一般的です。