MFC 调用对话框时，出现“OUT OF memory”错误

[dingqihui83]

自定义类中有指针数组作为成员，
TYPE** m_p;
int m_size;

	m_p = new TYPE*[size];

	for (int k = 0; k < m_size;k++)
	{
		m_p[k] = new TYPE;
	}

其中TYPE 是一个结构体，是对话框中表格的一行的数据的集合。
m_size是对话框数据的行数,是个变量。

在对话框中取某一行的ITEM的值赋给m_p[k]->结构成员时，出现读取错误。应该是自定义类中内存分配的问题。

但不知道具体错在哪里？

[hurryboylqs]

调试一下就知道了

[wmswxpmyl]

内存不足 分配的空间不存在

[dingqihui83]

[Quote=引用 2 楼 wmswxpmyl 的回复:]
内存不足 分配的空间不存在
[/Quote]

我也意识到应该是这个问题，但从分配空间到析构掉，代码逻辑方面没看出来具体错误在哪里。

代码：
自定义类中

class CDeal
{
public:
	CDeal();					 
	~CDeal();
public：
          //.....
         	REC**   m_pPREC;   //REC是一个结构体，m_pPREC[]是一个指针数组，这里用了指针
         int     m_nRows;   //数组元素个数，变量
          //....
};
//CPP
CDeal::CDeal()
{
	m_pPREC  = new REC*[m_nRows];

	for (int k = 0; k < m_nRows;k++)
	{
	       m_pPREC[k] = new REC;
	}
}
//析构的时候是反向的，先循环delete m_pPREC[k] ，然后delete m_pPREC;

对话框类 CMYDlg的函数中给自定义类的成员赋值，

////m_pdeal是CDeal指针，是CMYDlg中的成员
////m_wndGrid1是一个表格类的对象,是CMYDlg中的成员
CMYDlg::func
{
	m_pdeal->m_nRows = m_wndGrid1.GetRowCount();
         //自定义类中的m_nRows 与表格的行数是对应的。

	if(m_wndGrid1.GetRowCount() > 0)
	{
		for (int k = 0; k < m_wndGridLB.GetRowCount(); k++)
		{
			if ( NULL != m_wndGrid1.GetRow(k))
			{
				CBCGPGridRow* pRow = m_wndGrid1.GetRow(k);
			 	m_pdeal->m_pPREC[k]->nHeight = pRow->GetItem(0)->GetValue();
				m_pdeal->m_pPREC[k]->nWidth  = pRow->GetItem(1)->GetValue();
				m_pdeal->m_pPREC[k]->nTimes   = pRow->GetItem(2)->GetValue();
                               //.......
			  }
		}
	}
}

[mayudong1]

构造函数中用到了m_nRows，但没有给它做初始化

[uc_fly]

[Quote=引用 4 楼 mayudong1 的回复:]
构造函数中用到了m_nRows，但没有给它做初始化
[/Quote]
赞+1
补充：虽然你在CMYDlg::func中给m_pdeal->m_nRows赋值，但此时为时已晚，CDeal构造函数早已执行过了。

[TandyT]

[Quote=引用 5 楼 uc_fly 的回复:]
引用 4 楼 mayudong1 的回复:
构造函数中用到了m_nRows，但没有给它做初始化

赞+1
补充：虽然你在CMYDlg::func中给m_pdeal->m_nRows赋值，但此时为时已晚，CDeal构造函数早已执行过了。
[/Quote]

分析的不错

[dingqihui83]

[Quote=引用 5 楼 uc_fly 的回复:]
引用 4 楼 mayudong1 的回复:
构造函数中用到了m_nRows，但没有给它做初始化

赞+1
补充：虽然你在CMYDlg::func中给m_pdeal->m_nRows赋值，但此时为时已晚，CDeal构造函数早已执行过了。
[/Quote]

这个我明白。但是m_nRows本就是个变量，不管怎么初始化，只要它比CMYDlg赋值的数小，都会在CMYDlg::func的赋值过程中出现内存分配问题。

[mayudong1]

[Quote=引用 7 楼 dingqihui83 的回复:]
这个我明白。但是m_nRows本就是个变量，不管怎么初始化，只要它比CMYDlg赋值的数小，都会在CMYDlg::func的赋值过程中出现内存分配问题。
[/Quote]

所以申请空间的代码还是别放在构造函数中了，另写个函数来申请空间，用到的时候调用一下

[dingqihui83]

Quote=引用 8 楼 mayudong1 的回复:]

另写个函数来申请空间，用到的时候调用一下
[/Quote]

你的意思是在自定义类中写一个申请空间的函数，然后在CMYDlg类里面使用指针之前调用它？

但是在这个函数里要不要初始化m_nRows？

如果不要，那应该是把m_nRows作为参数了。

//m_pdeal->m_nRows = m_wndGrid1.GetRowCount();
Init()
{
   m_pPREC  = new REC*[m_nRows];

    for (int k = 0; k < m_nRows;k++)
    {
           m_pPREC[k] = new REC;
    }
}

在Dlg类里用到类指针的话，在哪里调用这个函数呢？在Dlg的构造函数里吗？

[dingqihui83]

我的自定义类中还有其他变量，怎么为它们申请空间？