Question

vector<ClassX> xVec;

if (inputFile.peek() == '$')
{
    classX classXInstance;           //<==================== local instantiation
    readFileElements(classXInstance);//<== pass by reference
    if(classXInstance.validate())
    {
       xVec.push_back(classXInstance);///<=============== added here
    }
    /// destructor of the local copy is called here
}

我得到一个核心转储，试图调试，但是我用gdb得到了这么多垃圾消息，我只能看到该向量已损坏，不确定是否因为析构函数被调用是一个原因??

修改

我的班级看起来像这样

class ClassX
{
 public:
    ClassX() { numberOfX=0; ppXX = NULL; };
   ~ClassX();
   void validate();
   char **setX(const vector<string>& Xss);
   inline char **getX() {return ppXX;};
 private:
    int numberOfX;
    char **ppXX;
};

它包含一个析构函数，如下所示

ClassX::~ClassX()
{
    if (ppXX != NULL)
    {
        for(int i=0; i < numberOfXX; i++)
        {
            if (ppXX[i] != NULL)
            {
                delete [] ppXX[i];
                ppXX[i] = NULL;
            }
        }

        // Free array of pointers.
        delete [] ppXX;
        ppXX = NULL;
    }
}

setX分配所有必要的内存

验证给我一个ppXX [i]的打印输出，如果元素的数量与字符串向量的大小匹配则返回true

Answer 1

classXinstance的副本存储在xVec中，指针ppXX存储到内存中的某个区域。现在您有两个指向同一区域的对象。片刻之后，classXinstance被销毁，因此该区域受delete的约束。 xVec中的元素现在指向无效的内存。

最好的选择是使用std::Vector<std::string>代替char **ppXX;字符串向量负责引用和分配，因此您无需担心正确的构造/复制/销毁。

类范围问题的向量？

1 个答案: