0

仕事の観点から、人に関する情報を保持するカスタム クラス 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;

私は以前に istream をまったく操作したことがありません。エラーはおそらく誤った理由によるものです。助けていただければ幸いです。

4

1 に答える 1

2

ここでサイズを指定する必要があります。

char** dataArray=new char*;

私はあなたが必要だと思います:

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

この変数のメモリをリークします。この配列の要素を のように操作しているように見えますがstd::string、単純な char 配列です。それらを割り当てて再割り当てする必要があります。そのすべての問題を回避するには、 と を使用std::vectorstd::stringます。あなたのエラーはそれ自体とは何の関係もありませんoperator<<

于 2013-02-14T15:02:08.703 に答える