JavaのServiceクラスでDAOクラスをインスタンス化できません。。。

Question

javaのServiceクラスでDAOクラスを呼びたいのですが、
うまくインスタンス化できません。。。
Serviceクラスの**dao.findByCondition();**でエラーが出ます。

エラー文 → 「型 ItemDAO のメソッド findByCondition(String, int) は引数 () に適用できません」

引数に型や引数名を入力してみても解決できませんでした。

わかる方がいらっしゃいましたらお力添えお願い致します。。

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

import jp.nib.ict21.ecsite.data.Item;

public class ItemDAO extends CommonDAO {
	//商品検索処理を行うクラス

	public List<Item> findByCondition(String keyword, int categoryId) throws SQLException{
		//エラーを説明する文字列（SQLException）をスルーする

		List<Item> itemsList = new ArrayList<>();
		//インスタンス化

		String sql =
				"SELECT "
				+ "item_Id,"
				+ "name,"
				+ "manufacturer,"
				+ "category_Id,"
				+ "color,"
				+ "price,"
				+ "stock,"
				+ "recommended "
				+ "FROM "
				+ "public.items "
				+ "WHERE "
				+ "name LIKE '%?%' "
				+ "AND category_Id = ?";

				/* 商品Id
				 * 商品名
				 * 製造会社
				 * カテゴリId
				 * 色
				 * 価格
				 * 在庫数
				 * オススメ
				 * public.itemsテーブルの
				 * カテゴリによる*/

		Connection con = connect();
		//CommonDAOから接続するメソッドを継承

		try(PreparedStatement ps = con.prepareStatement(sql)){
			//SQLを実行するための準備

			ps.setString(1, keyword);
			ps.setInt(2, categoryId);
			//プレースホルダーに値をセット


			ResultSet rs = ps.executeQuery();
							//SQLを実行


			while(rs.next() == true) {
				//データベースを繰り返し処理で検索する

				Item item = new Item();
				//Itemをインスタンス化

				item.setItem_Id(rs.getInt("item_id"));
				item.setName(rs.getString("name"));
				item.setManufacturer(rs.getString("manufacturer"));
				item.setCategory_Id(rs.getInt("category_Id"));
				item.setColor(rs.getString("color"));
				item.setStock(rs.getInt("stock"));
				item.setRecommended(rs.getBoolean("recommended"));
				//各リストのResultSetをgetterで取得して、itemのリストにsetterでセット

				switch(rs.getInt("category_Id")) {
				//カテゴリ別に結果を変える処理

					case 1:
						//帽子の時
						itemsList.add(item);
                                                break;
					case 2:
						//鞄の時
						itemsList.add(item);
                                                break;
					default:
						//全て（未選択）の時
						itemsList.add(item);
						/*各アイテムをリストにまとめて、
						 * addでAllayListに追加*/
				}
			}
		}catch(SQLException e) {
			e.printStackTrace();
			/*tryで実行できなかった場合、
			 * エラー説明文をキャッチする*/
		}

		return itemsList;
		//try処理の結果をitemsListインスタンスに戻して格納

	}
}

import java.util.List;

import javax.servlet.http.HttpServletRequest;

import jp.nib.ict21.ecsite.dao.ItemDAO;
import jp.nib.ict21.ecsite.data.Item;

public class SearchService {

	public static List<Item> findByCondition(HttpServletRequest request) throws SQLException{

		ItemDAO dao = new ItemDAO();
		List<Item> itemList = dao.findByCondition();

		return itemList;

	}
}

Answer 1

引数の考え方に誤解がありました。

下記のように修正することでエラー解除できました。

public class SearchService {

	public static List<Item> findByCondition(HttpServletRequest request) throws SQLException{

		String a = request.getParameter("keyword");
		String b = request.getParameter("category");
		/*request.getParameter("");で画面入力されたものを持ってくる。
		 * 引数はHTMLのnameに指定した値*/

		int c = Integer.parseInt(b);
		//getしたcategoryIdはint型なので、Integerで型変換
		//int型のcは = Int型のbですよ

		ItemDAO dao = new ItemDAO();
		List<Item> itemList = dao.findByCondition(a, c);
		//findByCondition(a, value);をItemDAOの引数に渡す

		return itemList;

	}
}

Answer 2

エラー内容提示されてないのでどんなエラー出てるか知りませんが

定義

findByCondition(String keyword, int categoryId) 

呼び出し

List<Item> itemList = dao.findByCondition();

定義を正とするなら呼び出し側に引数がないですし、
呼び出し側を正とするなら定義に余計な引数があります。

「どういう機能を持ったメソッドなのか」確実にしてください。

Comments

[music]

早急なご回答ありがとうございます。
説明が足りず申し訳ありませんでした。。

エラー文は
「型 ItemDAO のメソッド findByCondition(String, int) は引数 () に適用できません」
となっております。

商品検索画面で入力されたキーワードとカテゴリ指定を取得し、結果を返すのですが、
dao.findByCondition();
のdao.の後は
findByCondition(String keyword, int categoryId) ;
にすればよいのでしょうか。

javaを初めて１か月に満たず、基本的な事がわかっていないかもしれません。
お手数お掛け致しますが、よろしくお願いします。。。

[m.ts10806]

想定通りのエラー内容ですが、いずれにしても「メソッド」「引数」についてきちんとおさえてください。
（DAOやDTO、Beansに触れるのはまだ早いです）
それなりの入門書であれば１か月しなくても触れられる、どちらかというと中盤より前の内容です。

[music]

私の学習不足が原因なのは重々承知しております。
行き詰ってしまったのでこちらに質問させて頂いた次第です。

貴重なお時間取らせてしまい、申し訳ありませんでした。

[m.ts10806]

こちらから首突っ込んでるので謝られるのは違う気がするのですけど。。。
そもそも何を参考に書きましたか？
エラーの意味は理解できているのか、いないのかでもこちらのアドバイスの方向性も変わります。
「定義」と「参照」を理解できているか、いないのかでも違います。
何も理解できてないなら入門書をしっかりやったほうが良いです（これ理解できてないと何も組めないから。飛ばしていい場所ではない）

[m.ts10806]

ちなみに現状では「どう書けば解決か」そのままの回答はできません。
keyword、categoryIdにあたるものがどこからどう送られてくるのか今提示されているコードにはないからです。
ひとまず、メソッドが求める引数と呼び出し側の引数をあわせてください（型の話）

[music]

定義と参照は理解しているつもりですが、忘れているかもしれませんので復習致します。
ご意見ありがとうございました！

[dodox86]

質問者 @musicさん

m.ts10806さんが指摘されていることは、まず、

> 商品検索画面で入力されたキーワードとカテゴリ指定を取得し、

この、入力であるはずの「キーワード」と「カテゴリ」を取り出しているところが見当たらない、ということです。
たぶん、SearchServiceのfindByConditionメソッドがHttpServletRequest requestを引数としているので、そのリクエストに商品検索画面で入力されたフォームのパラメータが入っているのですよね？

もし、その辺りがご自身で明らかでないなら、コードは完成しません。

[music]

m.ts10806様、dodox86様、
お二方ともご回答ありがとうございました。

ご意見を参考に復習したところ、引数の考え方に少し誤解があったようです。

public class SearchService {
public static List<Item> findByCondition(HttpServletRequest request) throws SQLException{

String a = request.getParameter("keyword");
String b = request.getParameter("category");
/*request.getParameter("");で画面入力されたものを持ってくる。
* 引数はHTMLのnameに指定した値*/

int c = Integer.parseInt(b);
//getしたcategoryIdはint型なので、Integerで型変換
//int型のcは = Int型のbですよ

ItemDAO dao = new ItemDAO();
List<Item> itemList = dao.findByCondition(a, c);
//findByCondition(a, c);をItemDAOの引数に渡す

return itemList;

}
}

このように修正したところ、エラーを解消できました。

1つ間違えると全体に影響してしまうので、改めて難しさと楽しさを実感できました。
この度はお二方とも、本当にありがとうございました。

[m.ts10806]

細かいこと言うと、直ParseIntで良いです。1つメモリが無駄になりますし。
int b =Integer.parseInt( request.getParameter("category") );
また、getParameter()の値が絶対数値にできるとは限らないのでエラーハンドリングは必要です。

[music]

なるほど！参考になります！！

ありがとうございます！