java - DataDAO类有一百个静态方法，如何重构

Question

我有一个 DataDAO 类，我的 Servlet 使用它对多种对象类型进行 CRUD 操作。当我开始的时候，我在这门课上只做了一些事情，所以看起来还不错。但现在项目变得越来越大，我添加的每个功能都必须向此类添加一个新方法，所以我现在有大量的静态方法。看来我应该如何重构它，但不确定如何重构。有我可以使用的设计模式吗？或者有人可以解释为什么我应该或不应该担心它？我这样做只是为了学习目的，所以请不要告诉我使用一些简单的框架，我想尽可能精细地使用 java。这是一个典型的例子:

public static ArrayList<Card> getCardsForUser(UserAccount user) {

    //TODO: get the username and password then get all flashcards linked to that user and return them in a list
    ArrayList<Card> cardsForUser = new ArrayList<>();

    try(Connection conn = DriverManager.getConnection(DBURL, un, pw)) {
        PreparedStatement pstm = conn.prepareStatement("Select * From flashcard where fk_user_id = ?");
        pstm.setString(1,user.getUserID());
        ResultSet usersCards = pstm.executeQuery();

        while(usersCards.next()){
            String cat = usersCards.getString("category");
            if(cat == null) {
                cat = "null";
            }
            Card card_new = new Card(usersCards.getString("card"),usersCards.getString("answer"),usersCards.getInt("cardid"),cat,usersCards.getInt("times_right"),usersCards.getInt("times_wrong"));
            cardsForUser.add(card_new);
        }
        System.out.println("Card For User size: "+cardsForUser.size());

        return cardsForUser;


    } catch(SQLException e) {
        //TODO: what happens now ?
        e.printStackTrace();
        return null;
    }

}

我的应用程序基本上是一种创建“抽认卡”来学习的方法。

我尝试创建一个接口(interface)“DataDAO”，然后为每个实现该接口(interface)的不同对象创建一个子类。但有些操作与界面并不完全一致，而且看起来有很多不必要的工作。为什么这是一个好方法或不是一个好方法？

Answer 1

你的类名DataDAO本身就暗示设计中有问题。最好对不同的对象类型使用单独的 DAO 类。例如。 CardDAO、UserAccountDAO 等

为什么将此方法设为静态？我看不出有什么理由。我认为方法签名可能是这样的:

public List<Card> getCardsForUser(String userId)

请注意，我返回的是 List 而不是 ArrayList。

既然你是一个学习者，我也提一下以下几点:

避免在同一范围内使用多个具有相似名称的变量。您的代码中有 usersCards 和 cardsForUser 。这可能会令人困惑。对于 ResultSet，您可以使用 resultSet 或 rs 之类的名称。

遵循 Java 命名约定。因此，最好使用 cardNew 而不是 card_new。