逆整数キャッチオーバーフローC++

Question

こんにちは私は単純な逆整数演算をC++で試しています。以下のコード：

#include <iostream> 
#include <algorithm> 
#include <climits> 

using namespace std; 

class RevInteger { 

public: 
    int reverse(int x) 
    { 
     int result = 0; 
     bool isNeg = x > 0 ? false : true; 
     x = abs(x); 

     while (x != 0) 
     { 
      result = result * 10 + x % 10; 
      x = x/10; 
     } 

     if (isNeg) 
      result *= -1; 

     if (result > INT_MAX || result < INT_MIN) 
      return 0; 
     else 
      return (int)result; 
    } 
}; 

私は1534236469として入力します。私はそれが私にいくつかの迷惑な値を返す代わりに、私に0を返すようにしたい。私のプログラムで何が間違っていますか。また、私は目的のためにclimits libを使用しようとしています、同じことを行う簡単な方法はありますか？

Answer 1

最も簡単な方法は、結果をintの代わりにlong longを使用することで、最後にオーバーフローをチェックしてください。

long long result = 0; 
/* the rest of your code */ 
return (int)result; // Now the cast is necessary; in your code you could do without it 

別のアプローチは、文字列にintを変換し、それを逆にして、使用することですあなたは、整数内にとどまる必要がある場合は、アプローチはintermをチェックすることです

int rev(int n) { 
    auto s = to_string(n); 
    reverse(s.begin(), s.end()); 
    try { 
     return stoi(s); 
    } catch (...) { 
     return 0; 
    } 
} 

：戻ってそれを変換してみてください、そして道（demo）に沿って問題をキャッチする標準ライブラリediate結果10を乗じ、また、添加後、オーバーフローをチェックする前に：

while (x != 0) { 
    if (result > INT_MAX/10) { 
     return 0; 
    } 
    result = result * 10 + x % 10; 
    if (result < 0) { 
     return 0; 
    } 
    x = x/10; 
} 

Answer 2

@daskblinkenlightによって示唆されるように。結果を長く変更し、最後に型キャストすると問題が解決します。

ワーキングクラス：

class intReverse { 

public: 
int reverse(int x) { 

    long long result = 0; // only change here 
    bool isNeg = x > 0 ? false : true; 
    x = abs(x); 
    while (x != 0) { 
     result = result * 10 + x % 10; 
     x = x/10; 
    } 
    if (isNeg) { 
     result *= -1; 
    } 
    if (result > INT_MAX || result < INT_MIN) 
    { 
     return 0; 
    } 
    else 
    { 
     return (int) result; 
    } 

    } 
};