なぜこのコードが上記の場所でハングしますか？

Question

これは、C++演算子のオーバーロードを調べるためにコーディングした素朴な例です。私がそれを実行すると、コードは文c = a + bでハングします。コントロールが決して届かないc.display();

cout << ptr << '\n';を代入演算子のオーバーロードされた関数に置くと、デバッグの一部として、HelloWorldが出力されるため、stringは不正な形式ではないようです。

なぜそれがハングするのですか？私は何が欠けているのですか？

class mystring 
{ 
    char *ptr; 

    public: 

    mystring(char *str = "") 
    { 
     ptr = new char[strlen(str) + 1]; 
     strcpy(ptr,str); 

    } 

    mystring operator +(mystring s) 
    { 
     char *str = new char[strlen(ptr) + strlen(s.ptr) + 1];//where should this memory be freed 
     strcpy(str,ptr); 
     strcat(str,s.ptr); 
     return mystring(str); 
    } 

    void operator =(mystring s) 
    { 
     strcpy(ptr,s.ptr); 

     //cout << ptr << '\n'; \\Debug - this prints out HelloWorld but still hangs 
    } 

    void display() 
    { 
     cout << ptr << '\n'; 
    } 

    ~mystring() 
    { 
     delete [] ptr; 
    } 
}; 

int main() 
{ 
    mystring a="Hello",b="World",c; 

    c = a + b; 

    c.display(); 

    getchar(); 

} 

EDIT：コンパイラ：MS-のVisual C++ 2010 Expressの/ Windowsの。

Answer 1

あなたが得ているのはメモリエラーだと思います。このライン：

c = a + b; 

次ん：

c.constructor() 
c.operator=(a.operator+(b)); 

とあなたのオペレータ=必要なものメモリ

void operator =(mystring s) 
{ 
    // ptr is allocated enough memory for "", i.e. one byte 
    strcpy(ptr,s.ptr); // copying more than one byte into one byte array 
    //cout << ptr << '\n'; // this works, but you've trashed memory with the strcpy 
} // stack might be corrupted here, depends where this is, so execution can end up anywhere 

の割り当てに失敗したのである：

void operator = (mystring &s) // reference! 
{ 
    delete [] ptr; 
    ptr = new char [strlen (s.ptr + 1)]; 
    strcpy (ptr, s.ptr); 
} 

Answer 2

operator=が壊れています。 strcpyを実行する前に十分な（または任意の）メモリを適切に割り当てていません。これは未定義の動作につながります。

Answer 3

のchar * STR =新しいCHAR [STRLEN（PTR）+ STRLEN（s.ptr）+ 1]; //このメモリは、デストラクタで解放

メモリを解放する必要がある場合。

オペレータに=あなたが割り当てられたメモリを解放し、再び

Answer 4

オペレータがそれを割り当てる必要があります=古いバッファを解放し、あなたは、コンストラクタで行うようにして新しいものを割り当てる必要があります。

Answer 5

コードには、c.display()を打つ前にブロックするような明白な問題はないようですが、のように見えます。のようになります。

操作cout << ptr << '\n';はストリームをフラッシュしません。これは、出力が後でキャッシュされる可能性があることを意味します。プログラム全体でが基本的にで完了し、ユーザーがgetchar()に文字を入力するのを待っている場合があります。

デバッガでコードを実行するか、出力をcout << ptr << endl;に変更してみてください。

ところで、あなたはoperator+にメモリをリークしているので、おそらくそれを処理する必要があります。私はあなたがすでにそれを行う方法について尋ねたことを知っています、あなたは提案された解決策を好まなかった（または理解した）かもしれないが、それをそのまま残すことは解決策ではないかもしれません。

operator=は、完全な文字列を保持するのに十分な領域がないことを保証していません。つまり、未定義の動作を引き起こす可能性があります。