多留意一下就可以了，你这里只是交换了指针的指向
改成下面的：
#include <stdio.h>
void main()
{
     int a,b,c,f,*p,*p1,*p2;
     scanf("%d%d%d",&a,&b,&c);
     p=&a;
     p1=&b;
     p2=&c;
     if(a>b)
     {f=*p;*p=*p1;*p1=f;}
     if(a>c)
     {f=*p;*p=*p2;*p2=f;}
     if(b>c)
     {f=*p1;*p1=*p2;*p2=f;}
     printf("%d,%d,%d",*p,*p1,*p2);
     puts("\n");
}

代码我没写，但是楼主的问题应该不是交换值或交换指针的问题，而是交换算法的逻辑不太对。
楼主人工按照输入走走代码，最后的顺序就是231
试试把if的代码段换成以下：
//CodeStart.
p=&a;
p1=&b;
p2=&c;

if(*p>*p1){
   f=p;p=p1;p1=f;}
if(*p>*p2){
   f=p;p=p2;p2=f}

if(*p2<*p1){
   f=p2;p2=p1;p1=f;}
//Complete
我的方法是直接确定三个数当中的最大值和最小值，剩下的自然就是中间大小的了。