C++ コンストラクタからの変数参照

Question

前提・実現したいこと

C++で複数人の名前と年齢を連続で、最大100人分入力するプログラムを作っています。
年齢に負の値が入ると、100人分入力していなくても、入力を終了するようにします。
コンストラクタで入力を行い、人数をカウントしたいです。

発生している問題・エラーメッセージ

識別子"flag"が定義されていません
識別子"code"が定義されていません

該当のソースコード

#include <iostream>
#include <string>
using namespace std;

class person{
public:
	int age;
	string name;
	person();
};

person::person(){
	if (flag) {
		code++;
		cout << "名前と年齢は？:" << endl;
		cin >> name >> age;
		if (age < 0) {
			flag = false;
		}
	}
}

int main(void) {
	int num=0;
	bool flag=true;
	person w[100];
	
	return 0;
}

試したこと

別のクラスCを用意し、そこでflagとcodeを宣言し、そのCをcとしてインスタンス化し、codeの代わりにc.codeとしたが、“識別子が定義されていません”とのエラーが出た。

補足情報（FW/ツールのバージョンなど）

visual studio 2017
visual C++
windows デスクトップ　ウィザード

Comments

[cateye]

publicしか無いのに、構造体でなくクラスを使う意味は？
途中で終わるかも知れない（個数が不定）なら、なぜvector<person>を使わないのでしょう？
vector<person>を内包したclassを作り、終了条件と成るflag等の機能はその中に持たせれば良いのでは？
・・・そもそも、コンストラクタは、初期設定をするもなので、処理を埋め込むのはおすすめできません。

Answer 1

#include <iostream>
#include <string>

class person{
public:
    int age;
    std::string name;
    person();

    static bool flag;
    static int  size;
};

bool person::flag = true;
int  person::size = 0;

person::person(){
    if (flag) {
        std::cout << "名前と年齢は？:" << std::flush;
        std::cin >> name >> age;
        if (age < 0) {
            flag = false;
        } else {
          ++size;
        }
    }
}

int main(void) {
    using namespace std;
    person w[100];
    
    for ( int i = 0; i < person::size; ++i ) {
      cout << w[i].name << ' ' << w[i].age << endl;
    }
    return 0;
}

Comments

[kn59]

解決しました。ありがとうございます。

Answer 2

解決していますが，

コンストラクタで入力を行い…

という点に強いこだわりがあるのでなければ，
何と言うか，ただ普通に書けばその分苦労も減るのではないでしょうか．

class Person
{
public:
    Person( int age, std::string name ) : m_age(age), m_name(name) {}
    //※データがpublicなのはそのまま
    int m_age;
    std::string m_name;
};

int _main()
{
    std::vector< Person > Data;

    for( int i=0; i<100; ++i )  //最大100回の入力
    {
        int age;
        std::string name;
        //※終了したいときに「まず無意味に名前を入力してから年齢に負の値を入れる」という手間が何か嫌なので
        //年齢側から先に入力するようにしたが，それはそれで通常の入力としては順序がキモイ…
        std::cin >> age;
        if( age < 0 )break;
        std::cin >> name;
        Data.emplace_back( age, name );
    }

    std::cout << "人数=" << Data.size() << "\n";
    for( const auto &P : Data )
    {   std::cout << P.m_age << " " << P.m_name << "\n";    }

    return 0;
}

Answer 3

ローカル変数はその関数の中だけでしか有効ではありません

＃ほかにも問題がありそですが

Comments

[kn59]

ありがとうございます。