void swap(int *p1,int *p2) //定义swap函数 { int *temp; *temp=*p1; *p1=*p2; *p2=*temp; } 为什么这个不对

你提供的代码有一个问题，即`temp`指针没有被正确初始化。在执行`*temp = *p1;`这行代码时，由于`temp`指针没有指向任何有效的内存位置，会导致未定义的行为。

为了修正这个问题，你需要为`temp`指针分配内存空间，或者使用一个已经存在的变量来作为中间变量。下面是两种修正的方法：

1. 使用已经存在的变量作为中间变量：

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

2. 分配内存空间给`temp`指针：

void swap(int *p1, int *p2) {
    int *temp = (int*)malloc(sizeof(int));
    *temp = *p1;
    *p1 = *p2;
    *p2 = *temp;
    free(temp);  // 释放内存空间
}

这两种修正方法都可以正确地交换两个指针所指向的值。

希望能够帮助到你。如果还有其他问题，请随时提问。