奇妙なC++のエラーは、多分

Question

私のコードは次のようであるlong long型に相対的です。 しかし、2つの数字14159572 63967072を入力すると、負の数-527892768が出力されますが、正解は226436590403296になります。 出力が32ビットにカットされているようです。 ので、私は変数temp_gcdを印刷し、4.Iは、それが正しい答えを出力

long long result = abproduct/4; 

に表現

long long result = abproduct/temp_gcd; 

を変更しました。 aとbの両方intあるので

Answer 1

これはオペランドのデータ型であるため、intの乗算を実行しています。そのデータ型を変更します。キャストは常に政治家や外交官が失敗した後にのみ（戦争に行くような、最後の手段でなければなりません：つまり、キャストを使用するようにアドバイスを取ってはいけない

long long lcm(long long a, long long b) 

に

long long lcm(int a, int b) 

を変更します）。

関数の署名を変更しない場合は、a*bを1LL*a*bに置き換えて変換を強制することができます。しかし、私はここでそれをお勧めしません。適切なデータ型を使用するのが正しい方法です。

はさらに、コード

long long abproduct = a*b; 
long long result = abproduct/temp_gcd; 

は不用意結果がオーバーフローすることができるケースを追加します。

long long result = a*(b/temp_gcd); 

がこの情報を破棄することができないことに注意し、それを減らすために（一般的に、しかし、あなたは、整数の除算廃棄情報について注意する必要があります）。

Answer 2

問題がラインlong long abproduct = a*b;

上にある、それらの任意の操作は、この場合にオーバーフローをもたらすであろう、intをもたらすであろう。計算を行うときにaまたはbをlong longにキャストしてみてください。

long long abproduct = (long long)a * b;

Answer 3

は@ MahlerFiveの答えを編集したくなかった - あまりにも動作するはずです。私はaとbをlong long likeにキャストします：long long abproduct = static_cast<long long>(a)*static_cast<long long>(b)。