Cin参数到自定义类

时间:2013-02-14 14:58:17

标签: c++ operator-overloading istream

我已经创建了一个自定义类Person,它从工作的角度来保存有关人员的信息。

class Person
{
    char* _name;
    char* _lName;
    char* _department;
    int _age, _salary;
public:
    Person(char* name, char* lName, char* department, int age, int salary);
    Person(void);
    ~Person(void);

    char* getFName(){return _name;};
    char* getLName(){return _lName;};
    char* getDepartment(){return _department;};
    int getSalary(){return _salary;};

    friend std::ostream &operator<<(std::ostream &cout, Person &person);
    friend std::istream &operator>>(std::istream& is, Person &person);
};

我希望实现的目的是使用cin以逗号分隔的char *形式将数据流式传输到此类中。

cin重载:

std::istream &operator>>(std::istream &is, Person &person)
{
    char* tmp="";
    is >> tmp;
    int i=0;
    int number=0;
    char** dataArray=new char*;
    dataArray[0]="";
    while(tmp[i]!= '\0')
    {
        if(tmp[i]==',')
        {
            number++;
            dataArray[number]="";
        }
        else
            dataArray+=tmp[i];
        i++;
    }
    person._name=dataArray[0];
    person._lName=dataArray[1];
    person._department=dataArray[2];
    person._salary=(int)dataArray[3];
    person._age=(int)dataArray[4];
    return is;
}

但是,程序会因错误而停止;

  is >> tmp;

我之前根本没有操纵过istreams,错误可能是由于错误的推理造成的。 帮助将不胜感激。

1 个答案:

答案 0 :(得分:2)

您需要在此处指定尺寸:

char** dataArray=new char*;

我认为你需要:

char** dataArray=new char*[5];

您泄漏了此变量的内存。您似乎正在操纵此数组的元素,就好像它们是std::string一样,但它们是简单的char数组。您需要分配和重新分配它们。要避免所有问题,请使用std::vectorstd::string。您的错误与operator<<本身无关。