我怎样才能更改下面的代码,这样我可以使用new
和delete
,而不是malloc
和free
?
#include <iostream>
#include <cstdlib>
#include <cstdio>
using namespace std;
class myclass{
private:
float **m_R;
public:
myclass();
~myclass();
float **getR(void);
void setR(void);
};
myclass::myclass()
{
// first allocate rows (for pointers)
m_R = (float**)malloc(3*sizeof(float));
// next allocate columns for float values
for(int i=0;i<3;i++)
*(m_R+i) = (float*)malloc(3*sizeof(float));
}
myclass::~myclass()
{
// first free memory allocated by columns
for(int i = 0; i < 3; i++)
{
free(m_R[i]);
}
// next free memory allocated by rows
free(m_R);
}
void myclass::setR(void)
{
for(int i=0;i<3;i++)
{
for(int j=0;j<3;j++)
{
m_R[i][j] = 10*i+j;
//cout << m_R[i][j] << ", ";
}
//cout << "\n";
}
}
float **myclass::getR(void)
{
return m_R;
}
int main () {
myclass obj;
obj.setR();
for(int i=0;i<3;i++)
{
for(int j=0;j<3;j++)
{
printf("%02d, ",(int)obj.getR()[i][j]);
}
cout << "\n";
}
return 0;
}
EDIT1:请注意,我必须使用的功能(不是我写的),这需要float**
作为参数,我没有选择(如vector
)比使用float**
。
EDIT2:功能是从写为下面的邻近查询包(PQP):
int
PQP_Distance(PQP_DistanceResult *result,
PQP_REAL R1[3][3], PQP_REAL T1[3], PQP_Model *o1,
PQP_REAL R2[3][3], PQP_REAL T2[3], PQP_Model *o2,
PQP_REAL rel_err, PQP_REAL abs_err,
int qsize = 2);
-
T* a = (T*)malloc(sizeof(T))
成为new T
。 -
T* b = (T*)malloc(N * sizeof(T))
变为new T[N]
-
free(a)
变成delete a
。 -
free(b)
变成delete[] b
所以,你得到:
myclass::myclass()
{
// first allocate rows (for pointers)
m_R = new float*[3];
// next allocate columns for float values
for(int i=0;i<3;i++)
*(m_R+i) = new float[3];
}
myclass::~myclass()
{
// first free memory allocated by columns
for(int i = 0; i < 3; i++)
{
delete[] m_R[i];
}
// next free memory allocated by rows
delete [] m_R;
}
请注意,这其实不是很理想。 如果你想的3x3矩阵,你关在一个块分配9辆花车更好。
另外请注意,你的C不正确。
m_R = (float**)malloc(3*sizeof(float));
应该
m_R = (float**)malloc(3*sizeof(float*));
您的代码可能是“作品”,因为你正在编译为32位,其中float
和float*
4个字节。 在64位的构建, float*
为8个字节。
老实说,虽然,因为你的尺寸是固定的,小的,你应该存储在对象本身的一切:
class myclass{
private:
float m_R[3][3];
public:
myclass() {}
~myclass() {}
void setR(void);
float* operator[](unsigned i) { return m_R[i]; }
const float* operator[](unsigned i) const { return m_R[i]; }
};
void myclass::setR(void)
{
for(int i=0;i<3;i++)
{
for(int j=0;j<3;j++)
{
(*this)[i][j] = 10*i+j;
//cout << (*this)[i][j] << ", ";
}
//cout << "\n";
}
}
int main () {
myclass obj;
obj.setR();
for(int i=0;i<3;i++)
{
for(int j=0;j<3;j++)
{
printf("%02d, ",(int)(obj[i][j]));
}
cout << "\n";
}
return 0;
}
这就是我平时做的,所以我得到的[] []可读性与对象存储的效率。
假设使用标准C ++库是一种选择,你应该使用
std::vector<std::vector<float> > m_R;
而不是float**
。 没有绝对的缺点,你会得到许多便利的东西是免费的。 例如,你将能够找到矢量的大小和它的每个维度没有通过一个侧面的一对数字,或在一些假设的编码。 您将能够在所有分配没有循环,并删除不需要编写代码。
如果这不是一个选项,可以更换malloc
/ free
用new[]
/ delete[]
如下所示:
// Creating
float **m_R = new float*[10];
for (int i = 0 ; i != 10 ; i++) {
m_R[i] = new float[20];
}
// Deleting
for (int i = 0 ; i != 10 ; i++) {
delete[] m_R[i];
}
delete[] m_R;
如果你想在C(因此,没有班组长)或C ++(因此,没有纯C,当C ++库中存在)进行编程,你应该决定。
现在你只要C +一些类。
对于你做了什么(谁包装了一个二维数组固定的类)更合适的方式可以是这样的:
#include <iostream>
#include <iomanip>
#include <cassert>
class myclass
{
public:
mycalss() :m() {}
float& at(size_t r, size_t c)
{ return m[r][c]; }
const float& at(size_t r, size_t c) const
{ return m[r][c]; }
void setR()
{
for(int i=0;i<3;i++)
for(int j=0;j<3;j++)
m[i][j] = 10*i+j;
}
private:
float m[3][3];
};
int main ()
{
using namespace std;
myclass obj;
obj.setR();
for(int i=0;i<3;i++)
for(int j=0;j<3;j++)
cout << setw(2) << obj.at(i,j) << endl;
return 0;
}
请注意,有没有必要使用动态内存。
如果要使用动态内存(可能是因为数组的大小应大些),你可以依靠的std ::向量作为容器:
#include <iostream>
#include <iomanip>
#include <vector>
#include <cassert>
class myclass
{
public:
mycalss(unsigned rows_, unsigned cols_) :m(), rows(rows_), cols(cols_)
{ m.resize(rows_*cols_); }
float& at(size_t r, size_t c)
{ return m[r*cols+c]; }
const float& at(size_t r, size_t c) const
{ return m[r*cols+c]; }
void setR()
{
for(int i=0;i<rows;i++)
for(int j=0;j<cols;j++)
at(i,j) = 10*i+j;
}
private:
std::vector<float> m;
size_t rows, cols;
};
int main ()
{
using namespace std;
static const size_t R=4, C=4;
myclass obj(R,C);
obj.setR();
for(int r=0;r<R;r++)
for(int c=0;c<C;c++)
cout << setw(2) << obj.at(r,c) << endl;
return 0;
}