1 
class mapInfo
{
public:
    mapInfo();
    ~mapInfo();  
public:
    int dataType_m;

private:
    int *frequency;
};

//这里定义了构造函数。

    mapInfo::mapInfo() :
    dataType_m(0),
    frequency(NULL)
{
}

//这里定义了析构函数

mapInfo::~mapInfo()
{
    free(frequency);
    frequency = NULL;
}

Result_t Maps::add(mapInfo &mapInfo_r)
{
    if (maps_mp == NULL)
    {
        numMaps_m = 1;
        maps_mp = (mapInfo *) calloc(1, sizeof(mapInfo));
    }
    else
    {
        numMaps_m++;
        maps_mp = (mapInfo *) realloc(maps_mp, numMaps_m*sizeof(mapInfo));
    }
    maps_mp[numMaps_m-1] = mapInfo_r; // Default copy constructor
    return 1;
}

使用 gcc8 编译时，出现以下编译错误。看起来像上面定义的析构函数给出了 gcc8 的编译错误。

如何解决这个问题？

 error: 'void* realloc(void*, size_t)' moving an object of non-trivially copyable type 'class xyyz::mapInfo'; use 'new' and 'delete' instead [-Werror=class-memaccess].
4

2 回答 2

2

这根本不是正确的 C++。重写您的代码如下（我在这里猜测 的类型frequency，但绝对不要使用free它）：

#include <vector>

class map_info
{
public:
    map_info();

private:
    int data_type;
    std::vector<int> frequency;
};

std::vector<map_info> maps_mp;

map_info::map_info() : data_type(0), frequency() {}

// …

void maps::add(map_info& map_info)
{
    maps_mp.push_back(map_info);
}
于 2020-09-03T20:32:57.407 回答
1 
    maps_mp = (mapInfo *) realloc(maps_mp, numMaps_m*sizeof(mapInfo));

这是不明智的。如果该对象不重要，则不能仅将对象从一个内存区域移动到另一个内存区域。

例如，考虑一个保存指向字符串的指针的字符串对象。它可能看起来像这样：

class MyString
{
    char* inner_ptr;
    char buf[64];

    ...
};

它可能有这样的构造函数：

MyString::MyString (const char* j)
{
     if (strlen(j) < 64)
         inner_ptr = buf;
     else
         inner_ptr = malloc (strlen(j) + 1);
     strcpy(inner_ptr, j);
}

像这样的析构函数：

MyString::~MyString()
{
     if (buf != inner_ptr)
         free (inner_ptr);
}

现在，想想如果你调用relloc这些数组会发生什么。短字符串仍将其inner_ptrs 指向您刚刚释放的旧对象的缓冲区。

错误消息很好地解释了这个问题。realloc用它来移动一个重要的对象是不合法的。您必须构造一个新对象，因为该对象需要有机会处理其地址的更改。

于 2020-09-03T21:02:01.123 回答