質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.37%

for文中のreturn文について

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 769

score 42

以下の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", "蛍光ペン6色セット", 420, 0, 0));
        list.add(new Item("A00401", "シャープペンシル", 100, 0, 0));
        list.add(new Item("A00501", "鉛筆H(1ダース)", 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", "2穴パンチ", 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;
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2020/08/26 09:42

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

    キャンセル

回答 1

checkベストアンサー

+1

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


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

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

ができます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.37%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る