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

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() 之前删除映射中的命令模式对象时出现内存访问错误，如下所示：

// 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...
}

每次用户选择写出数据时都会调用它，因此各种exec() 使用的fileWriter 对象与用户选择的文件是当前的。

注意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;
};