6 
class Person {
private:
    string firstName;
    string lastName;
public:
    Person() {}

    Person(ifstream &fin) {
       fin >> firstName >> lastName;
    }

    void print() {
       cout << firstName
           << " "
           << lastName
           << endl;
    }
};

int main() {
    vector<Person> v;
    ifstream fin("people.txt");

    while (true) {
        Person p(fin);
        if (fin == NULL) { break; }
        v.push_back(p);
    }

    for (size_t i = 0; i < v.size(); i++) {
       v[i].print();
    }

    fin.close();
    return 0;
}

次のコードスニペットがどのように機能するか、説明していただけますか？if（fin == NULL）{ブレーク; }

finはスタック上のオブジェクトであり、ポインタではないため、NULLになることはできません。ifstreamクラスでオーバーロードされたoperator==関数を見つけることができませんでした。したがって、このスニペットがどのように機能するのか理解できません。

4

3 に答える 3

5

ifstreamクラスには（operator void *()またはoperator bool()C ++ 11では）があります。これは、テストするときに呼ばれるものです(fin == NULL)。

テストは、テストfin == NULLとまったく同じである必要がありますfin.fail()。

于 2012-05-21T07:01:47.197 に答える
3

およびの基本クラスにistreamはostream暗黙の変換関数があり、ブール値として使用できます。C ++ 11より前では、暗黙の変換はになりましたvoid*。

この変換の結果をポインターとして使用することは決して意図されていませんでした。のようなコードfin == NULLは、C++と標準ストリームの理解が非常に不十分であることを示しています。最初のループを書く慣用的な方法は、デフォルトのコンストラクターと operator>>forを定義してから、次のPersonように書くことです。

Person p;
while ( fin >> p ) {
    v.push_back( p );
}

（そして私がそれに取り組んでいる間：あなたは本当にの戻り値をテストするべきであり 、それが失敗した場合fin.close()は返さないでください：0

fin.close();
return fin ? EXIT_SUCCESS : EXIT_FAILURE;

。）

于 2012-05-21T08:29:08.307 に答える
2

これは、ストリームが使用されることになっている方法ではありません。確かに、これは（残念ながら！）コンパイルされ、「正しい」ことさえします。ただし、このようなコードは記述しないでください。このコードを書いた人は誰でも、おそらく彼らは賢いと思っていたでしょう。

しかし、彼らが実際に行ったことは、新しい、型にはまらないAPIを導入することで、C ++プログラマーの期待を打ち破ることでしたが、実際の利点はありませんでした。

Personこのコードは、入力ストリームからタイプのオブジェクトを初期化します。残念ながら、これを行うことにより、コードはオブジェクトの読み取り中にエラーをテストする機会を失います。これは良くありません。オブジェクトには、入力ストリームを受け入れるコンストラクターを含めるべきではありませんoperator>>。オーバーロードする必要があります。

于 2012-05-21T08:33:52.340 に答える