直角三角形を生成するC++

Question

私は入力に続いて与えられる3つの辺を生成するプログラムを作ろうとしています：最も長い許容斜辺と必要な三角形の数。辺は整数だけです。 私が書いたプログラムはちょうど私の上にハングアップし、出力を返しません。 あなたがなぜ私をdownvoteするのか説明してください。

#include <iostream> 
#include <cmath> 
#include <cstdlib> 

int generator(int number, int hypoth){ 

    int a,b,c; 

    while (number>0){ 
     c=rand()%(hypoth-1)+1; 
     for (a=1;a<hypoth-2;a++){ 
      for (b=1;pow(a,2)+pow(b,2)<=pow(c,2); b++){ 
       if (pow(a,2)+pow(b,2)==pow(c,2)){ 
        std::cout<<"sides: "<<a<<" "<<b<<" "<<c<<std::endl; 
        number--; 
       } 
      } 
     } 
    } 
    return 0; 
} 

int main(){ 
    int triangle_number, hypothenuse; 
    std::cout << "How many triangles to generate? "; 
    std::cin >> triangle_number; 
    std::cout << "How long is max hypothenuse?"; 
    std::cin >> hypothenuse; 
    generator(triangle_number, hypothenuse); 
    return 0; 
} 

私のアルゴリズムを改善する必要があると思う場合は、私に正しい方向を教えてください。 ありがとうございます。

Answer 1

あなたの提供したコードは私のマシンで正常に動作します.1と6を入力すると出力はsides: 3 4 5になります。

ただし、この問題はおそらく次の行から発生します。pow(a,2)+pow(b,2)==pow(c,2)。 powはdoubleを返します。平等のために浮動小数点数を比較することは滑りやすく、実際には決して良い考えではありません。なぜなら、それはわずかな量ではなく、誤っている可能性が高いからです。

a*a + b*b == c*c（とすぐ上のforループ内の条件はa*a + b*b <= c*c）に置き換えてください。