▲究竟何时该释放内存

JustUSTC

#include <vector>
#include <algorithm>
#include <iostream>

using namespace std;

typedef struct  
{
        int keyLen;     
        char *recordKey;  
}keyElement;

void allc(vector<keyElement> &idx)
{
        string names[] = {"jack", "mike"};

        for(int i = 0; i <2; i++)
        {
                keyElement tmp;
                tmp.keyLen = names[i].length();
                tmp.recordKey = new char[tmp.keyLen];
                strcpy(tmp.recordKey, (const char *)names[i].c_str());               
                idx.push_back(tmp);
                //delete [] tmp.recordKey;  //这里删除会出错：导致main中无法访问index中数据
        }
}


int main()
{
        vector<keyElement> index;

        allc(index);
        cout<<index[0].recordKey<<" "<<index[1].recordKey<<endl;
        cout<<index.size()<<endl;

//        delete [] index[0].recordKey;   //这里删除为什么也会出错呢？
//        delete [] index[1].recordKey;
        return 0;
}

kuaizaifeng

报什么错？？？
手头没有机器，不能帮你跑跑看。

JustUSTC

错误如下

tyc611

tmp.recordKey = new char[tmp.keyLen+1];

lenovo

tmp.recordKey = new char[tmp.keyLen];
是不是应该为：
tmp.recordKey = new char[tmp.keyLen+1];

tyc611

由于你的keyElement没有重载operator=和拷贝构造函数（即没有重新为字符串分配空间），所以不能在allc函数中删除分配的空间

JustUSTC

原帖由 tyc611 于 2007-1-1 00:33 发表
由于你的keyElement没有重载operator=和拷贝构造函数（即没有重新为字符串分配空间），所以不能在allc函数中删除分配的空间

怎么解决呢? 我还没怎么用过运算符重载;

JustUSTC

原帖由 lenovo 于 2007-1-1 00:30 发表
tmp.recordKey = new char[tmp.keyLen];
是不是应该为：
tmp.recordKey = new char[tmp.keyLen+1];

我修改了以后解决了, 原因是什么呢? 我分配了tmp.keyLen的内存, 又删除了这么多, 不是理所当然吗?为什么会出错?

lenovo

原帖由 JustUSTC 于 2007-1-1 00:40 发表


我修改了以后解决了, 原因是什么呢? 我分配了tmp.keyLen的内存, 又删除了这么多, 不是理所当然吗?为什么会出错?

你要弄清楚string和char *之间的差别。
那个1是用来存放'\0'的，
点到为止，自己好好想想。

tyc611

原帖由 JustUSTC 于 2007-1-1 00:40 发表


我修改了以后解决了, 原因是什么呢? 我分配了tmp.keyLen的内存, 又删除了这么多, 不是理所当然吗?为什么会出错?

可能原因是，因为你少分配了一个字节空间，因而最后的'\0'占用了分配的动态空间后面一个字节，而这个字节可能参予存放动态分配空间的大小等信息（不同编译器的分配策略不一样，不要做太多假设，这儿只是分析你的可能原因），所以这个数据被修改，很难预料释放空间时会发生什么

[ 本帖最后由 tyc611 于 2007-1-1 02:21 编辑 ]