c++ クラスに関してです。

質問

うまくクラス同士を連動させることができなくてエラーが出てしまいます。
どこが間違えているのかアドバイスくださると助かります。
#ifndef H22_H_
#define H22_H_
#include <string>
#include "account.h"
class Bank {
public:
Bank();
void deposit(double amount, const std::string& accountType);
void withdraw(double amount, const std::string& accountType);
void transfer(double amount, const std::string& accountType);
double getBalance() const;
double getSavingsBalance() const;
double getCheckingBalance() const;
private:
Account checking, savings;
};
#endif

#ifndef ACCOUNT_H
#define ACCOUNT_H
#include <string>
#include "account.h"
class Account{
public:
Account();
Account(double &amout);
double deposit(double amount);
double getBalance() const;
double withdraw(double amount);
double balance;
private:
};
#endif

#include <iostream>
#include "acount.h"
using namespace std;
Account::Account
{
balance = 0;
}
Account::Account(double amount)
{
balance = amount;
}
Account::double deposit() const
{
return balance;
}
Account::double getBalance(double balance)
{
balance += amount;
return balance;
}
Account::double withdraw(double amount)
{
if(amount > balance)
{
balance -= 5;
return balance;
}
elese
{
balance -= amount;
return balance;
}}

h22.h の中身
#include <string>
#include "h22.h"
#include "account.h"
#include <stdexcept>
using namespace std;
Bank::Bank()
{
Account checking;
Account saving;

}
void Bank::deposit(double amount, const string& accountType)
{
if (accountType != "S" && accountType != "C")
throw invalid_argument("Does not compute");
else if(accountType == "S")
savings.balance += amount;
else if(accountType == "C")
checking.balance += amount;

}
void Bank::withdraw(double amount, const string& accountType)
{
if (accountType != "S" && accountType != "C")
throw invalid_argument("Does not compute");
else if(accountType == "S")
savings.balance -= amount;
else if(accountType == "C")
checking.balance -= amount;
}
void Bank::transfer(double amount, const string& accountType)
{
if (accountType != "S" && accountType != "C")
{ throw invalid_argument("Does not compute");}
else if(accountType == "S")
{ savings.balance += checking.balance;
savings.balance = 0;}
else if(accountType == "C")
{ checking.balance += amount;
checking.balance = 0;}

}
double Bank::getBalance() const
{
return getBalance();
}
double Bank::getSavingsBalance() const
{
return savings.getBalance();
}
double Bank::getCheckingBalance() const
{
return checking.getBalance();
}

メイン.cpp
http://pastebin.com/rQTj8xciです。

なにかヒントやアドバイスお願いします

回答No.3

akinomyoga

初心者だと質問の仕方も分からなくて難しい所も多いかとは思いますが、初めに文句を述べさせて頂きます (わざと厳しく書いている所があるという事は御承知下さい)。

他の方も仰っている様に、質問文のコードを見るとまともにコンパイルが通らない様なコードです。とすれば、そもそも質問者さんの所でコンパイルが通っていないか、てもとのコードはコンパイルが通るが質問入力欄にコードを手で入力してミスっているかと思われるのですが…。前者ならば「連動させることができない」のではなくそれ以前の問題として「コンパイルが通らない」と書くべきだし、後者であるのならば実際のコードの内容を「コピー&ペースト」で貼り付けるべきです。

また、質問の仕方にも問題があるかと思います。コードを貼り付けるのではなく、何が目的で何が問題点になっているのかを説明するべきです。長いコードを貼り付けて単に問題点を教えてくれというのでは、回答する側も何を答えたらよいのやら分かりません。(問題点というのは目的によって変わりますよね。物凄く極端な例を言えば、もしコンパイルエラーを出す事が目的であれば、質問文のコードには問題点はないわけです。ちゃんとエラーを出す訳ですから(極端すぎるとは思いますが)。 あるいはもっと現実的な例として、いざコンパイルが通った後の動作が期待していた物と違うという場合には、期待していた動作が何かと実際にはどうなるかを説明しないと問題点は相手に伝わりません。)

で、以降は質問者さんが遭遇しているのはコンパイルエラーだと仮定して話を進めます。ソースコードの内容的に、上から順に h22.h, account.h, account.cpp, h22.cpp という(感じの名前の)ファイルの中身を貼り付けた物と推測致します(ファイル名は重要なので書いて下さい)。

1h22.h: コンストラクタ Bank(double,double); がない。(main.cpp から呼ばれている)
2 account.h: #include "account.h" →不要。(自分自身を include している。もし include guard がなかったら無限ループになる所です。)
3 account.h: ×Account(double &amout) → ○Account(double amount)
4 account.cpp: #include "acount.h" の acount は account のスペルミス
5 account.cpp: ×Account::Account → ○Account::Account()
6 account.cpp: ×Account::double 関数名(...) → ○ double Account::関数名(...) (3箇所)
7 account.cpp: ×関数名の deposit と getBalance が逆
8 account.cpp: ×getBalance(double balance) → ○deposit(double amount)。引数名が間違っている
9 account.cpp: ×elese → ○ else
10 地の文: ×h22.h の中身→×h22.cpp の中身
11 h22.cpp: Bank::Bank() の中身 Account checking; → 不要 (Account checking は this->checking と何の関係もない。更に、this->checking は Account のデフォルトコンストラクタで初期化済み)
12 h22.cpp: Bank::Bank() の中身 Account savings; → 不要 (同上)
13 h22.cpp: Bank::Bank(double checking,double saving) がない。追加する。例えば↓
Bank::Bank(double initialChecking,double initialSavings):checking(initialChecking),savings(initialSavings){}

これで全部かどうかは分かりません。(あと、質問の仕方が良くないなので、この様に回答が得られる事を当然とは思わないで下さい。)

> なにかヒントやアドバイスお願いします

質問者さんがするべき事は、個人的に以下の事と思います。

(1) 第一にコンパイラの出力するエラーメッセージを理解できるようになること。
コンパイラが沢山のエラーを出力するかもしれませんが、一つ一つちゃんと意味のある事が書かれています。コンパイルエラーが出る度に人に聞くというのをいつまでも続けるのは許されない事なので、これらのメッセージをちゃんと理解できる様になるべきです。
コンパイルエラーは、沢山出ても慌てずに、「一つずつ」解決していく事が基本です。エラーメッセージには大抵、エラーが起こったファイル名と行番号が書かれていますので、先ずはその箇所を確認する癖をつけて下さい。それができるようになったら、エラーメッセージが具体的にどういう事を意味するのかを調べるなり聞くなりして、段々と理解できる様になれば良いでしょう。

(2) 第二に質問の仕方を覚えること。
これは冒頭で述べた通り何が問題点なのか誰が聞いても分かる様に説明する事です (まあ、分かりやすく説明するには慣れの問題もあるのですぐには難しいかも知れませんが意識する様にして下さい)。
私自身、質問の仕方に自信があるわけでもないのですが、例えば今回の場合においては以下の様に質問するのが一つのやり方だったかなと思います。
-----
コンパイルをしようとすると以下の様なエラーメッセージが表示されコンパイルできません。

(エラーメッセージのコピーペースト)

このエラーメッセージはどの様な意味でしょうか。どの様な場合に起こる物でしょうか。
-----
ただし、大量に出たエラーメッセージを全部まとめて質問するのは止めて下さいね。飽くまで自分の分からなかったエラーの部分を抜き出して質問する事です。(何れにしても、自分自身で手を尽くしても解決できなかった場合に質問する事はもちろん前提ですよ。)

これらさえ習得できれば、取り敢えずの所はただ真っ直ぐ前に進むだけです。頑張って下さい。

回答No.2

επιστημη（@episteme）

デタラメやないですか。
質問はコンパイル・エラーが消えてから。

回答No.1

kmee

具体的、どんなことをやろうとして、それがどんな風になる/ならない のでしょうか?
それがわからなければ、「正解」を考えることができません。
明かなエラーとかを指摘することしかできません。

・ account.h についての説明がありません。
どこにあるのでしょうか?

> h22.h の中身
> #include <string>
> #include "h22.h"

・h22.h の中にh22.hをincludeするのですか?
・Bankの実装部分が含まれますが、これは、ここに書こうとしたものですか?
> Bank::Bank()
> {
> Account checking;
> Account saving;
> }

Account checking、Account savingは「(コンストラクの中だけで有効な)ローカル変数」になります。
this->checking, this->savingとは別です。
このままでは無意味な宣言です。

> #include <iostream>
> #include "acount.h"
> using namespace std;

・acount.h はどこでしょう? account.hとは別ですか?

> Account::Account
(以下略)

・コンストラクタの定義だと思われますが、 ()がありません。
・Account::Account(double) はクラスでの宣言がありません。
宣言されているのはAccount::Account(double&)です。
/* でも、ここで参照が必要でしょうか? */
・Account::double deposit() const 等、あきらかに変です。

などなど、現行では、コンパイルもまともにできません。