Question

所以，我有以下命令模式实现，它包含在std::map<CString, IWrite*> commandMap：

中

class IWrite
{
protected:
   CStdioFile* fileWriter;

public:
   IWrite(CStdioFile* _fileWriter)
      : fileWriter(_fileWriter)
   {
   }

   virtual ~IWrite()
   {
   }

   virtual BOOL exec() = 0;
};

class FloatWrite : public IWrite
{
private:
   float input;

public:
   FloatWrite(CStdioFile* _fileWriter, float _input)
      : IWrite(_fileWriter), input(_input)
   {      
   }

   BOOL exec()
   {
      CString fieldvalue;
      fieldvalue.Format("%f", input);
      fileWriter->WriteString(fieldvalue);

      return TRUE;
   }
};

我遇到的问题是我的静态分析工具抱怨在fileWriter的析构函数中IWrite未被释放或归零。但是，通过在析构函数中添加delete fileWriter，我在调用std::map.clear()之前删除地图中的Command Pattern对象时会出现内存访问错误，如下所示：

// free map memory
for ( std::map<CString, IWrite*>::iterator mapItr = commandMap.begin();
      mapItr != commandMap.end();
      ++mapItr)
{
   delete mapItr->second;
}
commandMap.clear();

我在这里错误地接近了内存管理吗？我没有做过很多关于STL地图的工作，所以我不熟悉惯用法。

编辑：我如何向地图添加元素：

void FooClass::initCommandMap(const MSG_DATA_STRUCT * msgdata)
{
   // Write a float, foo
   commandMap[_T("foo")] = new FloatWrite(&fileWriter, msgdata->foo);
   // Write an unsigned int, bar
   commandMap[_T("bar")] = new UIntWrite(&fileWriter, msgdata->bar);
   // etc...
}

每次用户选择写出数据时都会调用此方法，因此各种fileWriter使用的exec()对象与用户选择的文件一致。

请注意，CStdioFile fileWriter是FooClass的成员变量。

Answer 1

为什么要保留指向fileWriter的指针？从我看到的，您的Command对象假定在使用命令之前应该存在编写器。它也不应该尝试管理writer对象，因为它可以由多个命令对象共享。请尝试保留参考。

class IWrite
{
protected:
   CStdioFile &fileWriter;

public:
   IWrite(CStdioFile &_fileWriter)
      : fileWriter(_fileWriter)
   {
   }

   virtual ~IWrite()
   {
   }

   virtual BOOL exec() = 0;
};

使用命令模式进行内存管理

1 个答案: