在C++類中,有時候會使用到傳值調用(即使用對象實體做參數),當遇到這種情況,可要小心了!尤其是當你所傳值的對象生命周期較長,而非臨時對象(生命周期段)的時候。來看看下面的情況:
#include <iostream>using namespace std;class Text{private:char * str;public:Text(){str = new char[20];::memset(str,0,20);}void SetText(char * str){strcpy(this->str,str);}char * GetText() const{return str;}~Text(){cout << "~Text Destruction" << endl;delete [] str;cout << "~Text Over" << endl;}};void Print(Text str){cout << str.GetText() << endl;}int main(){Text t;t.SetText("abc");Print(t);return 1;}
上面執行的結果是程序崩潰了。原因是:
Print(Text str)在對str進行復制構造的時候,沒有進行深度拷貝;當 Print退出的時候,因為是臨時對象(函數初始時構造),對str進行析構,此時還沒有出現任何問題;但回到main,繼而退出main 的時候,又對t進行析構,但此時t內的str中的內容已經被銷毀。由于對一內存空間實施了兩次銷毀,于是就出現了內存出錯。
解決方法如下:
重寫前拷貝。像以下版本,不同的情況要作出適當的調整:
#include <iostream>using namespace std;class Text{private:char * str;public:Text(){str = new char[20];::memset(str,0,20);}Text(Text &t){str = new char[20];strcpy(str,t.GetText());}void SetText(char * str){strcpy(this->str,str);}char * GetText() const{return str;}~Text(){cout << "~Text Destruction" << endl;delete [] str;cout << "~Text Over" << endl;}};void Print(Text str){cout << str.GetText() << endl;}int main(){Text t;t.SetText("abc");Print(t);return 1;}
此處推薦不使用傳值調用。就像下面書寫如下Print版本:
void Print(Text &str){cout << str.GetText() << endl;}
除非對象內所有的成員讀屬非指針內存內容,那么謹慎使用文章前面提到的用法。
新聞熱點
疑難解答
圖片精選