Q&A
コードのインデントがぐちゃぐちゃなので中身読んでませんが、タイトルが気になったのでそこだけ指摘します。
ArrayListとHashMapでは構造も概念も大きく違います。役割も違います。
なぜそうする必要があるのでしょう。
sqlでデータベースを作りそれをjavaを使ってweb上で検索するというシステムを作っています。
映画のタイトルや公開年数などの情報が複数詰まったデーターベースを作成して、web上で検索しようとしているところのなのですがうまくいきません。
メールで次にどうしろと指示をいただいているのですが難しくよくわからないです。
今現在もらっている指示は
『開発中のシステムは，検索のキーワードを指定して，そのキーワード
が，タイトル名か監督名か主演名かそのどれでもよいかを指定する仕
様でしたよね．
この仕様を実現できるように，現在のModel.javaで実行している
SELECT文を変更してください．』
『この仕様に対応したSELECT文を実行するには，Model.javaは，
当然，入力された検索のキーワードと，タイトル名か監督名か主演
名かそのどれでもよいかを識別する情報を取得する必要があります．
取得する方法は，基本的に，現在のModelクラスで行っている方法
でいいでしょう．
つまり，setQのようなメソッドを作って，Controllerのクラス
から呼び出すようにしてください．』
という感じです、
Controller.javaのプログラム内で，検索のキーワードと，タイトル名か監督名か主演名かそのどれでもよい
かを識別する情報として，固定した値を設定して，Modelクラスに検索させる形にしようとしています。
DataAccessクラスではHashMapにTitleListResultクラスを入れている。
ModelクラスではArrayListにTitleListResultクラスを入れている状態なのですが
これをいま
ModelクラスでもHashMapにTitleListResultクラスを入れている
という状態にしたいのですがうまくいきません。
package movie; import java.util.*; import java.sql.*; import java.util.HashMap; public class Model { private String q; private String c; private Connection conn = null; private PreparedStatement stmt = null; private ResultSet resultSet = null; private HashMap<Integer,TitleListResult>Title_hash; public void execute(){ System.out.println("Hello"); try { conn = ConnectUtilMy.connectDatabase(); // 住所の検索 PreparedStatement pstmt = conn.prepareStatement( "SELECT title_name,years_publication,performance_income,production_cost,public_evaluation,screening_time,genre_name,original_name,cast_name,directed_name " + "FROM movie inner join original_list on movie.original_number=original_list.original_number "+ "inner join genre_list on movie.genre_number=genre_list.genre_number "+ "inner join directed_list on movie.directed_number=directed_list.directed_number "+ "inner join cast_list on movie.cast_number=cast_list.cast_number where title_name like ? or directed_name like ?"); pstmt.setString(1, q + "%"); pstmt.setMaxRows(50); ResultSet rs = pstmt.executeQuery(); // 検索結果の取得 results = new ArrayList<TitleListResult>(); TitleListResult result = new TitleListResult(); while (rs.next()) { TitleListResult[] result = new String[10]; result[0] = rs.getString("movie_number"); result[1] = rs.getString("years_publication"); result[2] = rs.getString("performance_income"); result[3] = rs.getString("production_cost"); result[4] = rs.getString("public_evaluation"); result[5] = rs.getString("screening_time"); result[6] = rs.getString("genre_number"); result[7] = rs.getString("directed_number"); result[8] = rs.getString("original_number"); result[9] = rs.getString("cast_number"); results.add(result); } // 切断 rs.close(); pstmt.close(); conn.close(); } catch (Exception ex) { ex.printStackTrace(); } } public HashMap<TitleListResult> getResults() { return results; } public void setQ(String q) { this.q = q; } public void setC(String c) { this.c = c; } }
package
1import java.sql.*; 2import java.util.HashMap; 3 4public class DataAccess 5{ 6 private Connection conn = null; 7 private PreparedStatement stmt = null; 8 private ResultSet resultSet = null; 9 private HashMap<Integer,TitleListResult>Title_hash; 10 11 12 13 14 private String sql = null; 15 16 17 public HashMap<Integer,TitleListResult>result_search(String search){ 18 Title_hash = new HashMap<>(); 19 int i = 0; 20 21 22 try { 23 sql = "SELECT title_name,years_publication,performance_income,production_cost,public_evaluation,screening_time,genre_name,original_name,cast_name,directed_name " + 24 "FROM movie inner join original_list on movie.original_number=original_list.original_number "+ 25"inner join genre_list on movie.genre_number=genre_list.genre_number "+ 26"inner join directed_list on movie.directed_number=directed_list.directed_number "+ 27"inner join cast_list on movie.cast_number=cast_list.cast_number where title_name like ? or directed_name like ?"; 28 conn = ConnectUtilMy.connectDatabase(); 29 stmt = conn.prepareStatement(sql); 30 stmt.setString(1,"%"+search+"%"); 31 stmt.setString(2,"%"+search+"%"); 32 System.out.println(search); 33 resultSet = stmt.executeQuery(); 34 while (resultSet.next()) { 35 TitleListResult result = new TitleListResult(); 36 result.Set_Title_name( resultSet.getString("title_name")); 37 result.Set_years_publication( resultSet.getString("years_publication")); 38 result.Set_performance_income( resultSet.getDouble("performance_income")); 39 result.Set_Production_cost( resultSet.getDouble("production_cost")); 40 result.Set_performance_income( resultSet.getDouble("performance_income")); 41 result.Set_Public_evaluation( resultSet.getDouble("public_evaluation")); 42 result.Set_Screening_time( resultSet.getString("screening_time")); 43 result.Set_Genre_name( resultSet.getString("genre_name")); 44 result.Set_directed_name( resultSet.getString("directed_name")); 45 result.Set_cast_name( resultSet.getString("cast_name")); 46 result.Set_original_name( resultSet.getString("original_name")); 47 48 Title_hash.put(i,result); 49 i++; 50 51 52 } 53 } 54 catch (SQLException ex) { 55 System.out.println("エラーコード：" + ex.getErrorCode()); 56 System.out.println("SQL状態：" + ex.getSQLState()); 57 ex.printStackTrace(); 58 } 59 finally { 60 try { 61 if (resultSet != null) { 62 resultSet.close(); 63 } 64 } 65 catch (SQLException ex) { 66 ex.printStackTrace(); 67 } 68 try { 69 if (stmt != null) { 70 stmt.close(); 71 } 72 } 73 catch (SQLException ex) { 74 ex.printStackTrace(); 75 } 76 try { 77 if (conn != null) { 78 conn.close(); 79 } 80 } 81 catch (SQLException ex) { 82 ex.printStackTrace(); 83 } 84 85 } 86 return Title_hash; 87 } 88 89 90 91 92} 93
実質、下記の前の質問の続きですので、回答およびコメントしていただいた方々の厚意を無にしないためにも、ここに記載しておきます。
[ArrayList<String[]>では対応できないので対応できるようにするには？]
https://teratail.com/questions/282387
dodox86さん
＞ TitleListResult[] result = new String[10];
は、コンパイルエラーだろと思ったが、なんでこのコード？という部分の背景がわかりました。
基礎の部分を理解してくれないとアドバイスのしようがないなぁ。
どういったところがおかしいのかも分からないので八方塞がりです
＞@momon-ga さん
> 基礎の部分を理解してくれないとアドバイスのしようがないなぁ。
正直、私も回答を躊躇していましたし、言葉は悪いのですが、案の定の成り行きです。コンパイルエラーが取れても、問題が出たらご自分で対応できないのではないかなぁ。。。
> kaibasiraさん
> どういったところがおかしいのかも分からないので八方塞がりです
意味もわからずコード書くのってしんどくないですか？
わかる範囲をすこしづつ書いていって増やした方がよい気がします。
あと、Eclipseはいれた方がよいです。（コンパイルエラーには、すぐに気づけるようにする）
あえて苦言になりますが、そこまで分からないのであれば一番確実なのはある程度でも分かる方と一緒にハンズオン（＜面と向かって直に）だと思いますね。文字では埒があきませんし、無償なら尚更です。teratailは無償の作業代行サイトではないので、回答をしても実際にその回答の意味が分かっていなくてただ消費されるようですと、回答は離れがちになります。
まじめな話、学校の課題ではなく消化しなければいけない仕事なのであれば、MENTAとか、ココナラとかで有償でアドバイスを得、結果を獲得した方が早いと思います。（いや、私は使ったことないのですけど）
あなたの回答
tips
プレビュー
