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