1

所以，我有以下命令模式实现，它包含在 a 中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在析构函数中添加 a ，当我在调用之前删除映射中的命令模式对象时，会出现内存访问错误，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...
}

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

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

4

1 回答 1

1

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

class IWrite
{
protected:
   CStdioFile &fileWriter;

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

   virtual ~IWrite()
   {
   }

   virtual BOOL exec() = 0;
};
于 2013-06-24T20:56:38.547 回答