データベースを読み込んでMapの戻り値を返すメソッド。戻り値が返っていないとエラーが出る。

Question

javaでPostgresqlデータベースを条件を付けて読み込んで、それを格納したMapの戻り値から情報を得るメソッドです。「このメソッドはMap＜String,List＜String>>の結果を戻す必要があります．」というエラーメッセージが出て、戻り値をvoidにするか、finally句の直後に、retuen null を付加することをEclipseから求められます。
try句のなかで　return map を置いて戻り値を返す命令を書いたつもりになっていたのですが、このコードではどこが悪くて、戻り値　map　が返らないのでしょうか。できましたら戻り値として　map　を返す適切なコーディング法をどうかご教示下さい。

public Map<String,List<String>> selectRepeaters() {
		try {
			Class.forName("org.postgresql.Driver");
		}
		catch(ClassNotFoundException e1) {
			e1.printStackTrace();
		}
		Connection con=null;
		try {
			con=DriverManager.getConnection("jdbc:postgresql://localhost:5432/mydb3","postgres","postgres724");

			/*リピーターのレコードをすべて読み込む処理*/
			PreparedStatement pstmt = con.prepareStatement("SELECT * FROM BOOK WHERE times>1 ORDER BY times DESC;");
			ResultSet rs = pstmt.executeQuery();

			HashMap<String,List<String>> map = new HashMap<String,List<String>>();
			while(rs.next()) {
				map.put(rs.getString(1),Arrays.asList(Integer.valueOf(rs.getInt(2)).toString(),rs.getString(3)));
			}
			return map;
		}	
		catch(Exception e){
			e.printStackTrace();
		}
		finally {
			if(con != null) {
				try {
					System.out.println("con.close");
					con.close();
				}
				catch(SQLException e){
					e.printStackTrace();
				}

			}
		}
		
		
	}

Answer 1

try の中で例外が発生した場合に return を通りません。

Comments

[onoko]

check句にも、return null を挿入したところ、エラーが消えました。ありがとうございました。いつもながらのご明察、感服します。

[jimbe]

正常に動作しない・できない場合にどうするかを決めて置くことも、メソッド、ひいてはシステム自体の設計として重要です。

例えばご提示のコードでは、ドライバの読み込みに失敗した場合には例外のログを表示してそのまま処理を続けています。

try {
Class.forName("org.postgresql.Driver");
}
catch(ClassNotFoundException e1) {
e1.printStackTrace();
}

しかし、ドライバが読めていないのに後続の処理が成功するわけは無いですよね。
ですのでこの e1.printStackTrace(); の後にも reurn null; を入れるか、もしくは例外そのものをさらに呼び出し側に伝える ( throw e1; ) 必要があるはずです。(例外を伝える場合はメソッドに throws の宣言も必要になります。)

また、ドライバが読めなければ、このメソッドだけでなくお作りになっているシステム全体としても問題なのではないでしょうか。
だとすれば、ドライバの読み込みはこのメソッドで行くのではなく、システムの起動時早々に読み込みを行って、問題があれば例外を表示して停止するようにしたほうが、ユーザーには問題があることが早く分かって良いのではないでしょうか。

[onoko]

教本にもドライバは最初に一度読めば済む、と書いてありました。おっしゃることがいちいちごもっともです。早速ドライバを起動時に一回読み込むようにシステム改変します。ご指導ありがとうございました。