[原創(chuàng)文章歡迎轉(zhuǎn)載，但請保留作者信息]
Justin 于 2009-12-14
55條軍規(guī)里，這第17條算是單看標(biāo)題就能記住的規(guī)矩之一。
說的東西其實(shí)也比較簡單：
processWidget(std::tr1::shared_ptr<Widget>(new?Widget),?priority()); 這行語句有問題，這個(gè)復(fù)雜的參數(shù)表包含了三個(gè)動作：
-
new 一個(gè) Widget
-
用new的Widget做為參數(shù)執(zhí)行share_ptr的構(gòu)造函數(shù)
-
執(zhí)行priority
C++的某個(gè)編譯器可能為了效率而自作主張，導(dǎo)致這三個(gè)動作的執(zhí)行順序是不確定的！因此上面的動作執(zhí)行順序可能是這樣的：
-
new 一個(gè) Widget
-
執(zhí)行priority
-
用new的Widget做為參數(shù)執(zhí)行share_ptr的構(gòu)造函數(shù)
這個(gè)時(shí)候如果priority的執(zhí)行出錯(cuò)而引發(fā)異常，就會發(fā)生內(nèi)存泄漏(Memory Leak)，因?yàn)閚ew出來的Widget再也無法跟蹤了。
而解決方法也很簡單，不要妄圖一行寫完所有程序，分開來老老實(shí)實(shí)寫就是了：
std::tr1::shared_ptr<Widget>?pw(new?Widget);
processWidget(pw,?priority());
|