2

下面是我的示例代码。它只是一个示例，类似于我在应用程序中使用的代码。

#define STR_SIZE 32

void someThirdPartyFunc(const char* someStr);

void getString(int Num, const char* myStr)
{
  char tempStr[] = "MyTempString=";
  int size = strlen(tempStr) + 2;
  snprintf((char*)myStr, size, "%s%d", tempStr, Num);
}

int main()
{
  const char * myStr = new char(STR_SIZE);
  getString(1, myStr); // get the formated string by sending the number
  someThirdPartyFunc(myStr); // send the string to the thirdpartyFunction
  delete myStr;

  return 0;
}

如果我使用此代码，我会遇到异常。我认为问题在于删除“myStr”。但是删除确实是必要的。

有没有其他方法可以格式化 getString 中的字符串并将其发送到 ThirdPartyFunc？

提前致谢。

4

2 回答 2

9

您分配的不是一个字符数组，而是一个字符与这一行：

 const char * myStr = new char(STR_SIZE);

并且一个分配的字符被初始化为STR_SIZE，在这种情况下导致“字符溢出”。

如果你想要一个大小的数组STR_SIZE：

 const char * myStr = new char[STR_SIZE];

（注意矩形 [ ]）。您必须使用delete[]运算符解除分配这样分配的内存块。

个人注意：您上面编写的代码（手动分配的字符串等）具有良好的教育意义；你会犯很多这样的错误，从而了解 C/C++ 的内部工作原理。对于您不想要的生产代码，对于您想要的生产代码std::string或其他字符串容器，以避免重复与字符串相关的错误。一般来说，您不是成功地重新发明字符串库如何工作的人。对于其他容器类型也是如此，例如动态可增长数组（std::vector）或字典类型或其他类型。但是对于上面的代码进行教育摆弄是一个很好的目的。

您的代码片段中还有其他问题（移交const char*给函数然后修改内存，size调用时未正确计算参数snprintf等），但这些与您的段错误问题无关。

于 2013-02-08T06:27:39.580 回答
5

重新技术，而不是

const char * myStr = new char(STR_SIZE);

做

char const myStr[STR_SIZE] = "";

请注意，两者都存在无法修改字符串的问题。

但是您只询问了分配/解除分配问题。

但是，在语言技术之上的水平有很多错误。

这是原始代码，完整的：

void someThirdPartyFunc(const char* someStr);

void getString(int Num, const char* myStr)
{
  char tempStr[] = "MyTempString=";
  int size = strlen(tempStr) + 2;
  snprintf((char*)myStr, size, "%s%d", tempStr, Num);
}

int main()
{
  const char * myStr = new char(STR_SIZE);
  getString(1, myStr); // get the formated string by sending the number
  someThirdPartyFunc(myStr); // send the string to the thirdpartyFunction
  delete myStr;

  return 0;
}

以下是在 C++ 级别执行此操作的方法：

#include <string>           // std::string
#include <sstream>          // std::ostringstream
using namespace std;

void someThirdPartyFunc( char const* ) {}

string getString( int const num )
{
    ostringstream stream;
    stream  << "MyTempString=" << num;
    return stream.str();
}

int main()
{
  someThirdPartyFunc( getString( 1 ).c_str() );
}

从更自然的#define代码中消失了，但请注意，即使使用所有大写的宏名称，它也很容易导致不需要的文本替换。无论如何，大写大写都是令人眼花缭乱的（这就是为什么它是宏名称约定，而不是其他约定）。在 C++ 中，只需使用const。

于 2013-02-08T06:26:20.153 回答