[英]Trouble with with concatenating two strings in to one
我需要编写一个将两个字符串连接到第一个字符串的程序。 我不能使用第三个参数来保存新字符串。 当我运行该程序时,没有任何输出,因此,我很确定问题出在连接两个字符串的函数中。
#include <stdio.h>
#define MAX_SZ 81
int concatString(char s1[], char s2[]); // concatenates s2 onto s1, returns length of s1
int getString(char c[], int len);
main(void)
{
char array1[MAX_SZ * 2];
char array2[MAX_SZ];
int string1 = 0;
int string2 = 0;
int concat = 0;
printf("Please String # 1 up to 80 characters long:\n");
string1 = getString(array1, MAX_SZ);
printf("Please enter String #2 up to 80 characters long:\n");
string2 = getString(array2, MAX_SZ);
concat = concatString(array1, array2);
printf("You entered \"%s\" (length = %i)\n", array1, concat);
return 0;
}
int getString(char c[], int len)
{
int i = 0;
char d = 0;
while (d != '\n' && i < len)
{
d = getchar();
c[i++] = d;
}
c[--i] = '\0';
return (i);
}
int concatString(char s1[], char s2[])
{
int i, j;
for (i = 0; s1 != '\0'; ++i)
s1[i] = s1[i];
for (j = i; s2 != '\0'; ++j)
s1[j] = s2[j];
s1 [i + j] = '\0';
return (i + j);
}
问题出在以下几行:
for (j = i; s2 != '\0'; ++j)
s1[j] = s2[j];
这里j
不为零,因此您将在s2
中以非零索引开始索引,甚至可能超出范围。
但这只是一个问题。 第二个问题是循环的条件,它永远不会是错误的,从而导致无限循环。
第一个循环的循环条件存在相同的问题。
至于为什么循环条件错误,字符'\\0'
等于零,这在大多数系统上也等于NULL
。 因此,您正在有效地检查是否s1 != NULL
。 由于两个字符串都是编译时数组,因此传递给函数的指针将永远不会为空指针。
有几个问题,主要是与for循环中的条件有关:
for (i = 0; s1 != '\0'; ++i)
除非字符串为null,否则永远不会为true。
您要输入的是
for (i = 0; s1[i] != '\0'; ++i)
这将检查您正在查看的字符是否为空终止字符,这可能就是您想要的。
第二个循环也是一个问题。
for (j = i; s2 != '\0'; ++j)
s1[j] = s2[j];
您是从j开始而不是从s2开始
for (j = 0; s2[j] != '\0'; ++j)
s1[i + j] = s2[j];
这将引导您完成s2,将s2 [0]添加到s1 [i],其中s1 [i]是最后一个for循环中的空字符。 然后递增j,然后s1 [i + 1]变成s2 [1],依此类推。
另外:
可以通过以下方式添加空字符(可能):
for (j = 0; s2[j-1] != '\0'; ++j)
s1[i + j] = s2[j];
如果您肯定知道s2之前的字符不为null,则可以通过声明:
char array1[MAX_SZ * 2];
char placeholder = 0xff
char array2[MAX_SZ];
如果那没有意义,也请忽略它,并以之前的方式附加null char
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.