大家帮忙看一下还有没有更好的方法

lovemoo

创建一个Employee类，该类中有字符数组，表示姓名、街道地址、市、省和邮政编码，把表示构造函数、ChangeName()、Display的函数原型改在类定议中，构造函初始化每个成员，Display()函数把完整的对象数据打印出来，其中数据成员是保护的，函数是共公的。
#include <iostream.h>;
#include <string.h>;
class Employee
{
        public:
           Employee();
           void ChangeName(char * pName,int i,char * pAddress,int j);
           void Display();
        private:
           char * name;
           char * address;
           char * city;
           char * province;
           double postcode;
};
Employee::Employee()
{
        name="jack";
        address="workers way";
        city="WuHan";
        province="HuBei";
        postcode=430012;
}
void Employee:isplay()
{
        cout<<"姓名:"<<name<<endl
            <<"地址:"<<address<<endl
            <<"城市:"<<city<<endl
            <<"省份:"<<province<<endl
            <<"邮政编码:"<<postcode<<endl;
}
void Employee::ChangeName(char * pName,int i,char * pAddress,int j)
{
        strcpy(name,pName,i);
        strcpy(address,pAddress,j);
}
void main()
{
        Employee a;
        a.Display();
        char x[10],y[30];
        int i,j;
        cout<<"请输入新的姓名:";
        for(i=0;i<10;i++)
        {
                a=getchar();
                if(a=='\n')
                        break;
        }
        a='\0';
        cout<<"请输入新的地址:";
        for(j=0;j<30;j++)
        {
                b[j]=getchar();
                if(b[j]=='\0')
                        break;
        }
        b[j]='\0';
        a.ChangeName(x,i,y,j);
        a.Display();
}




这样应该没问题吧？

hynjust

你编译没有？ 有没有问题？
不过咋一看， 建议你不要用strcpy之类的function, 为什么不用string呢？

有那么多的指针， 轻考虑你的copy constructor

F3&F5

那么多的指针，都没有动态分配内存，直接指向字符串常量，用strcpy操作要出问题的，但是编译会通过。
strcpy只有两个参数，strncpy才有长度参数，但是strncpy不会加字符'\0'，要自己加。
a是什么东西，b又是从哪里来的，编译不会通过。

如果是刚开始写c++的，只要没有其他特殊要求的话，建议你用stl的string吧。