文字列のコピーの関数の結果は

Question

私は、次の機能を持たをallocated`いませんでした解放されています。

---

（私はこの機能を堅牢にしようとしているとして）私は、私はそれを使用する方法が重要であるべきとは思わないが、ここでいくつかの例は以下のとおりです。

CD::CD(char * s1, char * s2, int n, double x) 
{ 
    stringcopy(performers, s1); 
    stringcopy(label, s2); 
    selections = n; 
    playtime = x; 
} 

と

私は残念ながら

CD::CD(const CD & d) 
{ 
    stringcopy(performers, d.performers); 
    stringcopy(label, d.label); 
    selections = d.selections; 
    playtime = d.playtime; 
} 

など

---

、関数を使用すると、次のエラーメッセージが表示されます。pointer being freed was not allocated

if (to != 0) { delete [] to; }になると思います。

---

なぜこの行は非割り当てられたメモリを割り当て解除から保護しないのですか？

Answer 1

あなたがここに

をやっている

if (to != 0) { delete [] to; } 
to = new char[size]; 

、新たにそれを割り当て、そこに文字列を格納し、これにローカル変数toポイントをメモリを解放しています。

しかし、この新しいローカルメモリアドレス（to1と呼ぶ）は、関数から返されないため、外部に公開されることはありません。関数getはtoのコピーです。これを修正するには、toをダブルポインタにする必要があります。ポインタへの参照。