我有这样的函数,它将2个数组作为char参数,我需要合并这些字符串。抱歉,问题很简单。
void Abc (char **a, char **b)
{
for (int i = 0; i < 200; i++)
{
a[i]="hello ";
}
for (int i = 0; i < 200; i++)
{
strcat(a[i], b[i]); //doesn't show mistake, but it's there
cout << b[i]<<'\n'; //ok
cout << a[i]<<'\n'; //ok
cout<<strlen(b[i])<<'\n'; //ok
cout<<strlen(a[i])<<'\n'; //ok
}
}
为什么Strcat在这里不起作用?我该怎么做才能使这项工作?
答案 0 :(得分:0)
问题出现在以下声明中。
a[i]="hello ";
不是复制字符串文字,而是分配其地址。
改为使用
strcpy( a[i],"hello " );
答案 1 :(得分:0)
首先,每个a[i]
指向一个字符串文字,修改它会调用未定义的行为。
其次,如果您是strcat(x, y)
,则应确保x
有足够的空间存储合并的x
和y
。
因此,x
应该已分配内存并进行双重检查,您应该使用strncat
来避免意外溢出。
答案 2 :(得分:0)
以下是问题的根源:
for (int i = 0; i < 200; i++)
{
a[i]="hello ";
编译器在这里不知道您的意图,因此a[]
的每个元素都被分配了指向相同字符串的指针
a [0] == a [1] == a [2] == a [3] == a [4] == ... a [199]
即使这有效,问题是现在的所有元素都指向大小为7 {'h', 'e', 'l', 'l', 'o', 0}
的char数组。你不能stract
到它们的最后。
您需要确保a[]
的每个元素都有一个指向唯一存储空间的指针,并且它足够大以接收您的字符串。如果您的a[]
元素已经指向某个有效的位置,那么您需要做的是:
for (int i = 0; i < 200; i++)
{
strcpy(a[i], "hello ");
}