作者KSJ (阿真)
看板C_and_CPP
标题[问题] 犯了十诫中的七跟六? c++ new delete 的问题
时间Thu Nov 26 13:25:11 2009
遇到的问题： (题意请描述清楚)
以下程式码 确定是犯了十诫中的七
也就是有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
1F：推 VictorTom:1. 加在return後面的code根本就不会执行到.... 11/26 13:31
2F：→ VictorTom:2. 加在return之前, 回return temp被人家用就GG了.... 11/26 13:31
3F：→ VictorTom:所以, 如果你这个function没办法自己new来的自己delete 11/26 13:32
4F：→ VictorTom:就是要有另一个对应的function或者caller自己做delete 11/26 13:32
5F：→ VictorTom:但是这样做很容易误用造成问题, 而且没有一致性与成对性 11/26 13:33
6F：→ VictorTom:理想的状况是caller自己产生好空间, 再一起传进来.... 11/26 13:33
7F：→ VictorTom:即改成 double * transpose(double *dst, double *src, 11/26 13:34
8F：→ VictorTom:int row, int col); 这样子的型式, 然後new/delete的空 11/26 13:34
9F：→ VictorTom:间在caller自己管好就行了....:) 11/26 13:34
10F：→ KSJ:感谢楼上...至少让我知道 还是难逃改____写的code的命 orz 11/26 13:35
11F：→ VictorTom:或者, 如果传入的*matrix是可以被更改的, 那就temp算好 11/26 13:35
12F：→ VictorTom:直接写回*matrix, 这样就可以直接delete temp, 然後连 11/26 13:36
13F：→ VictorTom:return都不必. 觉得麻烦的话就包出一个matrix的class, 11/26 13:36
14F：→ VictorTom:把该有的行为method都写好, 就不会让code写的很杂乱没有 11/26 13:37
15F：→ VictorTom:章法; 但是代价就是你可能要加/改更多的code....XD 11/26 13:37
16F：→ KSJ:也是有道理@_@ 我来看看那个matrix是不是能被盖掉 11/26 13:37