Question

指针问题求教

void CdemoView::OnDraw(CDC* pDC)
{
CdemoDoc* pDoc = GetDocument();
ASSERT_VALID(pDoc);
if (!pDoc)
return;
// CSize sizeTotal;
// TODO: calculate the total size of this view
// sizeTotal.cx =800; sizeTotal.cy =GetDocument()->m_array.GetSize();
//SetScrollSizes(MM_TEXT, sizeTotal);

CString str,strdr;
CPoint point(0,0);

int nCount=pDoc->m_data->m_stock.GetSize();
CMap<DWORD,DWORD,drdata,drdata&>* pMapTemp=new CMap<DWORD,DWORD,drdata,drdata&>;

for(int i=0;i<nCount;i++)
{
str.Format(TEXT("%s"),pDoc->m_data->m_stock.GetAt(i).code);
pDC->TextOut(20,point.y*20,str);

if(0!=pDoc->m_drmap->Lookup(str,(void*&)pMapTemp))
{
int x=0;

POSITION pos=pMapTemp->GetStartPosition();
while(pos!=0)
{
DWORD key;
drdata value;

pMapTemp->GetNextAssoc(pos,key,value);
CString str;
str.Format(TEXT("%d"),key);

pDC->TextOut(20+(x+1)*100,point.y*20,str);
x++;
}
}
point.y++;

}

ReleaseDC(pDC);
//delete pMapTemp;加上delete pMapTemp;重绘的时候就会报错，顺便推荐本这方面的书吧，近日老有这种问题

// TODO: add draw code for native data here
}

Answer 1

pDoc->m_drmap->Lookup(str,(void*&)pMapTemp))这里错了。

MSDN上CMap::Lookup的定义是：
BOOL Lookup(
ARG_KEY key,
VALUE& rValue
) const;
意思是，将按key查找到的值，赋值给rValue。

这样，pMapTemp的值就被修改掉了。注意此时，pMapTemp将不再指向刚刚new出来的那块地址了（这个new出来的内存泄露了)。
而当你在下面delete pMapTemp的时候，delete的其实是pDoc->m_drmap负责管理的内存。

改正方法很简单，只要将
CMap<DWORD,DWORD,drdata,drdata&>* pMapTemp=new CMap<DWORD,DWORD,drdata,drdata&>;

改成
CMap<DWORD,DWORD,drdata,drdata&>* pMapTemp;

即可。
delete pMapTemp;也是不需要的。

Answer 2

叉一句：
这VC++ 学的是累。我学C++BUILDER，简单方便多了。

追问

VB更简单

追答

VB简单，但目标程序太差了。C++BUILDER 可不是VB

Answer 3

ReleaseDC(pDC)是不需要的，而 delete pMapTemp 看上去没错

追问

首次运行没问题，重绘就报错

追答

在你的代码中有这样一句：
if(0!=pDoc->m_drmap->Lookup(str,(void*&)pMapTemp))

将pMapTemp的指针变量的引用传入 Lookup，那有没有在 Lookup函数中改变指针？