wap( int* p1,int* p2 ) { int *p; *p = *p1; *p1 = *p2; *p2 = *p; } 段错误，p没有空间，为其分配空间或者不使用指针

这段代码定义了一个名为wap的函数，它接收两个整数类型的指针p1和p2作为输入。函数的目的是交换这两个指针所指向的值，但是代码存在问题，因为它试图直接修改指针p本身，而不是它所指向的内容。

在C语言中，`p`是一个未初始化的指针，如果没有明确地给它分配内存（例如通过动态内存分配`malloc`），尝试通过`*p`访问存储内容会触发段错误，因为它的地址可能是不确定的，可能会导致程序崩溃。

修正这个错误的方式有：

1. **为p分配空间**：如果需要在栈上为p分配足够的空间，可以这样做：

   int temp;
   p = (int*)malloc(sizeof(int)); // 分配一块内存
   if (p != NULL) {
       temp = *p1; // 先保存原始值
       *p = *p2;
       *p1 = temp;
       *p2 = *p; // 现在p指向正确的值
       free(p); // 使用完后记得释放内存
   }

2. **不使用指针**：如果不打算改变p本身的值，可以直接操作指针所指向的数据，无需创建临时变量：

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

这样就避免了对`p`的间接引用。

相关问题

wap( int* p1,int* p2 ) { int *p; *p = *p1; *p1 = *p2; *p2 = *p; }

这段代码是一个函数定义，函数名为Swap，接受两个int类型的指针作为参数。函数的功能是交换指针所指向的两个变量的值。在函数内部，定义了一个int类型的指针p，并将p1指向的值赋给p，然后将p2指向的值赋给p1，最后将p指向的值赋给p2。这样就完成了两个变量值的交换操作。

需要注意的是，在这段代码中，指针p没有进行初始化，这可能会导致未定义的行为。正确的做法是在定义指针p时，为其分配内存空间，例如使用动态内存分配函数new来分配内存空间。

正确的Swap函数实现应该是这样的：

void Swap(int *p1, int *p2) {
int *p = new int;
*p = *p1;
*p1 = *p2;
*p2 = *p;
delete p;
}

这样，在交换完值后，还需要释放指针p所指向的内存空间，以避免内存泄漏。
#### 引用[.reference_title]
- *1* [7.2【指针编程】编写一个函数Swap(int *p1,int *p2)，利用指针实现两数据的交换](https://blog.csdn.net/qq_39977056/article/details/124062533)[target="_blank" data-report-click={"spm":"1018.2226.3001.9630","extra":{"utm_source":"vip_chatgpt_common_search_pc_result","utm_medium":"distribute.pc_search_result.none-task-cask-2~all~insert_cask~default-1-null.142^v91^insertT0,239^v3^insert_chatgpt"}} ] [.reference_item]
- *2* *3* [int *p1 = new int[10]; int *p2 = new int[10](); 区别](https://blog.csdn.net/boiled_water123/article/details/82941897)[target="_blank" data-report-click={"spm":"1018.2226.3001.9630","extra":{"utm_source":"vip_chatgpt_common_search_pc_result","utm_medium":"distribute.pc_search_result.none-task-cask-2~all~insert_cask~default-1-null.142^v91^insertT0,239^v3^insert_chatgpt"}} ] [.reference_item]
[ .reference_list ]

wap( int* p1,int* p2 ) { int * p; *p = *p1; *p1 = *p2; *p2 = *p; }

这段代码存在问题，因为指针变量 `p` 没有初始化就被解引用赋值，这会导致未定义行为，可能会导致程序崩溃或者出现其他不可预期的结果。

正确的方式应该是使用一个新的临时变量来保存 `p1` 所指向的值，然后将 `p1` 指向 `p2` 所指向的值，最后再将 `p2` 指向临时变量的值。例如：

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

这样就可以正确地交换 `p1` 和 `p2` 所指向的值。