我有一个令人沮丧的问题，我找不到答案。
我有这个功能：
// Append character to the end of a string
void str_AppendChar(char* s, const char* ch)
{
// +2 because of 2x ' '
char* buff = malloc(strlen(s)+strlen(ch)+2);
memset(buff, 0, sizeof(buff));
// Copy the whole string in buff
strcpy(buff, s);
// Append ch at the end of buff
int len = strlen(buff);
strcpy(buff+len, ch);
// Set end of the string
*(buff+strlen(buff)-3) = ' ';
strcpy(s, buff);
free(buff);
}
由于某种原因，我的程序在最后两次尝试免费执行。
我使用AppendChar((的代码是：(有点难看，但请耐心等待(
void it_GatherCmd(cmd_Details* lineptr[], char* cmd)
{
// Used to count number of rows in lineptr
int nlines;
Detailptr p;
char ch;
char* word = (char*)malloc(sizeof(char)+256);
memset(word, 0, sizeof(word));
nlines = 0;
while ((ch = *cmd++) != 'n')
{
if (ch != ' ' && ch != ' ' )
str_AppendChar(word, &ch);
else
{
int type = dict_CheckWord(word);
if (type != -1)
{
p = it_CopyInfo(word, type);
lineptr[nlines++] = p;
}
memset(word, 0, sizeof(word));
}
}
//EDIT*
free(word);
}
我的主要：
int main()
{
cmd_Details* arrCmd[MAXLINES];
char* str = "just some string";
it_GatherCmd(arrCmd, str);
printf("%s", str);
return 0;
}
AppendChar((一直工作正常，直到我创建了它_GetCharCmd((并在那里使用它。我花了大约3个小时在这上面，但我找不到问题。在网上搜索了一下，但我发现的东西与我的问题并不完全相关。
此代码存在一些问题。
首先，如果str_AppendChar实际上像它的名字所暗示的那样附加了一个字符，为什么要给它一个暗示C字符串的const char*？在这里传递指针而不是实际对象是零增益的，就像一些结构的情况一样；实际上，您仍然需要将4个字节推送到堆栈。
其次，正如我在评论中指出的，问题是你没有正确初始化分配的缓冲区——sizeof(buff)返回得很好，buff的大小，而buff是char*，这很可能是4。虽然简单地将sizeof(buff)更改为strlen(s)+strlen(ch)+2，也就是您实际分配的内存量，就解决了这个问题(由于sizeof(buff)可能比实际分配的多，所以您正在写超过该内存的内容(，但我建议简化函数如下：
// Append character to the end of a string
void str_AppendChar(char* s, char ch)
{
size_t sLen = strlen(s);
char* buff = (char*)malloc(sLen + 2); // 1 for the appended char, 1 for
//memset(buff, 0, sLen + 2); //not necessary, we'll overwrite the memory anyway
// Copy the whole string in buff
strcpy(buff, s);
// append our char and null-terminate
buff[sLen] = ch;
buff[sLen + 1] = ' ';
strcpy(s, buff);
free(buff);
}
请注意，此代码仍然很糟糕；它很高兴地假设s足够大，可以容纳一个额外的字符，但事实并非总是如此。
同样关于您的it_gatherCmd函数；它应该使用const char*，因为它不会以任何方式修改它(事实上，你调用它的方式是给它一个constchar*；修改字符串文字是未定义的行为，在Windows上，你可能会因为侵犯页面权限而崩溃(。
据我所见，在扫描命令的同时，您正在连续构建一个字符串。在附加字符时，绝对不需要复制要附加到的字符串两次。你正在有效地做的是：
void str_AppendChar(char* s, char ch)
{
int len = strlen(buff);
s[len++] = ch;
s[loen] = ' ';
}
请注意，每次使用strlen确定字符串长度，它将遍历整个字符串。更糟糕的是，你没有任何关于最大可用缓冲区大小的信息，所以尽管你进行了所有的分配、复制和释放，原始字符串可能会溢出。
代替
str_AppendChar(word, &ch);
在自动存储器中使用本地缓冲区：
char word[20];
int wlen = 0;
并像这样附加：
if (wlen + 1 < sizeof(word)) word[wlen++] = *cmd;
这将使字缓冲区未终止，因此在使用它之前，请附加它：
word[wlen] = ' ';
printf("next word: '%s'n", word);
(当然，您也可以确保字符串始终以null结尾。(
当您为一个新单词重置缓冲区时，不需要memset整个缓冲区；只是将CCD_ 11重置为零。