listとmapを使ってanimalショップ

Question

importとかは端折ってます。

public class Main {

	/**
	 * メインメソッド
	 * @param args
	 */
	public static void main(String[] args) {

		//動物リスト作成
		ArrayList<Animal> raionList = new ArrayList<Animal>();
		ArrayList<Animal> zouList = new ArrayList<Animal>();
		ArrayList<Animal> camelList = new ArrayList<Animal>();

		//動物を作成
		Animal raion = new Animal();
		raion.setName("ライオン");

		//System.out.println(raion.getName());

		Animal zou = new Animal();
		zou.setName("ゾウ");

		//System.out.println(zou.getName());

		Animal camel = new Animal();
		camel.setName("らくだ");

		//System.out.println(camel.getName());


		//上記で作成した動物をそれぞれ動物リストに格納する
		//ライオン3匹
		raionList.add(raion);
		raionList.add(raion);
		raionList.add(raion);
		//ゾウ2匹
		zouList.add(zou);
		zouList.add(zou);
		//らくだ1匹
		camelList.add(camel);


		//それぞれの動物リストをそれぞれの檻に格納

		Gage raionGage = new Gage();
		raionGage.setAnimalList(raionList);

		//三匹入っている。
		//System.out.println(raionGage.getAnimalList());

		Gage zouGage = new Gage();
		zouGage.setAnimalList(zouList);

		//２匹入っている。
		//System.out.println(zouGage.getAnimalList());

		Gage camelGage = new Gage();
		camelGage.setAnimalList(camelList);

		//１匹入っている。
		//System.out.println(camelGage.getAnimalList());

		//檻を動物ショップに設定
		AnimalShop shop = new AnimalShop();

		//ライオンを設定
		shop.setGageList(raionGage);

		//ゾウを設定
		shop.setGageList(zouGage);

		//らくだを設定
		shop.setGageList(camelGage);

		//ショップ内の動物をすべて表示
		shop.showAnimalAll();

	}

檻

/**
 * 檻クラス
 * @author
 */
public class Gage {

	/** 動物リスト */
	private ArrayList<Animal> animalList;

	/**
	 * コンストラクタ<br>
	 * 動物リストを初期化
	 */
	public Gage(){

		//ここは初期化は必要なのか？
		this.animalList = new ArrayList<Animal>();
	}

	/**
	 * animalListを設定するメソッド
	 * @param animalList 動物リストを設定する
	 */
	public void setAnimalList(ArrayList<Animal> animalList) {

		this.animalList = new ArrayList<Animal>();
		this.animalList = animalList;
		//System.out.println(this.animalList);

	}

	/**
	 * animalListを取得するメソッド
	 * @return animalList 動物リスト
	 */
	public ArrayList<Animal> getAnimalList() {
		return animalList;
	}
}

動物

/**
 * 動物クラス
 * @author
 */
public class Animal {
	/** 動物の名前 */
	private String name;

	/**
	 * 動物の名前を設定
	 * @param name 名前
	 */
	public void setName(String name){		
		this.name = name;
	}

	/**
	 * 動物の名前を取得
	 * @return String 動物の名前
	 */
	public String getName(){
		return this.name;
	}
}

animalショップ

/**
 * @author
 */
public class AnimalShop {

	/** 檻リスト */
	private HashMap<String, Gage> gageList;

	/**
	 * コンストラクタ<br>
	 * 檻リストを初期化
	 * @param shopName
	 */
	public AnimalShop(){
		this.gageList = new HashMap<String, Gage>();
	}

	/**
	 * 引数の檻をフィールドの檻リストに設定<br>
	 * 檻内の動物名をキーに檻リストに設定する
	 * @param gage 檻
	 */
	public void setGageList(Gage gage){

		//フィールドの檻リストを初期化
		//this.gageList = new HashMap<String, Gage>();
		//↑初期化ではなくてobjectを代入している。

		//檻内の動物名を取得
		ArrayList<Animal> animalList = gage.getAnimalList();

		//各動物リストの名前は同じであるため要素0番目を取得
		Animal animal = (Animal)animalList.get(0);

		//動物名を取得
		String name = animal.getName();

		//動物名をキーに檻を設定
		gageList.put(name , gage);

	}

	/**
	 * 檻リスト内の動物を全て表示するメソッド
	 */
	public void showAnimalAll(){

		for(Gage gage: gageList.values()){
			//動物リストを取得
			ArrayList<Animal> animalList = gage.getAnimalList();

			//動物名　引数を表示
			for(Animal animal: animalList){
				System.out.println("動物名:" + animal.getName());
				System.out.println("匹数:" + animalList.size());
			}
		}
	}
}

javaを学習中です。
今、コレクションフレームワークについて勉強をしています。

理想の表示
動物名：ゾウ
匹数：2匹
動物名：ライオン
匹数：3匹
動物名：らくだ
匹数：1匹

ここまで作れたのですが、
動物名：らくだ
匹数：1匹
しか表示しないです。

自分でどこがダメなのかがわからなくなってしまいました。

どのようにしたらよいのでしょうか？

Comments

[m.ts10806]

タイトルが起きている問題に対して大きすぎるので、もっと質問内容に寄せたタイトルに調整いただけたらと。

[swordone]

Mainが2つあるけど？？

[tanaka002]

申し訳ありません。
修正しました。

Answer 1

setGageListでgageListに毎度新たなオブジェクトを代入しているからでしょう。
まあ、setと言う名には反しないのかもしれませんが。

あと、肝心のAnimalShop#showAnimalAllはどこでしょうか。

質問編集を受けて

//動物名　引数を表示
for(Animal animal: animalList){
System.out.println("動物名:" + animal.getName());
System.out.println("匹数:" + animalList.size());
}

動物名は全て同一なのですから、for文を使う意味が無いです。

Comments

[tanaka002]

すみません。抜けてましたので追加しました。

[LouiS0616]

逆にAnimalShopクラス全体が抜け落ちてませんか？再編集して下さい。
また、回答前半の問題が解決しない限りはshowAnimalAllをどう実装しても完成は難しいです。

[LouiS0616]

質問を編集して実行結果あるいはエラーに変化があったのなら、そのことも追記して下さい。
手元で検証するのは正直億劫です。

Answer 2

解決済みなのに簡略化したいとはこれ如何に…って感じなのですが
前提：Shopクラスに
Map<String,Gage> gageList
が定義、要素の追加済み

  public void showAnimal() {
    for(Map.Entry<String, Gage> entry : gageList.entrySet()) {
      System.out.println("動物名：" + entry.getKey());
      System.out.println("匹数：" + entry.getValue().getList().size());
      System.out.println(); //ただの改行
    }
  }

Answer 3

		//HashMapからキーを選択してobjectに代入
		Gage oj= gageList.get("ゾウ");

		ArrayList<Animal> animalList = oj.getAnimalList();

		System.out.println("動物名:" + animalList.get(0).getName());
		System.out.println("匹数:" + animalList.size() +  "\n");

		Gage oj2= gageList.get("ライオン");

		ArrayList<Animal> animalList2 = oj2.getAnimalList();

		System.out.println("動物名:" + animalList2.get(0).getName());
		System.out.println("匹数:" + animalList2.size()  +  "\n");

		Gage oj3= gageList.get("らくだ");

		ArrayList<Animal> animalList3 = oj3.getAnimalList();

		System.out.println("動物名:" + animalList3.get(0).getName());
		System.out.println("匹数:" + animalList3.size()  +  "\n");

このような形で回答は求められましたが、簡略化をしたいです。

現在の表示
動物名:ゾウ
匹数:2

動物名:ライオン
匹数:3

動物名:らくだ
匹数:1