for文中のreturn文について

Question

以下のselectItemメソッドでitemを返したいのですが、

returnを記載していても
「このメソッドは型 Item の結果を戻す必要があります」
というエラーが出てしまいます。

itemを返しつつ解決する方法を教えて頂きたいです。
必要なソースは以下に記載しておきます。

selectItemメソッド

  public static Item selectItem(String name) {

    	//Item item = new Item(name,"",0,0,0);

    	for(Item item:list) {
            if (item.getName().equals(name)) {
                return item;
            }
        }
        //Item item = new Item(name,"",0,0,0);
　　　　　//return list.get(list.indexOf(item));
　　　　　//return item;
    }

RecordManager

package jp.practice.sales;

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

/**
 * 商品データ全体を保持するクラス
 */
public final class RecordManager {

    /** 商品データ */
    private static final List<Item> list = new ArrayList<Item>();
    static {
        list.add(new Item("A00101", "油性ボールペン", 60, 0, 0));
        list.add(new Item("A00201", "極細ボールペン", 120, 0, 0));
        list.add(new Item("A00301", "蛍光ペン６色セット", 420, 0, 0));
        list.add(new Item("A00401", "シャープペンシル", 100, 0, 0));
        list.add(new Item("A00501", "鉛筆H（１ダース）", 400, 0, 0));
        list.add(new Item("B00101", "無線綴ノートA4", 100, 0, 0));
        list.add(new Item("B00201", "リングノートA4", 120, 0, 0));
        list.add(new Item("B00301", "領収書", 350, 0, 0));
        list.add(new Item("C00101", "はさみ（青）", 128, 0, 0));
        list.add(new Item("C00201", "ステープラー", 338, 0, 0));
        list.add(new Item("C00301", "２穴パンチ", 128, 0, 0));
        list.add(new Item("C00401", "ゼムクリップ", 98, 0, 0));
        list.add(new Item("C00501", "消しゴム", 58, 0, 0));
    }

    /**
     * 引数で指定された商品名に一致する商品データを返却
     * @param name 検索キーとなる商品名
     * @return 検索結果の商品データ
     */
    public static Item findItem(String goodsName) {
    	int index = list.indexOf(new Item("", goodsName, 0,0,0));
    	return list.get(index);
    }

    /**
     * 商品データを返す。
     * @return 商品データ
     */
    public static List<Item> getNameist() {
        return list;
    }

    /**
     * 全員の名前のリストを返す。
     * @return 名前の配列
     */
    public static String[] makeNameList() {
        String[] nameList = new String[list.size()];
        for (int i = 0; i < list.size(); i++) {
            nameList[i] = (list.get(i)).getName();
        }
        return nameList;
    }

    /**
     * 名前で検索し、その人のスケジュールデータを返す。
     * @param name 商品名
     * @return 一商品データ
     */
    public static Item selectItem(String name) {

    	//Item item = new Item(name,"",0,0,0);

    	for(Item item:list) {
            if (item.getName().equals(name)) {
                return item;
            }
        }
        //Item item = new Item(name,"",0,0,0);
　　　　　//return list.get(list.indexOf(item));
　　　　　//return item;
    }
}

Item

package jp.practice.sales;

public class Item {

	/** 商品ID */
    private String id;
    /** 商品名 */
    private String name;
    /** 単価 */
    private int price;
    /** 売上点数 */
    private int quantity;
    /** 小景 */
    private int subtotal;

	public Item(String id, String name, int price, int quantity, int subtotal) {
		this.id = id;
		this.name = name;
		this.price = price;
		this.quantity = quantity;
		this.subtotal = subtotal;
	}

	/**
	 * @return id
	 */
	public String getId() {
		return id;
	}

	/**
	 * @param id セットする id
	 */
	public void setId(String id) {
		this.id = id;
	}

	/**
	 * @return name
	 */
	public String getName() {
		return name;
	}

	/**
	 * @param name セットする name
	 */
	public void setName(String name) {
		this.name = name;
	}

	/**
	 * @return price
	 */
	public int getPrice() {
		return price;
	}

	/**
	 * @param price セットする price
	 */
	public void setPrice(int price) {
		this.price = price;
	}

	/**
	 * @return quantity
	 */
	public int getQuantity() {
		return quantity;
	}

	/**
	 * @param quantity セットする quantity
	 */
	public void setQuantity(int quantity) {
		this.quantity = quantity;
	}

	/**
	 * @return subtotal
	 */
	public int getSubtotal() {
		return subtotal;
	}



	/**
	 * @param subtotal セットする subtotal
	 */
	public void setSubtotal(int subtotal) {
		this.subtotal = subtotal;
	}
	/* (非 Javadoc)
	 * @see java.lang.Object#hashCode()
	 */
	@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime * result + ((name == null) ? 0 : name.hashCode());
		return result;
	}
	/* (非 Javadoc)
	 * @see java.lang.Object#equals(java.lang.Object)
	 */
	@Override
	public boolean equals(Object obj) {
		if (this == obj) {
			return true;
		}
		if (obj == null) {
			return false;
		}
		if (!(obj instanceof Item)) {
			return false;
		}
		Item other = (Item) obj;
		if (name == null) {
			if (other.name != null) {
				return false;
			}
		} else if (!name.equals(other.name)) {
			return false;
		}
		return true;
	}
}

Comments

[m.ts10806]

解決されたようですが解決したようには思えません。
「一致しないときにどうするか」　設計どうなってますか？それ次第で作り方も違います。

Answer 1

public static Item selectItem(String name) {
    for(Item item:list) {
        if (item.getName().equals(name)) {
            return item;
        }
    }
    // 同じ名前がなかったときはnullを返す
    return null;
}

が一番単純な解決法ですが、nullを扱いたくない場合は、

• Optionalを使う
• 空のItemを作成するコンストラクタを別途作成し、空であることを判定するロジックを設ける

ができます。