ArrayListに商品情報を登録して表示させたい

Question

ShopMainクラスで実行されるshop.addItemの内容をShopクラスのArrayListに登録して表示させるメソッドを実装したい。
4つの商品情報を並べて表示したい。
↓の状況だとshop.addItemで最後のメソッドのみ拾われてしまう。

public class ShopMain {
	public static void main(String[] args) {
		Shop shop = new Shop();

		shop.addItem(new Video("V0001","ローマの休日　",2980,6,"ウィリアム・ワイラー"));
		shop.addItem(new Video("V0002","風と共に去りぬ",2980,5,"ヴィクター・フレミング"));
		shop.addItem(new CD(   "C0001","ビートルズ全集",3000,1,"ビートルズ"));
		shop.addItem(new CD(   "C0002","スリラー　　　",2500,9,"マイケル・ジャクソン"));

		System.out.println("CODE\tタイトル\t価格\t在庫\t監督／アーティスト");
		System.out.println("--------------------------------------------------------------");
		shop.showItem();
		System.out.println("--------------------------------------------------------------");
		System.out.println();
		shop.sellItem("V0001");
		shop.sellItem("C0002");
		System.out.println();
		System.out.println("CODE\tタイトル\t価格\t在庫\t監督／アーティスト");
		System.out.println("--------------------------------------------------------------");
		shop.showItem();
		System.out.println("--------------------------------------------------------------");
	}
}

public class Shop {

	private ArrayList<String> items; //商品情報を格納するリスト
	Item cd = new CD(null, null, 0, 0, null);
	Item video = new Video(null, null, 0, 0, null);

	/**
	 * @param items
	 * コンストラクタ
	 */
	public Shop() {
		ArrayList<String> items = new ArrayList<>();
		this.items = items;
	}

	//商品情報をitemsに登録する
	public void addItem(Item item) {
			items.add(item.getItemID());
			items.add(item.getItemName());
			items.add(String.valueOf(item.getPrice()));
			items.add(String.valueOf(item.getStock()));
			items.add(cd.getArtist());
			items.add(video.getMovieDirector());

			System.out.println("アイテムサイズ　"+items.size());
	}

	//itemsに登録されている商品情報を全て画面に表示する
	public void showItem() {
			cd.printItemDetail();
			video.printItemDetail();
			System.out.println(items);
	}

	//itemsに登録されている商品情報から該当する商品を検索しメッセージを表示する
	public void sellItem(String itemID) {
//		System.out.println(items.get(1) + "を" + items.get(2) + "で販売しました");
//		Item.decrementStock();
	}


}

class Item {
	private static String itemID; //商品ID

	private static String itemName; //商品名

	private static int price; //値段

	private static int stock; //在庫数

	//デフォルトコンストラクタ
	public Item() {}

	public String getMovieDirector() {
		return null;
	}

	public String getArtist() {
		return null;
	}

	//コンストラクタ
	public Item(String itemID, String itemName, int price, int stock) {
		Item.itemID = itemID;
		Item.itemName = itemName;
		Item.price = price;
		Item.stock = stock;
	}

	//商品ID、商品名、値段、在庫数を表示するメソッド
	public void printItemDetail() {
		System.out.println(itemID + "\t"+ itemName + "\t" + price + "円\t" + stock + "個");
	}

	/**
	 * @return itemID
	 * 商品ID を返却するメソッド
	 */
	public String getItemID() {
		return itemID;
	}

	/**
	 * @return itemName
	 * 商品名を返却するメソッド
	 */
	public  String getItemName() {
		return itemName;
	}
	/**
	 * @return price
	 * 値段を返却するメソッド
	 */
	public  int getPrice() {
		return price;
	}

	public  int getStock() {
		return stock;
	}

	//在庫数から1個販売するメソッド
	public static void decrementStock() {
		stock -= 1;
	}


}

class CD extends Item {
	private static  String artist; //アーティスト名

	/**
	 * @param itemID
	 * @param itemName
	 * @param price
	 * @param stock
	 * コンストラクタ
	 */
	public CD(String itemID, String itemName, int price, int stock, String artist) {
		super(itemID, itemName, price, stock);
		CD.artist = artist;
	}

	public  String getArtist() {
		return artist;
	}

	//商品ID、商品名、値段、在庫数、アーティスト名を表示するメソッド
	@Override
	public void printItemDetail() {
		System.out.println(super.getItemID() + "\t" + super.getItemName() + "\t" + super.getPrice() + "円\t" + super.getStock() + "個\t" + getArtist());
	}

}

class Video extends Item {
	private static String movieDirector; //監督名

	/**
	 * @param movieDirector
	 * コンストラクタ
	 */
	public Video(String itemID, String itemName, int price, int stock, String movieDirector) {
		super(itemID, itemName, price, stock);
		Video.movieDirector = movieDirector;
	}

	public String getMovieDirector() {
		return movieDirector;
	}

	//。商品ID、商品名、値段、在庫数、監督名を表示するメソッド
	@Override
	public void printItemDetail() {
		System.out.println(super.getItemID() + "\t" + super.getItemName() + "\t" + super.getPrice() + "円\t" + super.getStock() + "個\t" + getMovieDirector());
	}

宜しくお願い致します。

Answer 1

private ArrayList<String> items; //商品情報を格納するリスト

"商品情報" は Item クラスのことだと思います。
ArrayList<Item> とし、 addItem メソッドはパラメータの item をそのまま items に add する必要があります。
showItem も「itemsに登録されている商品情報を全て画面に表示する」なら items の内容をループ等で表示する必要があると思います。
Item 類のフィールドが static では意味がありません。

---

import java.util.*;

public class Main {
  public static void main(String[] args) {
    Shop shop = new Shop();

    shop.addItem(new Video("V0001","ローマの休日",2980,6,"ウィリアム・ワイラー"));
    shop.addItem(new Video("V0002","風と共に去りぬ",2980,5,"ヴィクター・フレミング"));
    shop.addItem(new CD("C0001","ビートルズ全集",3000,1,"ビートルズ"));
    shop.addItem(new CD("C0002","スリラー",2500,9,"マイケル・ジャクソン"));

    showAssortment(shop);

    System.out.println();
    shop.sellItem("V0001");
    shop.sellItem("C0002");
    System.out.println();

    showAssortment(shop);
  }
	
  private static void showAssortment(Shop shop) {
    System.out.println("CODE\tタイトル\t\t価格\t在庫\t監督／アーティスト");
    System.out.println("--------------------------------------------------------------");
    shop.showItems("%s   %-8s %4d円 %4d個    %s");
    System.out.println("--------------------------------------------------------------");
  }
}

class Shop {
  private List<Item> items = new ArrayList<>(); //商品情報を格納するリスト

  //商品情報をitemsに登録する
  public void addItem(Item item) {
    items.add(item);
    //System.out.println("アイテムサイズ　"+items.size());
  }

  //itemsに登録されている商品情報を全て画面に表示する
  public void showItems(String format) {
    for(Item item : items) System.out.println(item.formatting(format));
  }

  //itemsに登録されている商品情報から該当する商品を検索しメッセージを表示する
  public void sellItem(String itemID) {
    Item item = getItem(itemID);
    if(item == null) return; //無効な id なら無視
    System.out.println(item.name+ "を" + item.getPrice() + "円で販売しました");
    item.decrementStock();
  }
	
  private Item getItem(String id) {
    for(Item item : items) if(item.id.equals(id)) return item;
    return null; //無かった
  }
}

abstract class Item {
  final String id; //商品ID
  final String name; //商品名
  protected int price; //値段
  protected int stock; //在庫数

  //コンストラクタ
  public Item(String id, String name, int price, int stock) {
    this.id = id;
    this.name = name;
    this.price = price;
    this.stock = stock;
  }

  public int getPrice() { return price; }
  public int getStock() { return stock; }

  public void decrementStock() { --stock; }

  abstract String formatting(String format);
  protected String preformatString(String str, int len) {
    while(str.length() < len) str += "　";
    return str;
  }
}

class CD extends Item {
  private String artist; //アーティスト名

  public CD(String id, String name, int price, int stock, String artist) {
    super(id, name, price, stock);
    this.artist = artist;
  }

  @Override
  public String formatting(String format) {
    return String.format(format, id, preformatString(name,7), price, stock, artist);
  }
}

class Video extends Item {
  private String movieDirector; //監督名

  public Video(String id, String name, int price, int stock, String movieDirector) {
    super(id, name, price, stock);
    this.movieDirector = movieDirector;
  }

  @Override
  public String formatting(String format) {
    return String.format(format, id, preformatString(name,7), price, stock, movieDirector);
  }
}

CODE	タイトル		価格	在庫	監督／アーティスト
--------------------------------------------------------------
V0001   ローマの休日　  2980円    6個    ウィリアム・ワイラー
V0002   風と共に去りぬ  2980円    5個    ヴィクター・フレミング
C0001   ビートルズ全集  3000円    1個    ビートルズ
C0002   スリラー　　　  2500円    9個    マイケル・ジャクソン
--------------------------------------------------------------

ローマの休日を2980円で販売しました
スリラーを2500円で販売しました

CODE	タイトル		価格	在庫	監督／アーティスト
--------------------------------------------------------------
V0001   ローマの休日　  2980円    5個    ウィリアム・ワイラー
V0002   風と共に去りぬ  2980円    5個    ヴィクター・フレミング
C0001   ビートルズ全集  3000円    1個    ビートルズ
C0002   スリラー　　　  2500円    8個    マイケル・ジャクソン
--------------------------------------------------------------

Comments

[trainingnow]

ご丁寧にありがとうございます。
私が質問の際に記述し忘れたことがあります。

ShopMainクラスは修正せずに実装しなければなりません。
↑のクラスを修正せずに実装することは可能でしょうか。

[jimbe]

もちろん、 ShopMain をご提示のままにすることは可能です。
回答の ShopMain は、表示の共通部分を showAssortment メソッドに纏めたのと、表示の調整、 Shop.showItem() のメソッド名の変更およびパラメータの追加だけです。