看板 C_and_CPP 關於我們 聯絡資訊
遇到的問題: (題意請描述清楚) 以下程式碼 確定是犯了十誡中的七 也就是有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
VictorTom:1. 加在return後面的code根本就不會執行到.... 11/26 13:31
VictorTom:2. 加在return之前, 回return temp被人家用就GG了.... 11/26 13:31
VictorTom:所以, 如果你這個function沒辦法自己new來的自己delete 11/26 13:32
VictorTom:就是要有另一個對應的function或者caller自己做delete 11/26 13:32
VictorTom:但是這樣做很容易誤用造成問題, 而且沒有一致性與成對性 11/26 13:33
VictorTom:理想的狀況是caller自己產生好空間, 再一起傳進來.... 11/26 13:33
VictorTom:即改成 double * transpose(double *dst, double *src, 11/26 13:34
VictorTom:int row, int col); 這樣子的型式, 然後new/delete的空 11/26 13:34
VictorTom:間在caller自己管好就行了....:) 11/26 13:34
KSJ:感謝樓上...至少讓我知道 還是難逃改____寫的code的命 orz 11/26 13:35
VictorTom:或者, 如果傳入的*matrix是可以被更改的, 那就temp算好 11/26 13:35
VictorTom:直接寫回*matrix, 這樣就可以直接delete temp, 然後連 11/26 13:36
VictorTom:return都不必. 覺得麻煩的話就包出一個matrix的class, 11/26 13:36
VictorTom:把該有的行為method都寫好, 就不會讓code寫的很雜亂沒有 11/26 13:37
VictorTom:章法; 但是代價就是你可能要加/改更多的code....XD 11/26 13:37
KSJ:也是有道理@_@ 我來看看那個matrix是不是能被蓋掉 11/26 13:37