c++ - 无法创建新的wchar_t *

我有这个运算符重载器。
我的程序在创建新的wchar_t数组时崩溃。

myObject &operator += (const myObject &s) {
    wchar_t *cat = wcscat(data, s.data);
    int len = wcslen(cat);
    wchar_t *test = new wchar_t[len + 1]; //this is killing!
    wcscpy(test, cat);

    delete data;
    data = test;

    return *this;
}

有人知道发生了什么吗？

编辑完整的类定义

class myObject
{
    private:
        wchar_t *data;
    public:
        myObject() { data = 0; }
        ~myObject() { delete data; }

        myObject &operator += (const myObject &s) {
            wchar_t *cat = wcscat(data, s.data);
            int len = wcslen(cat);
            wchar_t *test = new wchar_t[len + 1];
            wcscpy(test, cat);

            delete data;
            data = test;

            return *this;
        }
};

最佳答案

该代码至少包含两个相当明显的问题:

您显然使用new wchar_t[n]分配数据，但使用delete p而不是delete[] p释放数据。

问题的可能原因是将两个字符串连接到一个字符串的内存中，然后分配足够的内存来复制数据。

您可能需要更多类似的东西:

myObject &operator += (const myObject &s) {
    size_t len = wcslen(this->data) + wcslen(s.data);
    std::unique_ptr<wchar_t[]> tmp(new wchar_t[len + 1]);
    wcscpy(tmp.get(), this->data);
    wcscat(tmp.get(), s.data);
    delete[] this->data;
    this->data = tmp.release();
    return *this;
}

实际上，我认为您想使用std::wstring:此类已经提供了逻辑，而且可能以更有效的形式提供了逻辑。

关于c++ - 无法创建新的wchar_t *，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/13323038/