最も一般的な複数のプログラム

Question

私は、任意の2つの数字の最低公倍数を見つけるプログラムを作ろうとしました。私はそこでの方法のほとんどを得ていますが、私のプログラムは最初のものの代わりに1-1000000のすべての共通の倍数を表示します。どのようにして最初のものだけを印刷させるのですか ？

#include <iostream> 
using namespace std; 

int main() { 
    cout << "Find the lowest common multiple of two numbers, just enter them one after the other" << endl; 
    int firstNum; 
    int secondNum; 
    cin >> firstNum; 
    cin >> secondNum; 
    int i; 

    for (i = 1; i < 1000001; i++) { 

     if (i % firstNum == 0 && i % secondNum == 0) { 

      cout << "these two number's LCM is" << i << endl; 
     } 
    } 



    system("pause"); 
    return 0; 

} 

Answer 1

ループを終了するには、breakを追加できます。あなたのケースでは、あなたif文の最後にそれを追加したい：

for (i = 1; i < 1000001; i++) { 
    if (i % firstNum == 0 && i % secondNum == 0) { 
     cout << "these two number's LCM is" << i << endl; 
     break; 
    } 
} 

Answer 2

あなたの問題は、他の人が言及したようbreak文です。

もっと良い解決策：lcmはC++で標準化されています17！だから、やるだけのことができます：あなたがC++ 17へのアクセス権を持っていない場合は、まだこれはnamespace experimentalですでに提供されています

cout << lcm(firstNum, secondNum) << endl; 

：最初のものを見つけた後http://en.cppreference.com/w/cpp/experimental/lcm

Answer 3

あなたがのために残しておく必要がありますそれで、それは他の値を印刷し続けるのです。

for (i = 1; i < 1000001; i++) { 

if (i % firstNum == 0 && i % secondNum == 0) { 
    cout << "these two number's LCM is" << i << endl; 
    break; 
} 

}