0

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

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

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

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

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;
    }
}

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

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

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

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

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

3
  • 1
    商品とユーザの検索ロジックを共通化する、というのは危険な匂いがします。この記事が参考になれば: iwatam-server.sakura.ne.jp/software/devintro/reuse
    – mok2pok
    Commented 2020年6月13日 8:02
  • ご回答いただきありがとうございます。とても勉強になります。それでは一度再利用のことは考えないものとして、DAOとは別でDBにアクセスするUserSearcherの設計に関してはどう思われますか?
    – user40629
    Commented 2020年6月13日 10:45
  • @出羽和之 さんの回答に同意です。利用者側のコードからは、ユーザのDAO1つだけしか見えない形がいいです。ユーザのDAOが、複雑な検索処理をUserSearcherに委譲するように組むのはありだと思います。
    – mok2pok
    Commented 2020年6月15日 5:23

1 件の回答 1

1

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

これはおっしゃるとおりだと思います。dao.UserDaoというクラスがあるにも関わらず、実は別のパッケージの別のクラスを使ってDBアクセスする必要がある可能性がある、というのは非常に煩わしいです。

今後商品テーブルのようなものを作る予定であり、そちらにも検索機能をつけたいと思っているため、検索に関するロジックに関しては共通で使えるクラスとして保持したい

ここが理解できていないです。(一般的に考えると)ユーザーと商品の検索であまり共通化できる要素があるようには思えません。

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

SearchQueryBuilderと呼称されているようなものを作る、という方向性については私もそうであるかな、と思います。
ただ、今回の例に挙げられているような、単純なSQLの組み立てに関する処理を想定しているのであれば(ユーザと商品の検索で共通化できる、というのはつまりそういうことでしょうか)、そのSearchQueryBuilderは利用者(※質問文中の"Servletクラス(コントローラー)"のこと)からは隠蔽し、DAOクラスが必要に応じて利用する、という形の方が良いのではないかと思います。
(利用者が意識するのはDAO(のメソッド呼び分け)だけでSearchQueryBuilderを直接使うことはない、ということになります。)

DAOに直接実装するわけではないので

DAOに検索ロジックをもたせてしまうと、同じロジックを今後のItemDAOにも実装することになってしまいます。

というようなことも無いかと思います。

この質問に回答するには、ログインする必要があります。