[問題] 犯了十誡中的七跟六? c++ new delete 的問題

看板C_and_CPP (C/C++)作者 (阿真)時間16年前 (2009/11/26 13:25), 編輯推噓1(1015)
留言16則, 2人參與, 最新討論串1/1
遇到的問題: (題意請描述清楚) 以下程式碼 確定是犯了十誡中的七 也就是有new 但沒有delete 這是一個把矩陣轉置的函式程式碼 // compute transpose of a matrix double * transpose(double *matrix, int row, int col) { int i, j; // double *temp = doubleAlloc(row * col); double *temp = new double[row*col]; for (i=0; i<row; i++) for (j=0; j<col; j++) temp[j*row + i] = matrix[i*col+j]; return temp; } 我希望能加入delete 可是加在return之前 感覺怪怪的 加在return之後 又沒用 感覺上是犯中十誡中的六 可是裡面是要 "自動變數" new算嗎?? 我想知道 是程式不該這樣寫 還是有相對應的delete方法 想請教大家 感謝 開發平台: (例: VC++ or gcc/g++ or Dev-C++, Windows or Linux) vc6 -- ※ 發信站: 批踢踢實業坊(ptt.cc) ◆ From: 140.112.63.180

11/26 13:31, , 1F
1. 加在return後面的code根本就不會執行到....
11/26 13:31, 1F

11/26 13:31, , 2F
2. 加在return之前, 回return temp被人家用就GG了....
11/26 13:31, 2F

11/26 13:32, , 3F
所以, 如果你這個function沒辦法自己new來的自己delete
11/26 13:32, 3F

11/26 13:32, , 4F
就是要有另一個對應的function或者caller自己做delete
11/26 13:32, 4F

11/26 13:33, , 5F
但是這樣做很容易誤用造成問題, 而且沒有一致性與成對性
11/26 13:33, 5F

11/26 13:33, , 6F
理想的狀況是caller自己產生好空間, 再一起傳進來....
11/26 13:33, 6F

11/26 13:34, , 7F
即改成 double * transpose(double *dst, double *src,
11/26 13:34, 7F

11/26 13:34, , 8F
int row, int col); 這樣子的型式, 然後new/delete的空
11/26 13:34, 8F

11/26 13:34, , 9F
間在caller自己管好就行了....:)
11/26 13:34, 9F

11/26 13:35, , 10F
感謝樓上...至少讓我知道 還是難逃改____寫的code的命 orz
11/26 13:35, 10F

11/26 13:35, , 11F
或者, 如果傳入的*matrix是可以被更改的, 那就temp算好
11/26 13:35, 11F

11/26 13:36, , 12F
直接寫回*matrix, 這樣就可以直接delete temp, 然後連
11/26 13:36, 12F

11/26 13:36, , 13F
return都不必. 覺得麻煩的話就包出一個matrix的class,
11/26 13:36, 13F

11/26 13:37, , 14F
把該有的行為method都寫好, 就不會讓code寫的很雜亂沒有
11/26 13:37, 14F

11/26 13:37, , 15F
章法; 但是代價就是你可能要加/改更多的code....XD
11/26 13:37, 15F

11/26 13:37, , 16F
也是有道理@_@ 我來看看那個matrix是不是能被蓋掉
11/26 13:37, 16F
文章代碼(AID): #1B3X4wvD (C_and_CPP)
文章代碼(AID): #1B3X4wvD (C_and_CPP)