2 qq 20880415 qq_20880415 于 2016.03.09 19:19 提问

关于内存delete报错的问题,以下代码在最后delete的时候 10C

(safe_deletevec是宏定义,如果指针不等于NULL,就删除并置NULL),就会报错,说是写入冲突,应该没有越界,可是就是想不通为什么出错,麻烦各位给些建议

short Nobone=m_sMinValue+50;
int m_nValueMax=0;
for(int i = 0; i < m_nOriImagePiece; i++)
{
int k = i*m_nWidth*m_nHeight;
for (int m = 0; m < m_nHeight; m++)
{
int v = m*m_nWidth + k;
for (int n = 0; n < m_nWidth; n++)
{
int a = ((short*)m_pProcessArray->GetDcmArray()[i]->GetData())[m_nWidth*m + n];
if (a>Nobone)
{
short m_nRadi = EDTdate[v+n];
if (m_nRadi>m_nValueMax)
m_nValueMax = m_nRadi;
}
}
}
}

m_nValueMax = m_nValueMax + 1;
short* occurs = new short[m_nValueMax];
memset(occurs , 0 ,sizeof(short)*m_nValueMax);

for(int i = 0; i < m_nOriImagePiece; i++)
{
    int k = i*m_nWidth*m_nHeight;
    for (int m = 0; m < m_nHeight; m++)
    {
        int v = m*m_nWidth + k;
        for (int n = 0; n < m_nWidth; n++)
        {
            int a = ((short*)m_pProcessArray->GetDcmArray()[i]->GetData())[m_nWidth*m + n];
            if (a>Nobone)
            {
               short m_nRadi = EDTdate[v+n];
               occurs[m_nRadi] = 1;
            }
        }
    }
}

numRadii = 0;
for (int i=0;i<m_nValueMax;i++)
{
    if (occurs[i])
        numRadii++;
}
//make an index of the distance-squared values
short* distSqIndex = new short[m_nValueMax];
memset(distSqIndex,0,sizeof(short)*m_nValueMax);
short* distSqValues = new short[numRadii];
memset(distSqValues,0,sizeof(short)*numRadii);
int indDs = 0;
for (int i=0;i<m_nValueMax;i++)
{
    if (occurs[i])
    {
        distSqIndex[i] = indDs;
        distSqValues[indDs++]=i;
    }
}
Safe_DeleteVec(occurs);

4个回答

cxsmarkchan
cxsmarkchan   2016.03.09 19:37

检查两个地方:
1. short m_nRadi = EDTdate[v+n];得到的m_nRadi是否可能越界
2. occurs指向数组,删除时应该是delete[] occurs而不是delete occurs
希望对你有帮助。

cxsmarkchan
cxsmarkchan 回复小看我没门: 是否能具体定位到出错的代码?感觉代码不会在delete那里报错Orz...
接近 2 年之前 回复
qq_20880415
qq_20880415 删除是delete[] occurs的,越界之前也检查过了,其他还能想到什么地方可能出错吗?真的很困惑,明明没错的
接近 2 年之前 回复
WinsenJiansbomber
WinsenJiansbomber   2016.03.09 20:08

不能把delete理解为删指针啊!delete是内存回收,它需要一个指向内存地址的指针,这样内在管理系统就会标记所目标内存是可用的。但是不能对同一个地址连续执行两次。

qq_20880415
qq_20880415 我清楚的,不好意思表达有问题,我并没有删两次的
接近 2 年之前 回复
visoncomputer
visoncomputer   2016.03.09 20:28

short m_nRadi = EDTdate[v+n];应该是这里的问题了。或者memset(occurs , 0 ,sizeof(short)*m_nValueMax);这里好像只对char型有效。另外的就看不出了。

li744831579
li744831579   2016.03.09 21:18

你可以每一块加个输出,看看哪里有问题,加个断点看看。int和short类型有隐式转换,都检查下

Csdn user default icon
上传中...
上传图片
插入图片
准确详细的回答,更有利于被提问者采纳,从而获得C币。复制、灌水、广告等回答会被删除,是时候展现真正的技术了!