注册 登录
编程论坛 C++教室

简单程序错在哪里?

hmsabc 发布于 2010-08-15 10:26, 938 次点击
程序代码:
//指针作为函数参数--谭浩强《C++程序设计》第一版 P170页 例6.3
/*
函数的参数不仅可以是整型、浮点型、字符型等数据,还可以是指针类型。它的作用是将一个变量的地址传送给被调用函数的形参*/


#include<iostream>
using namespace std;

int main( )
{
    void swap( int * p1,int * p2);                         //函数声明
    int * pointer_1,* pointer_2,a,b;                        //定义两个指针变量和两个整形变量
    cin >> a >> b;
    pointer_1 = &a;                                         //使 pointer_1 指向 a
    pointer_2 = &b;                                         //使 pointer_2 指向 b
    if( a < b) swap( pointer_1,pointer_2);                  //如果 a < b,使两指针互换
    cout << "max = " << a <<" " << "min = " << b <<endl;            //输出 a 和 b
    system("pause");
    return 0;
}

void swap(int * p1,int * p2)                                //函数的作用是将 * p1 的值与 * p2 的值交换
{
    int * temp;
    *temp = * p1;
    * p1 = * p2;
    * p2 = temp*;
}

运行出错,错在哪里呀?
8 回复
#2
weble2010-08-15 10:38
temp是一个野指针,你不需要这么定义,直接使用int temp就可以了
#3
东海一鱼2010-08-15 11:28
同意楼上。如果一定要用指针,可以如下:(很丑哦)

void swap(int * p1,int * p2)                                //函数的作用是将 * p1 的值与 * p2 的值交换
{
    int * temp;
    *temp = * p1;    //*(int*)&temp = *p1
    * p1 = * p2;
    * p2 = temp*;   //*p2 = *(int*)&temp;
}
#4
x884845322010-08-16 10:11
没看出3楼和1楼后面代码的不同啊?
#5
zgxyz20082010-08-16 10:54
void swap(int *p1,int *p2)                              
{
    int * temp=NULL;     //声明指针变量时应初始化;
    temp = p1;           //楼主指针变量赋值是错误的;
    p1 = p2;             //应该采用这种方法;
    p2 = temp;           
}
#6
哈哈帝王2010-08-17 12:54
void swap(int * p1,int * p2)                                //函数的作用是将 * p1 的值与 * p2 的值交换
{
    int * temp = new int;//你没有给他分配内存
    *temp = * p1;
    * p1 = * p2;
    * p2 = *temp;
    delete temp;
}

[ 本帖最后由 哈哈帝王 于 2010-8-17 13:04 编辑 ]
#7
cllpyl2010-08-17 21:14
顶五楼!
#8
ToBeOOP2010-08-17 21:32
如5楼的,交换指针地址不需要接触对指针的引用的。。。还有就是记得初始化指针。。。
#9
ycsky852010-08-17 21:59
顶五楼,指针地址交换直接交换指针,*temp最好初始化
1