#include <stdio.h>
main()
{
    int a,b,c;
    int *p1,*p2,*p3,*p;
    printf("Please input 3 numbers\n");
    scanf("%d,%d,%d",&a,&b,&c);
    p1=&a,p2=&b,p3=&c;
    if(*p1<*p2)
        *p=*p1;
        *p1=*p2;
        *p2=*p;
     if(*p1<*p3)
         *p=*p1;
         *p1=*p3;
         *p3=*p;
      if(*p2<*p3)
          *p=*p2;
          *p2=*p3;
          *p3=*p;
     printf("%d,%d,%d\n",*p1,*p2,*p3);

}

IF里面应该用括号括起来吧?

# include <stdio.h>

int max(int *p1, int *p2, int *p3) /*用函数进行比较*/
{
    int t;
    if(*p1<*p2)
    {
        t=*p1;
        *p1=*p2;
        *p2=t;
    }
    if(*p1<*p3)
    {
        t=*p1;
        *p1=*p3;
        *p3=t;
    }
    if(*p2<*p3)
    {
        t=*p2;
        *p2=*p3;
        *p3=t;
    }
    return 0;
}

int main(void)
{
    int a, b, c;
    int *p1, *p2, *p3;

    printf("请输入三个数：\n");
    scanf("%d%d%d", &a, &b, &c);

    p1=&a;/*p1指向a的地址*/
    p2=&b;
    p3=&c;


    max(p1, p2, p3);/*通过指针改变a b c的值*/

    printf("排序好的值是：%6d%6d%6d\n", a, b, c);

    return 0;
}

你的p指针是指向NULL的  是没有空间放东西的

可是谭浩强的书中一例题就是这么编写的
指针P也是这样写的啊

#include <stdio.h>
main()
{
    int a,b;
    int *p1,*p2,*p;
   
    scanf("%d,%d",&a,&b）;
    p1=&a,p2=&b;
    if(a<b)
        p=p1;
        p1=p2;
        p2=p;
    printf("%d,%d\n",*p1,*p2);
}
有什么区别么？

谭的例题是交换2个地址p1和p2，而你是交换2个数值*p1和*p2，你说有区别么？

哦 谢谢啦