Skip to main content
本文に 1654 文字追加
ソース リンク
user40629
user40629

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

SearchConditionクラスは、部分一致や前方一致などの検索条件の親クラスです。
それを継承した検索条件のクラスでは部分一致であればgetWhereで"name LIKE ?"を返し、getValueで"%hoge%"などを返します。

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

SearchConditionクラスは、部分一致や前方一致などの検索条件の親クラスです。
それを継承した検索条件のクラスでは部分一致であればgetWhereで"name LIKE ?"を返し、getValueで"%hoge%"などを返します。

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

本文に 1654 文字追加
ソース リンク
user40629
user40629
public class UserSearcher {
    private Connection conn;

    private List<SearchCondition<?>> conditions;

    public UserSearcher(Connection conn, List<SearchCondition<?>> conditions) {
        this.conn = conn;
        this.conditions = conditions;
    }

    public List<User> execute() throws Exception {
        if (isEmpty(this.conditions)) return null;

        List<String> wheres = new ArrayList<>();
        for (SearchCondition<?> condition: conditions) {
            if (condition.isEmpty()) continue;

            String where = condition.getWhere();
            wheres.add(where);
        }

        String where = String.join(" AND ", wheres.toArray(new String[0]));
        String sql = "SELECT * FROM users WHERE " + where;

        PreparedStatement pStmt = conn.prepareStatement(sql);
        pStmt = bindParams(pStmt);

        ResultSet resultSet = pStmt.executeQuery();

        List<User> users= new ArrayList<>();
        while(resultSet.next()) {
            User user = new User();
            // userに値をset

            users.add(user);
        }

        return users;
    }

    private PreparedStatement bindParams(PreparedStatement pStmt) throws Exception {
        int i = 1;
        for (SearchCondition<?> condition: conditions) {
            Object value = condition.getValue();
            if (value instanceof String) {
                pStmt.setString(i, (String) value);
            } else if (value instanceof Integer) {
                pStmt.setInt(i, (Integer) value);
            }

            i++;
        }

        return pStmt;
    }
}

public class UserSearcher {
    private Connection conn;

    private List<SearchCondition<?>> conditions;

    public UserSearcher(Connection conn, List<SearchCondition<?>> conditions) {
        this.conn = conn;
        this.conditions = conditions;
    }

    public List<User> execute() throws Exception {
        if (isEmpty(this.conditions)) return null;

        List<String> wheres = new ArrayList<>();
        for (SearchCondition<?> condition: conditions) {
            if (condition.isEmpty()) continue;

            String where = condition.getWhere();
            wheres.add(where);
        }

        String where = String.join(" AND ", wheres.toArray(new String[0]));
        String sql = "SELECT * FROM users WHERE " + where;

        PreparedStatement pStmt = conn.prepareStatement(sql);
        pStmt = bindParams(pStmt);

        ResultSet resultSet = pStmt.executeQuery();

        List<User> users= new ArrayList<>();
        while(resultSet.next()) {
            User user = new User();
            // userに値をset

            users.add(user);
        }

        return users;
    }

    private PreparedStatement bindParams(PreparedStatement pStmt) throws Exception {
        int i = 1;
        for (SearchCondition<?> condition: conditions) {
            Object value = condition.getValue();
            if (value instanceof String) {
                pStmt.setString(i, (String) value);
            } else if (value instanceof Integer) {
                pStmt.setInt(i, (Integer) value);
            }

            i++;
        }

        return pStmt;
    }
}

本文に 3 文字追加
ソース リンク
cubick
  • 2.1万
  • 5
  • 27
  • 64

Javaプログラミングにおいて、検索とそれに関わる機能はどのクラスが持つべき責務でしょうか?

現在ServletとJspでユーザー管理画面のようなものを作成しており
・ユーザー作成以下の機能があります。

・ユーザー作成
・ユーザー編集機能
・ユーザー一覧表示機能
・ユーザー検索機能
・ユーザー削除機能
の機能があります。

Model
|
----dto
| |
| ---User.java
-----dao
| |
| ---UserDao.java
|
--- UserSearcher.java

Model
  |
  ----dto
  |    |
  |    ---User.java
  -----dao
  |    |
  |    ---UserDao.java
  |
  --- UserSearcher.java

そしてモデルの中でDTOとしてUser.java、DAOとしてUserDao.javaがあります。
UserDAOにCRUD処理が記述されており、それをServletクラス(コントローラー)から呼び出しております。
検索機能は独立してモデルの中にUserSearcherクラスとして存在しており、UserSearcherのsearchメソッドで、DBにアクセスして結果を取得しております。

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

こういう役割をどのように分担するべきなのか、教えていただきたいです。

Javaプログラミングにおいて、検索とそれに関わる機能はどのクラスが持つべき責務でしょうか?

現在ServletとJspでユーザー管理画面のようなものを作成しており
・ユーザー作成機能
・ユーザー編集機能
・ユーザー一覧表示機能
・ユーザー検索機能
・ユーザー削除機能
の機能があります。

Model
|
----dto
| |
| ---User.java
-----dao
| |
| ---UserDao.java
|
--- UserSearcher.java

そしてモデルの中でDTOとしてUser.java、DAOとしてUserDao.javaがあります。
UserDAOにCRUD処理が記述されており、それをServletクラス(コントローラー)から呼び出しております。
検索機能は独立してモデルの中にUserSearcherクラスとして存在しており、UserSearcherのsearchメソッドで、DBにアクセスして結果を取得しております。

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

こういう役割をどのように分担するべきなのか、教えていただきたいです。

Javaプログラミングにおいて、検索とそれに関わる機能はどのクラスが持つべき責務でしょうか?

現在ServletとJspでユーザー管理画面のようなものを作成しており以下の機能があります。

・ユーザー作成
・ユーザー編集
・ユーザー一覧表示
・ユーザー検索
・ユーザー削除

Model
  |
  ----dto
  |    |
  |    ---User.java
  -----dao
  |    |
  |    ---UserDao.java
  |
  --- UserSearcher.java

そしてモデルの中でDTOとしてUser.java、DAOとしてUserDao.javaがあります。
UserDAOにCRUD処理が記述されており、それをServletクラス(コントローラー)から呼び出しております。
検索機能は独立してモデルの中にUserSearcherクラスとして存在しており、UserSearcherのsearchメソッドで、DBにアクセスして結果を取得しております。

そしてここからが本題なのですが、DAOが存在するアプリケーションにおいて、別クラスでDBにアクセスする設計にするべきではないでしょうか?私の感覚ではDAOが存在しているのに、DAO以外でDBにアクセスすることに違和感を感じております。

ただ今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい(今はUserSearcherですがいずれはSearcherになる予定)のです。DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

となるとSearcherクラスをクエリを組み立てるクラス(SearchQueryBuilder)として作り直し、クエリとパラメータを渡せば実行できるメソッドをDAOに作るべきなのか?などと考えておりますが、なんかパッとしません。

こういう役割をどのように分担するべきなのか、教えていただきたいです。

ソース リンク
user40629
user40629
読み込んでいます