0

我有相同的代码:

.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]

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

4

2 回答 2

2

我们强烈建议使用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];
于 2013-09-04T15:57:13.840 回答
0

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

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

于 2013-09-04T15:57:23.473 回答