Question

输入3个字符串，按由小到大的顺序输出，用指针的方法，我不知道我这哪里错了，C语言高手看看吧！

#include <stdio.h>
void main()
{
void swap(char *p1,char *p2);
char cs[3][20];
int i;
char *pointer_1,*pointer_2,*pointer_3;
for(i=0;i<3;i++)
scanf("%s",cs[i]);
pointer_1=cs[0];pointer_2=cs[1];pointer_3=cs[2];
if(cs[0]>cs[1])
swap(pointer_1,pointer_2);
if(cs[0]>cs[2])
swap(pointer_1,pointer_3);
if(cs[1]>cs[2])
swap(pointer_2,pointer_3);
for(i=0;i<3;i++)
printf("%s\n",cs[i]);
}
void swap(char *p1,char *p2)
{
char temp;
temp=*p1;
*p1=*p2;
*p2=temp;
}

Answer 1

#include <stdio.h>
#include <string.h>//字符串比较大小不能直接比较它们的指针,应该使用strcmp(a,b)来判断a,b大小.必须引用此头文件
void main()
{
void swap(char *&p1,char *&p2);//不能交换式因为你传的不是地址
char cs[3][20];
int i;
char *pointer_1,*pointer_2,*pointer_3;
for(i=0;i<3;i++)
scanf("%s",cs[i]);
pointer_1=cs[0];pointer_2=cs[1];pointer_3=cs[2];
if(strcmp(cs[0],cs[1])>0)
swap(pointer_1,pointer_2);
if(strcmp(cs[0],cs[2])>0)
swap(pointer_1,pointer_3);
if(strcmp(cs[1],cs[2])>0)
swap(pointer_2,pointer_3);
puts(pointer_1);
puts(pointer_2);
puts(pointer_3);
}
void swap(char *&p1,char *&p2)
{
char *temp;
temp=p1;
p1=p2;
p2=temp;
}

swap无效时因为你没有把指针地址传进去，导致了无用功.
指针char *也是类型,必须加上'&'表示传地址，才可以在函数结束以后对原来的参数起到作用
PS.原来你的函数中temp 是char类型,而你交换的是char *,所以这里也需要修改

Answer 2

对指针做交换应该用二级指针定义函数应该这样
void swap(char **p,char **p2);
调用该
swap(&p,&p2);