使用 malloc 连接字符串

Question

这是一个使用malloc连接字符串的程序

#include<stdio.h>
#include<conio.h>
#include<stdlib.h>

char *sconcat(char *ptr1,char *ptr2);

void main()
{
char string1[20],string2[20],*ptr;
clrscr();

printf("enter string 1: ");
gets(string1);

printf("enter string 2: ");
gets(string2);

ptr=sconcat(string1,string2);

printf("output string : %s",ptr);
getch();
}

char *sconcat(char *ptr1,char *ptr2)
{
int len1,len2,i,j;
char *ptr3;

len1=strlen(ptr1);
len2=strlen(ptr2);

ptr3=(char *)malloc((len1+len2+1)*sizeof(char));

for(i=0;ptr1[i]!='\0';i++)
ptr3[i]=ptr1[i];

j=i;i=0;
for(;ptr2[j]!='\0';j++,i++)
ptr3[j]=ptr2[i];

ptr3[j]='\0';
return(ptr3);
}

output:
enter string 1 : this program does
enter string 2 : not give output
output string : this program does 

连接字符串需要什么更正。当我在void main() 之后使用char string1[20],string2[20],*ptr; 时，

output:
enter string 1 : is this 
enter string 2 : correct ?
output string : correct? ?

Answer 1

第二个for 循环中的测试不正确；它应该是ptr2[i] != '\0'，而不是ptr2[j] != '\0'。

对代码的几点说明：

• Don't cast the return value of malloc.
• sizeof(char) 定义为 1，因此乘以 sizeof(char) 只会使代码更难阅读。
• 将sconcat 的参数声明为const char *，因为它们不会修改接收到的字符串。
• malloc 可以返回 NULL；您必须在程序中处理这种情况，例如显示错误消息并退出。
• gets 是不安全的，如果用户输入的字符多于分配的字符，它将使您的程序崩溃。将gets(string) 替换为fgets(string, sizeof(string), stdin)，并去掉尾随的换行符。
• clrscr() 和 getch()，以及臭名昭著的 <conio.h> 标头，不是标准 C 并且不可移植；在像这样的简单程序中避免使用它们。

Answer 2

你可以更简单地使用strcat

printf("enter string 1: ");
gets(string1);

printf("enter string 2: ");
gets(string2);

strcat(string1,string2);

不过，它会更改 string1，因此您可能也想使用 strcpy（将 string1 复制到另一个字符串然后返回）。