C语言，输入三个字符串，用指针实现字符串按由小到大的顺序输出，不知道问题出在哪里

Question

#include  <stdio.h>
#include  <string.h>
       void swap(char*p1,char*p2)
    {char*p;strcpy(p,p1);strcpy(p1,p2);strcpy(p2,p);}
    int main()
       { 
              char str1[20],str2[20],str3[20];          
         printf("Please  enter  three  string:\n");
         gets(str1); gets(str2);gets(str3);
         if (strcmp(str1,str2)>0)swap(str1,str2);
        if (strcmp(str1,str3)>0)swap(str1,str3);
          if (strcmp(str2,str3)>0)swap(str2,str3);
         printf("\n");
         printf("%s\n%s\n%s\n",str1,str2,str3);      
    return 0;
       }
     

Answer 1

将swap函数中的声明char* p，更改为char p[20],即可。strcpy的用途是将第二个参数的内容复制到第一个参数的地址中。而你对于p的声明的作用是将p指定为一个存储字符串地址的变量，说白了其作用是用于存储字符串的地址，而不是字符串中的数据，所以在需要运行时会出现分段错误提示。将p声明为字符数组则可以起到存储字符串内容的作用。另外为了保证这一简易程序的运行，在输入三个字符串时，每个字符串的字符数应该保持一致，比如同时为5个，或者同时为10个等。这样可以避免潜在的其他BUG发生。