Re: [問題] 大數字加法

[freesamael (燒賣)]

※ 引述《steve1012 (steve)》之銘言：
: → PkmX:我猜應該是你沒寫copy constructor的問題                     12/25 01:06

HugeInteger HugeInteger::Add( HugeInteger num2 )
{
        HugeInteger temporary;
        ...
        return temporary;
}

這一行會隱含 copy construction。
temporary 在這個 function 結束之後就 destruct 了，
所以實質上會有另一個 compiler 自行加上的 temp 物件，相當於這樣：

        HugeInteger compiler_added_temp(temporary);
        temporary.~HugeInteger();
        return compiler_added_temp;

因為沒有實作 copy contructor，預設的 copy contructor 是把 data member
逐一複製，於是 compiler_added_temp.IntegerArr == temporary.IntegerArr，
所以該陣列會被 delete 兩次。

題外話，你的程式碼變數/functon命名原則很凌亂...。

--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 111.250.29.121
※ 編輯: freesamael      來自: 111.250.29.121       (12/25 04:36)
推 steve1012:真的是這樣 太感謝大大了!
→ steve1012:不好意思寫得很凌亂讓你們看得很累QQ
→ steve1012:另外可以請教一下為什麼會delete兩次嗎?
→ steve1012:我只有寫一個destructor而已~
→ steve1012:還有我命名不好的地方是function定義裡面的變數嗎
→ steve1012:還是整個class的function, variables的命名都不好?
→ james732:譬如說，為什麼Add, Subtract是大寫開頭，其他是小寫開頭
→ WJAider:簡而言之，記住 class 中使用 new 做動態記憶體配置的話
→ WJAider:最好把 constructer, copy constructer,
→ WJAider:assignment operator(operator=), destructor 都自己寫
→ WJAider:不要用預設的，以免發生沒考慮到的狀況
→ freesamael:一般來說變數也不使用大寫開頭，以區隔object和class
→ freesamael:例如 HugeInteger *hugeInteger = new HugeInteger();
→ freesamael:你的data member都是用大寫開頭的，function有大有小
→ freesamael:看起來比較凌亂。常見的data member命名法是加 m 或 _
→ freesamael:比如說 mIntegerArr 或 _integerArr
→ diabloevagto:_要加在後面，不要加在前面
推 LPH66:補充一下, WJ 說的這件事叫做 Rule of three
→ LPH66:"three" 即是指 copy ctor, operator = 和 dtor 三個成員
→ LPH66:只要其中一個是手動定義的  那有很大機會三個都要自己寫
推 steve1012:謝謝你們 真是太感謝你們了