无法创建新的WCHAR_T *

Cannot create new wchar_t *

本文关键字：WCHAR 创建      更新时间：2023-10-16

我有此操作员Overloader。我的程序在创建新的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;
        }
};

此代码至少包含两个相当明显的问题：

  1. 您显然使用new wchar_t[n]分配数据，但您使用delete p而不是使用delete[] p。
    2. 。
  2. 您问题的可能原因是您将两个字符串串联成一个字符串的内存，然后然后分配足够的内存以复制数据。

您可能想要更多的东西：

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：此类已经提供了逻辑，可能还是以更有效的形式。

正如Dietmar所说（但有更多详细信息可能发生的事情发生在幕后）：

1）呼叫wcscat(data, s.data)覆盖了data指向的缓冲区的末端。如果没有超支，那么您就不需要分配一个新的缓冲区，因为现有的缓冲区将足够大。我期望它 都足够大。

2）由于缓冲区的压制，记忆分配器使用的数据结构被丢弃。当您尝试分配内存时，这会导致崩溃。当您免费内存时，它可能很容易引起崩溃，或者根本不会崩溃。

相关文章：