Java ArrayListのindexOfメソッドの返却値が-1となってしまう。

Question

■調べたいこと
Java ArrayListのindexOfメソッドの返却値が-1となってしまう。
返却値に0以上が返却されるように該当ソースを修正したい。
想定では検索したIDでリストを捜索すると該当する位置が返却されます。

■試したこと
・items.indexOfの引数にitemIDだけを指定
・items.indexOfの引数にCDクラスを指定
・items.indexOfの引数にItemクラスを指定

該当箇所
int index = items.indexOf(new Video(itemID,"",0, 0, ""));

プログラム全体

import java.util.ArrayList;
import java.util.List;

public class Shop {

	private List<Item> items;

	public Shop(){

		items = new ArrayList<Item>();

	}

	public void addItem(Item item){

		items.add(item);
		List<Item> items2 = new ArrayList<Item>();
		items2 = items;


	}

	public void showItem(){

		for(int i = 0; i < items.size(); i++){

			Item item = items.get(i);

			item.printItemDetail();

		}
	}

	public void sellItem(String itemID){

		int index = items.indexOf(new Video(itemID,"",0, 0, ""));

		System.out.println(items.get(index).getItemName()+"を"+items.get(index).getPrice()+"円で販売しました。");

		Item item = new Item();
		item.decrementStock();

	}


}

class Video extends Item {

	private String movieDirector;

	public Video(String itemID,String itemName,int price,int stock,String movieDirector){
		this.itemID = itemID;
		this.itemName = itemName;
		this.price = price;
		this.stock = stock;
		this.movieDirector = movieDirector;
	}

	public void printItemDetail(){

		System.out.println(itemID + "　" +itemName  + "　" + price  + "円　" + stock + "個　" + movieDirector + "　");

	}
}

class CD extends Item {

	String artist;

	Item item = new Item();


	public CD(String itemID,String itemName,int price,int stock,String artist){

		Item item = new Item();

		this.itemID = itemID;
		this.itemName = itemName;
		this.price = price;
		this.stock = stock;
		this.artist = artist;
	}

	public void  printItemDetail(){

		System.out.println(itemID + "　" +itemName  + "　" + price  + "円　" + stock + "個　" + artist + "　");

	}

}

class Item {

	protected String itemID;
	protected String itemName;
	protected int price;
	protected int stock;

public Item(){

	}

	public Item(String itemID, String itemName, int price, int stock){

		this.itemID = itemID;
		this.itemName = itemName;
		this.price = price;
		this.stock = stock;

	}

	public void printItemDetail(){

		System.out.println(itemID + " " +itemName  + " " + price  + " " + stock + " " );

	}

	public String getItemID(){
		return itemID;
	}

	public String getItemName(){
		return itemName;
	}

	public int getPrice(){
		return price;
	}

	public void decrementStock(){
		stock = stock-1;
	}

}

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

>Java ArrayListのindexOfメソッドの返却値が-1となってしまう。

だから何でしょうか。
質問がありません。

自身が調べたこと試したこと、あと「どうなる想定なのか」「このコードが達成したい目的」「仕様」を明記してください。

Answer 1

参考：
http://deepfolte.hatenablog.com/entry/20080523/1211511791
のように、Videoクラスにequals(Video)比較メソッドを作れば良いそうです。

Answer 2

indexOfは同一のものがない場合に -1が返されます。

int index = items.indexOf(new Video(itemID,"",0, 0, ""));

新規生成したインスタンス（アドレス？）は、いかなる既存のインスタンスとは同一になりえません
-1にしかなりようがないですね

Comments

[momon-ga]

ちょっと言葉足らずでは？
以下のコードは、-1でなく0が返ってきますけど。

List<String> list = new ArrayList<>();
list.add("s");

System.out.println(list.get(0) == "s");
System.out.println(list.get(0) == new String("s"));
System.out.println(list.indexOf(new String("s")));

[退会済みユーザー]

arraylistのindexofの実装はequals判定では？