「このメソッドは型 ○○○○ の結果を戻す必要があります」のエラーの原因がわからない

Question

初心者です。

手持ちの参考書にある以下のコードをeclipseにて実行したところ、
次のエラーメッセージが出てしまい、原因をネットで色々調べたのですが分からず困っています。

ItemDAOクラスにて、戻り値であるItem型ArrayListのitemsを「return items;」と書いて返しているのになぜでしょうか...？

ご教授をいただけると有り難いです。

Exception in thread "main" java.lang.Error: Unresolved compilation problem:

このメソッドは型 ArrayList<Item> の結果を戻す必要があります

at ItemDAO.findByMinimumPrice(ItemDAO.java:9)
at Main.main(Main.java:5)

import java.util.ArrayList;
public class Main {
	public static void main(String[] args) {
		System.out.println("１円以上のアイテム一覧表を表示します");
		ArrayList<Item> items = ItemDAO.findByMinimumPrice(1);
		for(Item item : items) {
			System.out.printf("%10s%4d%4d",item.getName(), item.getPrice(), item.getWeight());
		}
	}

}

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class ItemDAO {
    public static ArrayList<Item> findByMinimumPrice(int i) {
        try {
            Class.forName("org.h2.Driver");
        }catch(ClassNotFoundException e) {
            e.printStackTrace();
        }
        Connection con = null;
        try {
        	con = DriverManager.getConnection("jdbc:h2:~/example", "sa", "");
            PreparedStatement pstmt = con.prepareStatement
            		("SELECT * FROM ITEMS WHERE PRICE > ?");
            pstmt.setInt(1, i);
            ResultSet rs = pstmt.executeQuery();
            // ここでItemを入れていくArrayListを準備
            ArrayList<Item> items = new ArrayList<Item>();
            while(rs.next()) {      // 結果表の全行を１つずつ処理
                Item item = new Item();
                item.setName(rs.getString("NAME"));     /* 1行分の情報を */
                item.setPrice(rs.getInt("PRICE"));      /* インスタンスに変換 */
                item.setWeight(rs.getInt("WEGIHT"));
                items.add(item);                        /* インスタンスをArrayListに追加 */
            }
            rs.close();
            pstmt.close();
            return items;                               /* 最後にArrayListを返す */
        } catch(SQLException e) {
            e.printStackTrace();
        } finally {
            if(con != null) {
                try {
                    con.close();
                } catch(SQLException e) {
                    e.printStackTrace();
                }
            }
        }
    }
}

public class Item {
    private String name; 
    private int price;
    private int weight;
    public String getName() {
    	return this.name;
    }
    public int getPrice() {
    	return this.price;
    }
    public int getWeight() {
    	return this.weight;
    }
    public void setName(String name) {
    	this.name = name;
    }
    public void setPrice(int price) {
    	this.price = price;
    }
    public void setWeight(int weight) {
    	this.weight= weight;
    }
}

Answer 1

SQLExceptionが発生した場合、return items;を通りません。

Comments

[Isikoro]

ご回答ありがとうございました。

頂いた回答をヒントにエラー部分を特定しようと思い、デバッグ機能で1行ずつチェックしてみたのですがどうも、

ItemDAOクラスの　public static ArrayList<Item> findByMinimumPrice(int i) {

の時点（SQL接続の前？）で、エラーメッセージが出ているようです。

もしそうなら、SQLException以前の問題なのかなと思ったのですが、どうなのでしょうか....。

[Isikoro]

あ、ちなみに、

ItemDAOクラスの　public static ArrayList<Item> findByMinimumPrice(int i) {

の行に、実行前に既に赤い×がついていてカーソルを合わせると
既に以下のメッセージが表示されています。
************************************************************
この行に複数マーカーがあります
- このメソッドは型 ArrayList<Item> の結果を戻す必要があります
- メソッド・ブレークポイント:ItemDAO [エントリー] -
************************************************************

[momon-ga]

> SQLException以前の問題なのかなと思ったのですが、どうなのでしょうか....。
はい。その通りです。コンパイルエラーなので、それを直してください。
実際に、SQLExceptionが発生するか関係なく、発生した場合に戻り値が返せないコードになっています。

[Isikoro]

そうなんですね。頑張ってエラーの原因を見つけたいと思います。
ありがとうございました。

[momon-ga]

えっと、コンパイルエラーの場所というか、原因は
SQLExceptonが発生する場合、戻り値を返さないコードになってるからと回答していますが
実際に、どうしたいっていうの方針がありますか？
なければ、e.printStackTrace();のあとに、return null;でコンパイルエラー自体は消えると思います。

[Isikoro]

＞e.printStackTrace();のあとに、return null;

これを教えて頂いてようやく回答の意味するところが理解できました。
最初からそこまで頭が回れば良かったのですが。。。すみません。。。
上記の修正をしたら目的通りの結果を得ることが出来ました。
本当に助かりました。
ありがとうございました。

Answer 2

既に書かれているとおり、return items;を通らないルートが存在するのが問題です。

JAVAでは、返り値のあるメソッドでは必ず返り値を返さないといけません。
例えば以下のように書いた場合、aが0以下である場合には、returnを通ることなく、
メソッドは終了します。JAVAではこのようなコードはエラーとしてはじいてくれます。

if(a<0){
  return 0;
}

今回の場合は、tryブロックの中で、例外が発生した場合、ブロックのそれ以降を飛ばして、
catch()やfinallyブロックの中に処理が飛ばされることになります。
その為、tryブロックの中の最後にあるreturn items;は飛ばされてしまい、実行されません。
JAVAのコンパイラはそういう事態を検知して、return items;が実行されない場合がある、と
エラーを出しているのです。
たとえ、例外が起きた時でも、何らかのreturnを返せるよう、見直してみてください。

Comments

[Isikoro]

詳細にありがとうございました！
理解が深まりました。