0

我尝试将 std::map 与

class DEMO {
    public:
        DEMO();
        virtual ~DEMO();
        DEMO &operator =(const DEMO &d);
        DEMO(const DEMO& d);
        BYTE* Arr() const;
    private:
        BYTE *m_array;
};

DEMO &DEMO::operator =(const DEMO &d) {
    memcpy(m_array, d.Arr(), 1);
    return *this;
}

DEMO::DEMO(const DEMO& d) {
    //call operator=
    *this = d;
}

const BYTE* DEMO::Arr() const {
    return m_array;
}

DEMO::DEMO() {
    m_array = new BYTE[1];
    }

DEMO::~DEMO() {
    if (m_array != 0)
        delete [] m_array;
}

class MyClass {
    private:
        typedef map<unsigned int,DEMO> t_mapType;
        t_mapType m_map;
        void Test();
}

void MyClass::Test() {
    DEMO myDEMO;
    m_map[1] = myDEMO;
}

如果我在类中调用 Test()，我得到错误断言 _CrtIsValidHeapPointer...
我检查断点，我看到，在 assingment (m_map[1] = myDEMO;) 期间，DEMO 的析构函数被调用，我得到错误删除 [] m_array; - 我如何让它运行？

4

4 回答 4

4

复制构造函数和赋值运算符是错误的。你需要m_array在做之前分配内存memcpy。当您尝试使用正在创建的 myDEMO副本将DEMO对象插入地图时。要创建副本，使用类的复制 ctor。由于在执行此复制时未分配内存，因此调用崩溃。m_map[1] = myDEMO;DEMOm_arraymemcpy

于 2010-08-25T08:27:55.583 回答
1

将复制 ctor 实现基于赋值运算符几乎总是错误的。赋值运算符必须拆除对象的旧状态并建立新状态。复制 ctor 只需建立一个新状态。
如果你想这样做，你的复制ctor首先需要为赋值运算符建立一个默认状态才能拆除。当然，这是在浪费资源。

也就是说，实际上您应该使用复制和交换习语将赋值运算符基于您的 dtor 并改为复制 ctor。有关上述内容的更详尽解释，请参见此处。

于 2010-08-25T10:19:59.167 回答
0

我将代码更改为使用指向对象的指针而不是映射中值部分的对象：

typedef map<unsigned int,DEMO*> t_mapType;

所以我可以评估如下：

m_map[1] = new DEMO();

我必须使用自己的免费内存方法：

void DEMO::Clear() {
    if (m_map.size() > 0) {
        for (t_mapType::const_iterator it = m_map.begin(); it != m_mp.end(); ++it) {
            if (it->second != 0)
                delete it->second;
        }
        m_map.clear();
    }
}
于 2010-08-25T10:43:32.577 回答
0

使用 std::vector 管理您的 BYTE*。

class DEMO {
    public:
        DEMO();
        virtual ~DEMO();
        //DEMO &operator =(const DEMO &d);
        //DEMO(const DEMO& d);
        const std::vector<BYTE>& Arr() const;
    private:
        std::vector<BYTE> m_array;
};

// Doesn't even need to be implemented by us anymore
//DEMO &DEMO::operator =(const DEMO &d) {
//    memcpy(m_array, d.Arr(), 1);
//    return *this;
//}

// Again, doesn't need to be implemented
//DEMO::DEMO(const DEMO& d) {
//    //call operator=
//    *this = d;
//}

// Just changed to return const vector&.
const std::vector<BYTE>& DEMO::Arr() const {
    return m_array;
}

// Changed to use initializer list
DEMO::DEMO() 
    : m_array(1) {
    // Old code: m_array = new BYTE[1];
}

// Doesn't need any code by us
DEMO::~DEMO() {
    //if (m_array != 0)
    //    delete [] m_array;
}
于 2010-08-25T10:25:35.097 回答