Java インスタンスおみくじについて

Question

class FortuneTeller{
public static void main(String[] args){
Fortune f = new Fortune( );
f.words= new String[]{"大吉", "中吉", ...};
f.draw( );
} }
class Fortune {
String[] words;
void draw( ){
int r = (int)(Math.random()*words.length);
System.out.println(words[r]);
} }

これらから

Fortune f1 = new Fortune();
f1.words = new String[]{"ラブラブ","新たな出会い","マンネリ","すきま風","破局"};
f1.type = "恋愛運";
f1.draw();
Fortune f2 = new Fortune();
f2.words = new String[]{"旗揚げ","昇進","現状維持","窓際送り","解雇"};
f2.type = "仕事運";
f2.draw();
Fortune f3 = new Fortune();
f3.words = new String[]{"一攫千金","小金持ち","ちりも積もれば...","赤字","自己破産"};
f3.type = "金銭運";
f3.draw();
を加え拡張for文を使って表示するにはどうすればいいですか？

Answer 1

ループで回すには配列やリストにする必要があります。
期待する動きとしては、最初にカテゴリと候補を Fortune という1固まりにしてそれを配列に追加、最後にカテゴリでループさせてそれぞれの候補をランダムで表示という事になります。

なのでこのデータを作成する際になるべく手を掛けないクラスの設計が必要になります。
通常だと

Fortune f;
List<Fortune> fs = new ArrayList<Fortune>();
f = new Fortune();
f.type = "xxx";
f.words = new String[]{...};
fs.add(f);

と、とても冗長な書き方になります。そこでクラスのコンストラクタを変更して簡単に書ける様な細工を入れます。

class Fortune { 
	private String category;
	private String[] words;

	private Fortune() {
	}

	public Fortune(String category, String[] words) {
		this.category = category;
		this.words = words;
	}

	public void draw() {
		int r  = (int)(Math.random()*words.length); 
		System.out.println("[" + category + "]" + words[r]);
	}
} 

こうする事で

public class FortuneTeller{
	public static void main(String[] args){ 
		List<Fortune> fs = new ArrayList<Fortune>();

		fs.add(new Fortune("おみくじ", new String[]{"大吉", "中吉", "小吉"}));
		fs.add(new Fortune("恋愛運", new String[]{"ラブラブ","新たな出会い","マンネリ","すきま風","破局"})); 
		fs.add(new Fortune("仕事運", new String[]{"旗揚げ","昇進","現状維持","窓際送り","解雇"})) ;
		fs.add(new Fortune("金銭運", new String[]{"一攫千金","小金持ち","ちりも積もれば...","赤字","自己破産"}));

		for (Fortune f : fs) {
			f.draw();
		}
	}
} 

データの初期化が簡単に出来るようになり、別のカテゴリや候補を足す際にも手数が増えなくなります。

[おみくじ]中吉
[恋愛運]新たな出会い
[仕事運]窓際送り
[金銭運]赤字

今日は新たな出会いがあるそうです。

Comments

[nakabou33]

拙い質問文から素晴らしい回答ありがとうございました！