メソッドを作って他のクラスから呼び出すようにするには

Question

コード
// パッケージの指定
package movie;

import java.util.*;
import java.sql.*;

public class Model {
	private String q;
	private ArrayList<TitleListResult> results;
        private Connection conn = null;
        private PreparedStatement stmt = null;
        private ResultSet resultSet = null;
       
               
                    
        public void execute(){
    System.out.println("Hello");  
		
                   try {      
                conn = ConnectUtilMy.connectDatabase();
	        TitleListResult result = new TitleListResult();
      	                        
			// 住所の検索
			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<String[]>();
			while (rs.next()) {
				String[] 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 ArrayList<TitleListResult> getResults() {
		return results;
	}

	public void setQ(String q) {
		this.q = q;
	}
     //この下が追加して書いたところです。
　　　　public void setC(String c) {
		this.c = c;
	}
}

// パッケージの指定
package movie;

import java.util.*;
import java.io.*;
import javax.servlet.*;
import javax.servlet.http.*;
import javax.servlet.annotation.*;

// サーブレットの名前（ここでは，Controllerというクラス名）を指定する
@WebServlet(name = "Controller", urlPatterns = {"/Controller"})
public class Controller extends HttpServlet {
	protected void doPost(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException {
         
		// view.jspで入力された検索キーkeyNumの値を取得する
		String q = request.getParameter("keyNum");
                q = new String(q.getBytes("8859_1"), "Windows-31J");

		if (q != null) {
			// Modelクラスのインスタンスを生成する
			Model model = new Model();
			// 検索キーを設定する
			model.setQ(q);
                        
			// 検索を実行する
			model.execute();
			// 検索結果を取得する
			ArrayList<String[]> results = model.getResults();
			// view.jspに検索結果を渡す
			request.setAttribute("results", results);
		}

		// view.jspに処理を転送する
		getServletContext().getRequestDispatcher("/view.jsp").forward(request, response);
	}
}
コード
```sqlでデータベースを作りそれをjavaを使ってweb上で検索するというシステムを作っています。


Controller.java
DataAccess.java
Model.java
TitleListResult.java
ConnectUtilMy.java

の五つと
view.jspというファイルをつかっています。


なお，実際のシステムでは，画面から入力された情報でデータベースを検索しますが，まずは，Controllerのプログラム内で，検索
のキーワードと，タイトル名か監督名か主演名かそのどれでもよいかを
識別する情報として，固定した値を設定して，Modelクラスに検索させる形でかまわないということになりました。


『開発中のシステムは，検索のキーワードを指定して，そのキーワード
が，タイトル名か監督名か主演名かそのどれでもよいかを指定する仕
様です．
この仕様を実現できるように，現在のModel.javaで実行している
SELECT文を変更してください．』

と言われ

『この仕様に対応したSELECT文を実行するには，Model.javaは，
当然，入力された検索のキーワードと，タイトル名か監督名か主演
名かそのどれでもよいかを識別する情報を取得する必要があります．
取得する方法は，基本的に，現在のModelクラスで行っている方法
でいいでしょう．
つまり，setQのようなメソッドを作って，Controllerのクラス
から呼び出すようにしてください．』

ということなのですがほかのメソッドを作ってControllerのクラスから呼び出すようにする
というのはどうしたらいいのでしょうか？

一応すでに書いてある部分をマネしてみて追加してみたのですがこれ以外に何を書き込めば
メソッドを作ったことになり呼び出せるのでしょうか？

Comments

[momon-ga]

まず「コードやエラーはマークダウンのcode機能を利用してご提示ください」

あと、かたくなに「コードやエラーはマークダウンのcode機能を利用してご提示ください。」という
指摘を受け入れないのは、なぜですか？

お前らの言うことは聞かない ＝ 回答も聞かない（いらない）

という意思表示でしょうか？

[kaibasira]

すみません、見落としていました。

[m.ts10806]

>html上で検索する

この表現を頑なに使い続ける理由を教えてください。以前も指摘したはず。
誰が見ても「HTMLにデータを持ってる」と解釈します。そして、それは物理的に無理であることを大抵の回答者は知っています。
そしてそれは、正しい用語を使えない人へのアドバイスの難しさを物語ります。
固有名詞は当然ですが、自分だけがわかる表現は控えたほうが良いかと思います。

[kaibasira]

web上で検索する　と変えてみましたのですがこれで分かりづらかったりなどは大丈夫でしょうか？

Answer 1

考え方というか、やってることは

//この下が追加して書いたところです。
public void setC(String c) {
    this.c = c;
}

で、問題ないと思います。

呼び出しも、すでに書かれているように

            // Modelクラスのインスタンスを生成する
            Model model = new Model();
            // 検索キーを設定する
            model.setQ(q);
            // 検索種別を設定する(タイトル名、監督名、主演名、どれでもよい)
            model.setC(c);

            // 検索を実行する
            model.execute();

に、近いものになるはずです。
Modelクラスにメソッド(setC)を作って、別のクラス(Controller)から呼び出す。
は、実現できていると考えてよいと思います。（使いやすさとか、メソッド名とか考慮することはありますが）