swap(int *p1,int *p2)
{int *temp;
 *temp=*p1;      /*此语句有问题*/
 *p1=*p2;
 *p2=temp;
}



swap(int *p1,int *p2)
{int temp;
 temp=*p1;
 *p1=*p2;
 *p2=temp;
}

谁能给我解释一下为什么第一个不对？？？

swap(int *p1,int *p2)
{int *temp; －－－>这个指针指向的空间是不确定的，是野指针，它有可能指向程序空间的任何地方
*temp=*p1;      /*此语句有问题*/ －－－－ >给野指针赋值，我们会把不确定的地方的内容更改，严重的会使程序崩溃
*p1=*p2;
*p2=temp;
}

swap(int *p1,int *p2)
{
    int *temp;
    temp=p1;
    p1=p2;
    p2=temp;
}

为什么这样写也不对呢？

因函数的参数的值只是和你传进来的值相等的副本而已，并不是你传进来的参数它本身，你交换的只是形参的值，其实实参的值并没有改变

swap(int *p1,int *p2)
{
    int temp;
    temp=*p1;
    *p1=*p2;
    *p2=temp;
}
这样才对

实参变量和形参变量是单纯的“值传递”方式。指针做变量时也是遵循这一原则。所以函数调用时不会改变实参
指针变量值。

我觉得2，4，6说的都很对，再罗嗦一下，我看楼主还没懂。

swap(int *p1,int *p2)
{
    int temp;
    temp=*p1;
    *p1=*p2;
    *p2=temp;
}

是对的，函数swap要做的是要交换传进来的两个地址所指示的地方上的两个值。

swap(int *p1,int *p2)
{int *temp;
*temp=*p1;
*p1=*p2;
*p2=*temp;
}

很显然，temp不知指向哪里，*temp = *p1,是很危险的，如果一开始这么定义，就是对的,

swap(int *p1,int *p2)
{int a=0,*temp=&a;
*temp=*p1;      /*此语句有问题*/
*p1=*p2;
*p2=temp;
}

,不过似乎有点傻。

swap(int *p1,int *p2)
{
    int *temp;
    temp=p1;
    p1=p2;
    p2=temp;
}

，这样swap函数执行完后，只是p1,p2指向的地址改变了，例如，swap执行前，p1指向a=12,p2指向b=23,swap执行完后，p1指向b,p2指向a，但a还是等于12，b还是等于23，a，b的值还是没有交换。

谢谢7楼的，我明白啦。

swap(int *p1,int *p2)
{int a=0,*temp=&a;
*temp=*p1;     
*p1=*p2;
*p2=temp;
}

不过这样子编译也是不行的，只调换了形参指针地址，实参的值不变。