勉強しようとすると消えるバグ

Question

これは、C++でプログラミングするときに私に起こったことの中で最も変わったことです。

これは私のメインのファイルです：

#include <iostream> 
#include "lib/utils.h" 

using namespace std; 

int main(int argc, const char *argv[]) { 
    cout << bin2dec(101000010); 
    return 0; 
} 

、これは/ libにutils.cppです：

#include <iostream> 
#include "utils.h" 

int bin2dec(int bin) { 
    // 101000010 
    int dec; 
    //std::cout << ""; // If you uncomment this, it works. 
    for (int i = 1; bin > 0; i *= 2, bin /= 10) { 
     if (bin % 2 == 1) { 
      dec += i; 
     } 
    } 
    return dec; 
} 

プログラムは警告なしでコンパイルし、実行したとき、それは450を出力します。 450は10進数で101000010ではありません。最初の奇妙なことは、482と322の差が正確に128であるということです。これは変換しようとする2進数で起こります。しかし、本当に奇妙なのは、binとdecの値をforの中に出力しようとすると、関数をデバッグしようとすると、突然正しく動作するようになりました。

基本的には何らかの理由で、関数が返される前に何かstd::coutが返されても動作します。そうしないと、結果に128が追加されます。

私はG ++ 4.6.0を使用して、次のようにコンパイルしています：

g++ -c -D NDEBUG -O2  -o 10.o 10.cpp 
g++ -c -D NDEBUG -O2  -o lib/utils.o lib/utils.cpp 
g++ -o 10 -Wl,-S 10.o lib/utils.o lib/menu.o 

Answer 1

あなたはdecを初期化していません。

あなたはあなたの関数でゼロに decを初期化する必要が

int dec = 0; 

Answer 2

機能のすべての操作がオンになっているので、かなり単に、あなたの問題は、おそらくスタックME-するスタックスマッシングまたはヒープを叫びますスタックこの種の問題に陥ることはありませんが、プログラム内の他の場所で不定な動作を呼び出す必要はありません。

編集：ああ、例えば、decの初期化から始めることができます。

Answer 3

int dec; 

decを作成しませんが、決してそれを（0のような）の初期値を与えます。その後、値を追加して、ランダムな結果に終わります。

Answer 4

：

int dec = 0; 

それを初期化しないことで、それが何であれ、ランダムなCRUDで始まり、変数の場所でメモリ内にあります。そしてcout <<への呼び出しは、その領域に何が荒れているのか、それゆえの変更に影響します。ところで、あなたのコードを試したとき、私は450を得ていませんでした。私はこれの例を示す134514688を得ていた。

Answer 5

小さなバグはdec変数を初期化していないことです。

大きなバグは、コンパイル時に-Wallと-O2を追加しないことです（書き込みするプログラムが複雑になるため、苦労するでしょう）。

-Wallはすべての警告を有効にし、-O2には最適化が必要です（最適化には、初期化されていない変数に関する警告が表示されます）。