如何改进我的 DAO？ - Java EE

Question

我想访问我的申请人数据库，这就是我为其创建 DAO 类的原因。

我认为我有很多代码味道，因为我不断重复一些代码。那么我该怎么做才能使我的代码更简单以减少代码异味呢？我违反了什么规则？我该如何改进我的代码？谢谢。

我的代码如下：

public class ApplicantDAO {

private static ApplicantDAO me = null;

private ApplicantDAO(){};

public static synchronized ApplicantDAO getInstance() {
    if(me == null) {
        me = new ApplicantDAO();
    }
    return me;
}

public Applicant getApplicant(int applicantNumber) throws SQLException {
    Applicant applicant = null;

    Connection conn = null; 
    Statement statement= null;
    String query = null;
    ResultSet rs = null;

    try {
        conn = ConnectionManager.getConnection();
        statement = conn.createStatement();
        query = "SELECT * FROM applicant WHERE applicant_no = '" + applicantNumber +"'";    //check applicant_number
        rs = statement.executeQuery(query);

        while(rs.next()){
            applicant = new Applicant();

            applicant.setApplicantNumber(rs.getInt("applicant_no"));
            applicant.setApplicationDate(rs.getString("applicant_date")); 
            applicant.setfName(rs.getString("first_name"));
            applicant.setlName(rs.getString("last_name"));
            applicant.setmName(rs.getString("middle_name"));
            applicant.setAge(rs.getInt("age"));
            applicant.setGender(rs.getString("gender"));
            applicant.setEmail(rs.getString("email_address"));
            applicant.setContactNumber(rs.getString("contact_no"));
            applicant.setCity(rs.getString("city"));
            applicant.setSchool(rs.getString("school"));
            applicant.setCourse(rs.getString("course"));
            applicant.setYearGraduated(rs.getInt("year_graduated"));
            applicant.setYearWorkExp(rs.getInt("year_work_exp"));
            applicant.setSourceChannel(rs.getString("source_channel"));
            applicant.setStatus_id(rs.getInt("status_id"));

        }

    } catch (SQLException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }finally {
        if (rs != null) try { rs.close(); } catch (SQLException logOrIgnore) {}
        if (statement != null) try { statement.close(); } catch (SQLException logOrIgnore) {}
        if (conn!= null) try { conn.close(); } catch (SQLException logOrIgnore) {}
    }


    return applicant;
}

public ArrayList<Applicant> getApplicants() throws SQLException{
    ArrayList<Applicant> applicantList = null;
    Applicant applicant = null;

    Connection conn = null; 
    Statement statement= null;
    String query = null;
    ResultSet rs = null;

    try {
        conn = ConnectionManager.getConnection();
        statement = conn.createStatement();
        query = "select * from applicant";
        rs = statement.executeQuery(query);

        while(rs.next()){
            if(applicantList == null){
                applicantList = new ArrayList<Applicant>();
            }
            applicant = new Applicant();

            applicant.setApplicantNumber(rs.getInt("applicant_no"));
            applicant.setApplicationDate(rs.getString("applicant_date")); 
            applicant.setfName(rs.getString("first_name"));
            applicant.setlName(rs.getString("last_name"));
            applicant.setmName(rs.getString("middle_name"));
            applicant.setAge(rs.getInt("age"));
            applicant.setGender(rs.getString("gender"));
            applicant.setEmail(rs.getString("email_address"));
            applicant.setContactNumber(rs.getString("contact_no"));
            applicant.setCity(rs.getString("city"));
            applicant.setSchool(rs.getString("school"));
            applicant.setCourse(rs.getString("course"));
            applicant.setYearGraduated(rs.getInt("year_graduated"));
            applicant.setYearWorkExp(rs.getInt("year_work_exp"));
            applicant.setSourceChannel(rs.getString("source_channel"));
            applicant.setStatus_id(rs.getInt("status_id"));

            applicantList.add(applicant);
        }

    } catch (SQLException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }finally{
         if (rs != null) try { rs.close(); } catch (SQLException logOrIgnore) {}
         if (statement != null) try { statement.close(); } catch (SQLException logOrIgnore) {}
         if (conn!= null) try { conn.close(); } catch (SQLException logOrIgnore) {}
    }
    return applicantList;
}

Answer 1

我看到的巨大而明显的问题：

1. DAO 是一个单例。为什么？
2. 您没有使用 EntityManager。还有，为什么？

如果您实际上使用的是 Java EE（正如问题所标记的那样）而不是 J2EE（这是围绕 Java 1.4 构建的一个糟糕的规范），那么 DAO 模式是完全没有必要的。 EntityManager 是新的 DAO。

查看Java EE 6 教程 - 持久性<的前几节/a>.

---

我们需要使用 J2ee..:(

好吧，所以您仍然需要修复单例实现。没有理由只创建该对象的一个​​实例，因为它不存储任何内部状态。有一个简单的方法修复：

• 完全删除 private static ApplicantDAO me = null;

• 将 getInstance() 实现更改为

  public static ApplicantDAO getInstance() {
      返回新的ApplicantDAO()；
  }
  

其他气味：

• 您几乎总是希望声明 List 而不是 ArrayList，因此更改声明，例如

  public ArrayList<申请人>; getApplicants() 抛出 SQLException
  // 到
  公开列表<申请人> getApplicants() 抛出 SQLException
  
  // 和
  
  ArrayList<申请人>申请人列表=空；
  // 到
  列表<申请人>申请人列表=空；
  

Answer 2

为了让您了解更现代的 DAO 是什么样子：

@Stateless
public class JPAApplicantDAO implements ApplicantDAO {

    @PersistenceContext(unitName = "myPU")
    private EntityManager entityManager;

    @Override
    public Applicant getByID(Long applicantID) {
        return entityManager.find(Applicant.class, applicantID);
    }

    @Override
    public void update(Applicant applicant) {
        applicant.setLastModifiedDate(new Date());
        entityManager.merge(applicant);
    }

    @Override
    public void delete(Applicant applicant) {
        Applicant deletedApplicant = applicant;
        if (!entityManager.contains(applicant)) {
            deletedApplicant = entityManager.merge(applicant);
        }

        entityManager.remove(deletedApplicant);
    }

    @Override
    public List<Applicant> getBySomethingID(Long somethingID) {
        return entityManager.createNamedQuery("Applicant.getBySomethingID", Applicant.class)
                            .setParameter("somethingID", somethingID)
                            .getResultList();
    }
}

现在有人建议放弃整个 DAO 概念并直接在任何地方使用实体管理器。我并不完全同意这一点。

此 DAO 示例展示了 4 种不同的方法。第一种方法是通过实体的主 ID 进行简单获取。正是这种方式，让人怀疑是否还需要DAO抽象。但请继续阅读。

第二种方法示出了更新方法。在这种情况下，应用程序可能想要对实体执行一些额外的操作，例如设置上次修改日期。 DAO 是执行此操作的一个非常自然的地方。是的，它也可以在数据库中完成，但是 DAO 仍然很方便，因为您可能必须读回实体才能了解所设置的日期。

第三种方法是删除方法。由于 JPA 规范中的一个特殊问题，您只能删除处于附加状态的实体。这意味着需要一些额外的逻辑来检查它是否已附加（包含在持久性上下文中），如果没有附加它（合并它）。

第四种方法显示通过 (JPQL) 查询进行数据检索。查询名称和参数名称都不是类型安全的。 DAO 方便地将其隐藏在类型安全的 Java 方法后面。是的，您可以将这些名称提取为常量，但仍不会强制执行此特定参数和此特定查询之间的关联。

一般来说，DAO 允许进行一定量的重构。在某些时候，我可能想用条件查询替换 JPQL 查询。在所有调用站点更改此设置可能会出现问题。还有一种情况是，实体管理器作为通用 DAO 实在是太强大了。我不想将这些发送到所有客户端站点（对于远程客户端，这甚至是不可能的或非常糟糕的做法）。

最后，从本身不具有事务性的客户端代码中使用实体管理器，意味着该客户端必须担心事务。这给代码增加了很多冗长的内容。使用 DAO，客户端代码变得更加简单：

@Named
@RequestScoped
public class SomeBean {

    @EJB
    private ApplicantDAO applicantDAO;

    public void someMethod() {
        applicantDAO.delete(applicant);
    }
}

Answer 3

您的示例看起来像是一张完美的“之前”图片，在使用 Hibernate 将对象映射到关系数据库并使用 Spring 依赖注入功能支持连接和事务管理之后，显示了漂亮的“之后”图片。

Answer 4

您可以使用Java Persistence API，并且大多数这些无聊的代码都是不必要的。

Answer 5

有史以来最糟糕的代码味道：你再次重新发明轮子。为什么不使用一些开放库，例如 Spring JDBC ？ （注意：您也可以阅读代码来从中学习！）

如果您在这里练习，和/或不允许使用外部代码，那么这里有一些增强您的代码的提示：

• 所有未包含的代码关于定义语句（及其参数）或处理结果集，可以分解为可重用方法以减少代码重复。这可能会导致您使用回调式参数或用于语句创建的可变参数，以及用于申请人重构的适配器式组件（尝试查看 Spring 的 JdbcTemplate 和 RowMapper 类的代码）
• 你的签名太精确了：当你可以返回一个List时，为什么要返回一个ArrayList？
• 为什么你的 getInstance() 方法是同步的？您可以通过在类初始化时初始化 me 来避免这种情况：
  

  private static ApplicantDAO me = new ApplicantDAO();
  公共静态 ApplicantDAO getInstance() { 返回我; }
  

• 您的日志策略似乎未定义：我认为这是故意的，但请记住，这在现实应用程序中非常重要。