c++ - char* 和 new 的一些错误

Question

我有相同的代码：

.hpp 文件：

class CConsoleModel
{
char* ParametersBuffer;

...

public:
CConsoleModel() ;  // - basic constructor;
~CConsoleModel() ; // -basic destructor
char *DeterminationParameter(std::string _command, int _parametersize);
...
};

.cpp 文件：

char *CConsoleModel::DeterminationParameter(std::string _command, int _parametersize)
{
  ParametersBuffer = new char[_parametersize];
  unsigned int HexValue;
 _command = _command.substr(_command.length() - (_parametersize*2),(_parametersize*2));
  //do conversion of the string to the required dimension (_parametrsize):
  for (int i(0); i<_parametersize;i++)
  {
    std::stringstream CommandSteam;
    CommandSteam<< std::hex <<_command[2*i];
    CommandSteam<< std::hex <<_command[2*i +1];
    CommandSteam >> std::hex >> HexValue;
    ParametersBuffer[i] = static_cast<char> (HexValue);
  }
  return  ParametersBuffer;
}

程序构建，但运行时崩溃。

如果我改变ParametersBuffer = new char[_parametersize]

到char* ParametersBuffer = new char[_parametersize]

一切正常。我该如何解决这个问题？

Answer 1

我们强烈建议使用std::vector而不是手动分配内存。

class CConsoleModel
{
    std::vector<char> ParametersBuffer;

和

ParametersBuffer.resize(_parametersize);

...

return &ParametersBuffer[0];

顺便提一句

std::stringstream CommandSteam;
    CommandSteam<< std::hex <<_command[2*i];
    CommandSteam<< std::hex <<_command[2*i +1];
    CommandSteam >> std::hex >> HexValue;

太可怕了，当你有个位数的值时将无法工作。尝试

HexValue = (_command[2*i] << 8) | _command[2*i+1];

Answer 2

我的心理调试技能告诉我，您缺少复制构造函数或复制赋值运算符（或者您的析构函数没有正确清理缓冲区）。

如果您只是使用std::string 此功能，您的问题将得到解决。