我试图创建一个函数,用 str1
替换文本 t
中所有出现的 str2
,但我不断收到“缓冲区溢出”错误消息。你能告诉我我的功能有什么问题吗?
#include <stdio.h>
#include <string.h>
#include <assert.h>
//replace all *str1 in *t with *str2, put the result in *x, return *x
char * result(char *str1,char *str2,char *t)
{
char *x=NULL,*p=t,*r=t;
x=malloc(400*sizeof(char));
assert(x!=NULL);
x[0]='\0';
r=strstr(t,str1); //r is at the first occurrence of str1 in t, p is at the beginning of t
while(r!=NULL)
{
strncat(x,p,r-p); //copy r-p chars from p to x
strcat(x,str2); //copy str2 to x
p=r+strlen(str1); //p will be at the first char after the last occurrence of str1 in t
r=strstr(r+strlen(str1),str1); //r goes to the next occurrence of str1 in t
}
strcat(x,p);
return x;
}
我没有使用
gets()
函数读取任何 char
数组。我的编译器是 gcc 版本 4.6.3
我更新了代码,它有效,但结果并不如预期。
main()
函数:int main(void)
{
char *sir="ab",*sir2="xyz",*text="cabwnab4jkab",*final;
final=result(sir,sir2,text);
puts(final);
free(final);
return 0;
}
打印字符串:
b
我期待
cxyzwnxyz4jkxyz
最佳答案
看起来您的 strncpy
参数混淆了:第二个参数是 源 字符串,而不是要复制的字符数的限制,这应该是第三个参数:
strncpy(x, p, r - p); // copy r - p chars from p to x
此外,您想使用
strcat
而不是 strcpy
。使用 strcpy
,您只需每次都用替换字符串覆盖结果的内容。使用 strcat
,请确保在开始之前使用 \0
初始化结果。最后,您要从您的函数返回对局部变量
x
的引用:您不能这样做,因为在函数返回后内存不可用。关于C 缓冲区溢出,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/12046068/