#include <stdio.h>
#include <string.h>
void swap(char *p1,char *p2)
{
    char *p;
    strcpy(p,p1);
    strcpy(p1,p2);
    strcpy(p2,p);
}
main()
{
    char *str1,*str2,*str3;
    printf("please input three strings\n");
    scanf("%s,%s,%s",str1,str2,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("%s\n%s\n%s\n",str1,str2,str3);
}

  你在主函数中定义了字符型指针str1,str2,str3,到了后两行却想以字符串的格式将它们从键盘输入，有点矛盾。还有就是如果想用scanf读入字符串最好不要连着用"%s%s%s",尽管编译的时候可能不会报错，运行的时候肯定会报错，建议用分开来的scanf("s",str1);scanf("s",str2); 或者用gets(str1)来读入数据。

最后printf应该是*str1,*str2,*str3吧

这怎么可能运行的了，char *是指向字符的指针，而你用它去存字符串是不行的。用char *类型区存储字符串只能在初始化字符串时行的通，比如char *s="xidian"，这时编译系统会自动的为s分配一定的数组来存储这些字符。直接用char *来存放字符串是行不通的。

//C语言字符串排序
#include<stdio.h>

#define N 3   //设定的字符串个数 

int cmp(char *p1,char *p2)   //字符串比较函数 
{
    int i=0;
    while(*(p1+i)==*(p2+i))
    {
       if(*(p1+i)=='\0')
          return 0;
          i++;
    }
       return *(p1+i)-*(p2+i);
}

void sort(char *p[N])    //字符串排序 
{
   char *temp;
   for(int i=0;i<N-1;i++)
      for(int j=i+1;j<N;j++) 
          if(cmp(p[i],p[j])<0) /* 交换的是字符串的地址，不是字符串的内容，可更改 */
          {
              temp=p[i];
              p[i]=p[j];
              p[j]=temp;
          }
}

int main()
{
    char s[N][21];/* 设定每个字符串不超过20字节 */ 
    char *t[N];
    printf("Please input %d strings one by one:\n",N); 
    for(int i=0;i<N;i++)
       {
             scanf("%s",s[i]);
          t[i]=s[i];
       }
    sort(t);
    printf("The sequence after sort is:\n"); 
    for(int i=0;i<N;i++) 
    printf("%s\t",t[i]); 
    return 0; 
}

五楼的这位兄弟，主函数里i 都没有定义就用了，编译肯定过不了，不知从哪里粘贴的图片来糊弄人。楼主也没复制一下代码，自己编译一下，就认同了五楼的代码，真是佩服啊~

把char*改成char数组。还有最好不要用scanf，改用fgets函数。