明解Java入門編演習９−３

Question

この演習はDayクラスを使わずに行う演習なのでしょうか？
Dayクラスを使うのなら、銀行口座クラスにtoStringメソッドを作る理由が分かりません。
また、もしDayクラスを使わない場合、toStringメソッドで購入日の表示に使うyear, month, dateといった変数は銀行口座クラスのフィールドに追加して行うのでしょうか？
今のところ、Dayクラスと同じtoStringメソッドを銀行口座クラスに使っているため、そこで使われているyear, month, date変数でエラーが出てしまいます

分かる方いらっしゃいましたら、是非ともご教示賜りたいです。
よろしくお願いします。

Comments

[swordone]

その本を持っていないので、問題の内容およびコードが分かりません。ここに記載してください。

[退会済みユーザー]

こちらの質問が他のユーザから「質問の範囲が広すぎる」という評価を受けています
わからない点を明確にし、調査したこと・試したことと共に記入していただくと、回答が得られやすくなります。

Answer 1

hyper-drums-koさんの回答で、銀行口座クラスのtoString内で日付クラスのtoStringを呼べばいいのかと思ったのですが、つけてみるとシンボルを見つけられずエラーになりました。
しかしtoStringをつけない場合だと無事動作しました。
一体なぜこのようなことが起こるのでしょか？
また、toString内で複数の文字列を返すにはどうすれば良いのでしょうか？
String.formatに全部書くのが一般的なのでしょうか？
質問が多いのですが、どうかよろしくお願いします。

//成功例
public String toString()
	{
		return String.format("口座開設日：" + this.o_date);
	}
//失敗例
public String toString()
	{
		return String.format("口座開設日：" + this.o_date.toString());
	}

Comments

[swordone]

そもそもformatの使い方がおかしい気がしますが…

Answer 2

こういうことですか？

//銀行口座クラス
class Account
{
    private String name;
    private String no;
    private long balance;
    //追加
    private Day o_date;

    Account(String name, String no, long balance, Day o_date)
    {
        this.name = name;
        this.no = no;
        this.balance = balance;
        //追加
        this.o_date = new Day(o_date);
    }

    public String getName()        { return name; }
    public String getNo()        { return no; }
    public long getBalance()    { return balance; }
    void deposit(long k)        { balance += k; }
    void withdraw(long k)        { balance -= k; }

    //追加
    public Day getODate()
    {
        return new Day(o_date);
    }

    public String toString()
    {
        StringBuilder sb = new StringBuilder();
        sb.append("＝＝＝").append(this.name).append("の口座＝＝＝").append("\\n");
        sb.append("口座名義：").append(this.name).append("\\n");
        sb.append("口座番号：").append(this.no).append("\\n");
        sb.append("預金残高：").append(this.balance).append("\\n");
        sb.append("口座開設日：").append(this.o_date.toString()).append("\\n");
        return sb.toString();
    }
}

class AccountTester
{
    public static void main(String[] args)
    {
        Account adachi = new Account("足立太郎", "123456", 1000, new Day(2015, 12, 25));
        Account nakata = new Account("中田次郎", "654321",  200, new Day(2015, 11, 11));

        // 足立太郎の口座
        System.out.println(adachi.toString());

        // 中田次郎の口座
        System.out.println(nakata.toString());
    }
}

Comments

[tatataro]

銀行口座クラスのtoStringの中で日付クラスのtoStringを呼べばよかったんですね！
ありがとうございます！
ただ、hyper-drums-koさんのtoStringメソッドではreturn文がないためエラーになりました。
StringBuilder()というものは使ったことがないため、printlnで代用しようとしたのですが、同じくreturnをどうしたら良いのか分かりませんでした。

[hyper-drums-ko]

失礼しました。。。

修正しました。

[tatataro]

ありがとうございます。
よくよく考えたらreturnで返さなくても実行されているため、表示させたい最後の1文をreturnで返せば問題なく動いてくれました。

Answer 3

口座開設日のフィールドとしてDay型の変数(o_date)を用意し、口座開設日のゲッタとしてコピーコンストラクタを利用したものを追加したのですが、なおのこと銀行口座クラスにtoStringメソッドを作る意味が分からず、また、作り方も分かりません。
演習の通り、銀行口座クラスにtoStringメソッドを追加する場合、どうようにして実現させるのでしょうか？

//銀行口座クラス
class Account
{
	private String name;
	private String no;
	private long balance;
	//追加
	private Day o_date;

	Account(String name, String no, long balance, Day o_date)
	{
		this.name = name;
		this.no = no;
		this.balance = balance;
		//追加
		this.o_date = new Day(o_date);
	}

	public String getName()		{ return name; }
	public String getNo()		{ return no; }
	public long getBalance()	{ return balance; }
	void deposit(long k)		{ balance += k; }
	void withdraw(long k)		{ balance -= k; }

	//追加
	public Day getODate()
	{
		return new Day(o_date);
	}
}

class AccountTester
{
	public static void main(String[] args)
	{
		Account adachi = new Account("足立太郎", "123456", 1000, new Day(2015, 12, 25));
		Account nakata = new Account("中田次郎", "654321",  200, new Day(2015, 11, 11));

		System.out.println("＝＝＝足立太郎の口座＝＝＝");
		System.out.println("口座名義：" + adachi.getName());
		System.out.println("口座番号：" + adachi.getNo());
		System.out.println("預金残高：" + adachi.getBalance());
		System.out.println("口座開設日：" + adachi.getODate().toString());

		System.out.println("＝＝＝中田次郎の口座＝＝＝");
		System.out.println("口座名義：" + nakata.getName());
		System.out.println("口座番号：" + nakata.getNo());
		System.out.println("預金残高：" + nakata.getBalance());
		System.out.println("口座開設日：" + nakata.getODate());
	}
}

Answer 4

質問者です

演習9−3

• 第8章で作成した<<銀行口座クラス>>に、口座開設日のフィールドとtoStringメソッド

を追加せよ。
※コンストラクタを変更したり、口座開設日のゲッタ（口座開設日フィールドが参照する日付インスタンスのコピーを返す）などのメソッドを追加したりすること。

//銀行口座クラス
class Account
{
	private String name;
	private String no;
	private long balance;

	Account(String name, String no, long balance)
	{
		this.name = name;
		this.no = no;
		this.balance = balance;
	}

	public String getName()		{ return name; }
	public String getNo()		{ return no; }
	public long getBalance()	{ return balance; }
	void deposit(long k)		{ balance += k; }
	void withdraw(long k)		{ balance -= k; }
}

class AccountTester
{
	public static void main(String[] args)
	{
		Account adachi = new Account("足立太郎", "123456", 1000);
		Account nakata = new Account("中田次郎", "654321",  200);

		System.out.println("＝＝＝足立太郎の口座＝＝＝");
		System.out.println("口座名義：" + adachi.getName());
		System.out.println("口座番号：" + adachi.getNo());
		System.out.println("預金残高：" + adachi.getBalance());

		System.out.println("＝＝＝中田次郎の口座＝＝＝");
		System.out.println("口座名義：" + nakata.getName());
		System.out.println("口座番号：" + nakata.getNo());
		System.out.println("預金残高：" + nakata.getBalance());
	}
}

//日付クラス
class Day
{
	private int year;
	private int month;
	private int date;

	public Day(int year, int month, int date)	{ set(year, month, date); }
	public Day(Day d)							{ set(d.year, d.month, d.date); }

	public int getYear()	{ return year; }
	public int getMonth()	{ return month; }
	public int getDate()	{ return date; }

	public void setYear(int year)		{ this.year = year; }
	public void setMonth(int month)		{ this.month = month; }
	public void setDate(int date)		{ this.date = date; }

	public void set(int year, int month, int date)
	{
		this.year = year;
		this.month = month;
		this.date = date;
	}

	int dayOfWeek()
	{
		int y = year, m = month;

		if(m == 1 || m == 2)
		{
			y--;
			m += 12;
		}

		return ((y+y / 4-y / 100+y / 400+(13*m+8) / 5 + date) % 7);
	}

	public String toString()
	{
		String[] wd = {"日", "月", "火", "水", "木", "金", "土"};
		return String.format("%04d年%02d月%02d日(%s)", year, month, date, wd[dayOfWeek()]);
	}
}

このような内容なのですが、この演習の前に日付に関するDayクラスを作っているため、そちらを使えば銀行口座クラスにtoStringメソッドを作る必要がないように思えます。それともDayクラスを使わずに口座開設日を表示させることが目的なのでしょうか？
プログラミングは初心者なので、著者の意図を汲み取れない状況です。
是非よろしくお願いします。

Comments

[退会済みユーザー]

意図は銀行口座クラスと日付クラスを関連付けろってことなんじゃないですか？
今の時点では銀行口座情報として口座開設日がないから、それを日付クラス使ってフィールドを追加しろということかと。
その際のtoStringメソッドは銀行口座情報すべてを出力しろってことなのかと。
まぁ本持ってないのでわかりませんが。

[tatataro]

銀行口座クラスにtoStringメソッドを追加する場合、どうやってDayクラスのyear, month, dateを持ってくるのでしょうか？
Dayクラスのgetメソッドを使いたいのですが、toStringメソッドの中でインスタンスを指定できない？ので困っています。
（toStringメソッドの中で、adachi.getYear()やnakata.getMonth()のadachiやnakataの部分の指定ができない）