プログラムのどこがエラーか教えてください(C++)

質問

問題は、buildingという基本クラスを作って、建物の階数、部屋数、床の総面積を格納するクラスを作成し、次にhouseという派生クラスを作り、buildingを継承し、寝室数と、浴室数を格納するクラスで、officeという派生クラスを作り、これもbuildingを継承し、消火器台数と、電話台数を格納するクラスを作成し、それぞれの結果を出力せよ。っていう問題です。
下に僕が作成したプログラムを書きます。雑なので見にくいかもしれませんがお願いします。

#include <iostream>
using namespace std;

class building{
int Floor,Room,Area;
public:
building(int f,int r,int a){ Floor=f; Room=r; Area=a; }
void get_FRA(int &f,int &r,int &a){ f=Floor; Room=r; Area=a; }
};

class house:public building{
int Bedroom,Bathroom;
public:
house(int f,int r,int a,int Be,int Ba):building(f,r,a){ Bedroom=Be; Bathroom=Ba; }
void Hshow();
};

class office:public building{
int FireExtinguisher,Telephone;
public:
office(int f,int r,int a,int fe,int t):building(f,r,a){ FireExtinguisher=fe; Telephone=t; }
void Oshow();
};

void house::Hshow()
{
int x,y,z;

get_FRA(x,y,z);

cout << "階数:" << x << "階建て" << endl;
cout << "部屋数:" << y << "室" << endl;
cout << "床の総面積:" << z << "m2" << endl;
cout << "寝室:" << Bedroom << "室" << endl;
cout <<"浴室:" << Bathroom << "室" << endl;
}

void office::Oshow()
{
int x,y,z;

get_FRA(x,y,z);

cout << "階数:" << x << "階建て" << endl;
cout << "部屋数:" << y << "室" << endl;
cout << "床の総面積:" << z << "m2"<< endl;
cout << "消火器の数:" << FireExtinguisher << "個" << endl;
cout << "電話の数:" << Telephone << "個" << endl;
}

int main()
{
house H_ob(5,2,300,3,2);
office O_ob(3,5,500,2,8);

H_ob.Hshow();
O_ob.Oshow();

return 0;
}

あと、プログラムの処理結果を貼っておきます。
できれば、詳しく教えていただければ幸いです。

回答No.4

麻野 なぎ（@asano_nagi） · ベストアンサー

No.2 さらに訂正

よく考えたら、main() は

int main()
{

house H_ob(5,2,300,3,2);
office O_ob(3,5,500,2,8);

H_ob.show();
O_ob.show();

return 0;
}

でも問題ないです。
おもしろみには欠けますが。

お礼

3回も回答ありがとうございます。
詳しく教えていただきよくわかりました。

回答No.5

Tacosan

「プログラムのどこがエラーか教えてください」とはどういうことでしょうか? コンパイルエラーがあるならコンパイルすればわかるはずですし, 論理的なエラーがあったとしても実行すれば「期待する結果と違う」ことから自分で調べればいいだけのことです.

せめてコメントくらい書こうよ. あと, 例えば「階数」だけ知ることを許さないのはなぜだろう.

お礼

すみません。
回答ありがとうございました。
以後気をつけます。

回答No.3

麻野 なぎ（@asano_nagi）

No.2 訂正です。

> さらに言えば、public な継承関係にあるのだから、get_FRA という関数は不要です。
> house::show() の中でも、office::show() の中でも、Floor, Room, Area は、何もせずにアクセスできます。

これ間違いです。
なぜか、Floor, Room, Area が private: なので。

（ただし、この場合でも、building の表示を building::show() に任せてしまうのはOK)

Floor, Room, Area という基本的な情報が、継承先で使えないのはかなり不自然なので、せめて、
protected:
にしたほうがいいのではないでしょうか？

回答No.2

麻野 なぎ（@asano_nagi）

基本的に、「継承」関係にあるのだから、表示する関数が、Oshow と Hshow と別々なのは、多分、減点対象でしょう。

おそらく、「正しい」 main() は、

int main()
{
building *H_ob = new house(5,2,300,3,2);
building *O_ob = new office(3,5,500,2,8);

H_ob->show();
O_ob->show();

return 0;
}

だと思うわけです。
なぜか？　このあたりを理解すると C++ の継承がもう少しわかってくると思います。

さらに言えば、public な継承関係にあるのだから、get_FRA という関数は不要です。
house::show() の中でも、office::show() の中でも、Floor, Room, Area は、何もせずにアクセスできます。

さらに言えば、building 自身も show() を持っていて、

void house::show()
{
building::show(); // ここで、Floor, Room, Area は表示できる。
std::cout << （自分だけが持っているものを表示）
}

とするのがいいです。
このあたりが、「継承」

回答No.1

Lchan0211b

void get_FRA(int &f,int &r,int &a){ f=Floor; Room=r; Area=a; }
↓
void get_FRA(int &f,int &r,int &a){ f=Floor; r=Room; a=Area; }