void swap(int * p1,int * p2) { int * t; *t=*p1;*p1=*p2;*p2=*t; }这串代码为什么错误

这段代码错误的地方在于没有为指针变量`t`分配内存空间，导致`t`指向一个未知的内存地址。正确的代码应该是：

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

或者使用指针来实现：

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

这样就可以正确地实现交换两个整数的值了。

相关问题

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

以下是一个演示如何使用void swap(int *p1,int *p2)函数交换两个指针所指向的值的例子：

#include <stdio.h>

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

int main() {
    int num1 = 10;
    int num2 = 20;
    
    printf("交换前：num1 = %d, num2 = %d\n", num1, num2);
    
    swap(&num1, &num2);
    
    printf("交换后：num1 = %d, num2 = %d\n", num1, num2);
    
    return 0;
}

输出结果：

交换前：num1 = 10, num2 = 20
交换后：num1 = 20, num2 = 10

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);  // 释放内存空间
}

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

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