1

我有以下课程：

class Op
    {
    private:
        std::vector<std::string> m_operands;
        std::string m_op;
    public:
        Op(std::string op = "") : m_op(op){}
        std::string GenCode();
        void AddOperand(std::string& operand) ;
        std::vector<std::string> GetOperands() const { return m_operands; }
        std::string GetOp() const { return m_op; }
    };

类的实例保存到std::vector<Op> m_movOpV；即下面的方法增加了一个新的操作：

Op* AddMovOp()
{
    Op op("MovOp");
    m_movOpV.push_back(op);
    return &m_movOpV.back();
}

另外我定义了：

typedef std::pair<std::string,Op*> Assignment;
std::vector<Assignment> m_proceduralAssign;

因此，在添加 Op 之后，我可以Assignment使用以下函数生成一个：

void AddAssignment(ModuleVCodeGen::Op* op,const std::string& lExp)
{
    Assignment assignment = (Assignment)std::make_pair(lExp,op);
    m_proceduralAssign.push_back(assignment);
}

以下场景会产生问题：

1.Op* op1 = AddMovOp();
2.op1->AddOperand("operand1");
3.AddAssign(op,"1");
4.Op* op2 = AddMovOp();

执行步骤 4 后m_movOpV，用新的 . 正确更新Op。但m_proceduralAssign有一个垃圾数据：Assignment已经存在的m_proceduralAssign依次有空的 m_operands 和错误的指针m_op。并且没有添加新的Assignment（我试图添加）。

您能否告知问题发生在哪里？以及如何解决？

4

1 回答 1

2

你应该重新审视你的设计。我现在在您的代码中看到的一个大问题是，对向量对象的引用和指针经常无效（在 push_backs 和其他操作期间），因此通过返回指向新插入项的指针，您正在自找麻烦。如果我是你，我会考虑保存索引。

于 2013-07-21T15:57:06.963 回答