java spring mvc使用hibernate做更新

Question

两个月前，我开始使用 Java Spring MVC 为 iOS 应用程序开发 API。 我会用一个例子来解释我的问题，以便澄清。

假设我想更新用户名。 MySQL 用户表有列：id、user_id、email、display_name。 我的做法是：

1. 定义用户：

用户.java：

package bean;
public class User {

  String displayName;
  String email;
  String userId;

  getters/setters... 
}

2.定义一个DAO：

UserDao.java：

package dao;
import bean.StandardResponse;

public interface UserDao {
    public StandardResponse updateUserName(String user_id,String userName);
}

UserDaoImpl.java：

package implement;
import bean.User;
import common.DatabaseConnect;

public UserDaoImpl implements UserDao {
    private DatabaseConnect dbConnect; 

    public UserDaoImpl(DatabaseConnect dbConnect) {
      this.dbConnect = dbConnect;
    }

    public StandardResponse updateUserName(userId,userName) {
      if ((userId == null || userId.isEmpty()) ||(userName == null || userName.isEmpty())) return new StandardResponse("Error","Parameters not set!");
      String sql = null;
      Statement smt = dbConnect.createDataBaseConnectResourse();
      StandardResponse result = new StandardResponse("Error","Fail to update the record!");
      sql = "update User set user_name="+userName+" where user_id='"+userId+"'";
      int updateResult = 0;
      try {
        updateResult = smt.executeUpdate(sql);
      } catch (SQLException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
      } finally {
        dbConnect.closeDatabaseConnectResource();
      }
      if (updateResult == 1) {
        result = new StandardResponse("Success","The record has been updated!");
      }

      return result;        
  }
}

3.控制器

import org.springframework.stereotype.Controller;
import java.util.List;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.bind.annotation.RestController;

import bean.StandardResponse;
import bean.User;
import common.DatabaseConnect;
import common.SpringApplicationContext;
import dao.UserDao;
import implement.UserDAOImpl;

@Controller
@RestController
@RequestMapping("user")
public class UserController {

  DatabaseConnect dbConnect = SpringApplicationContext.getApplicationContext().getBean("databaseConnect", DatabaseConnect.class);
  UserDao uiObject = new UserDaoImpl(dbConnect);

  @RequestMapping(value = "/updateUserName", method = RequestMethod.POST)
  public StandardResponse updateUserName(HttpServletRequest request, HttpServletResponse reponses, Model model) {
    StandardResponse srObject = uiObject.updateUserName(request.getparameter("userId"),request.getParameter("userName"));
    return srObject;
  }
}

我只是把关键的课程放在这里。我相信你能明白我在这里做什么。因此，如果有人访问提供 userId 和 userName 的 URL：****/user/updateUserName，他可以更新该记录的用户名。它是功能性的。

我用同样的方法完成了整个项目。这是工作。然后，我请一位经验丰富的程序员查看我的代码，因为我根据自己的理解弄清楚了所有这些。我想知道他们在工业界的表现如何。他给了我一些有价值的cmets。

1. 整个结构错误。我的道中不应该有逻辑。我至少应该有 dao、service 和 action 层。 dao 只处理数据库访问，service 处理所有逻辑，action 处理通信并决定调用哪个服务。
2. 在代码中手写 SQL 是一种非常糟糕的方法。我应该使用 Hibernate。
3. 我应该使用控制反向和依赖注入。（我在这篇文章中不涉及这个。）

所以我开始更新我的代码以使用 Hibernate。

1. 定义用户：

用户.java：

package model;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.Table;

@Entity
@Table(name="User")
public class User {

  @Id
  @Column(name="id")
  @GeneratedValue(strategy=GenerationType.IDENTITY)
  private int id;

  @Column(name = "user_id")
  private String userId;

  @Column(name = "user_name")
  private String displayName;

  @Column(name = "email")
  private String emai;

  all getters/setters...
}

1. 道：

UserDao.java：

package dao;
import model.User;
import java.util.List;
public interface UserDAO {

  public void updateUser(User p);
  other methods....

}

UserDaoImpl.java

package dao;
import model.User;
import java.util.List;

import org.hibernate.Session;
import org.hibernate.SessionFactory;
import org.springframework.stereotype.Repository;

import model.User;

@Repository
public class PersonDAOImpl implements PersonDAO {

    private SessionFactory sessionFactory;

    public void setSessionFactory(SessionFactory sf){
        this.sessionFactory = sf;
    }

    @Override
    public void updatePerson(Person p) {
        Session session = this.sessionFactory.getCurrentSession();
        session.update(p);
    }

    implementations of other methods....

}

1. 服务：

UserService.java：

package service;

import java.util.List;

import model.User;

public interface UserService {

    public void updateUser(User p);
    other methods....

}

UserServiceImpl.java：

package service;

import java.util.List;

import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import dao.UserDAO;
import model.User;

@Service
public class UserServiceImpl implements UserService {

    private UserDAO userDAO;

    public void setUserDAO(UserDAO userDAO) {
        this.userDAO = userDAO;
    }

    @Override
    @Transactional
    public void updateUser(User p) {
       this.userDAO.updateUser(p);
    }

    implementation of other methods...

}

现在，我只需要编写一个类来处理请求并调用此 updateUser 服务。整个结构看起来比我的好。但是，该请求不会给我整个用户对象。我能得到的只是 user_id 和 user_name。我也不允许将逻辑放在道中。所以我必须先查询表以获取整个用户对象，然后更新用户名。与我以前的做法相比，一个 SQL 处理更新。现在使用 Hibernate，我需要 1 个查询和 1 个更新。

我在 StackOverflow 上找到了一个解决方案，我可以使用 HQL 一步完成。但我认为使用 Hibernate 的目的是让我们从编写 SQL 中解放出来。如果我需要写HQL，为什么不继续使用写SQL的方法。

那么有没有办法在不先在这个结构中查询表的情况下使用 Hibernate 进行更新？或者这是为了更好的结构而做出的权衡？

很抱歉，我为这个问题使用了一个很长的示例。但我想不出其他方法来清楚地解释整个故事。

Answer 1

那么有没有办法在不先在这个结构中查询表的情况下使用 Hibernate 进行更新？或者这是为了更好的结构而做出的权衡？ 没有折衷，您可以使用 HQL（Hibernate 查询语言）进行更新，就像在 SQL 中一样。

可以看下面的代码：

UserDAOImpl 类：

    @Repository
    //you are calling this in ur code as PersonDAOImpl..change it to UserDAOImpl
    public class UserDAOImpl implements UserDAO {

        private SessionFactory sessionFactory;

        public void setSessionFactory(SessionFactory sf){
            this.sessionFactory = sf;
        }

        @Override
        public int updateUser(String userId, String userName) {
            Session session = this.sessionFactory.getCurrentSession();
         Query query = session.createQuery("update User set userName =:userName where userId = :userName ");
         query.setString("userName", userName);  
         query.setString(userName, userName);  
         int result = query.executeUpdate();
         return result;
        }
    }

UserServiceImpl 类：

@Service
public class UserServiceImpl implements UserService {

    private UserDAO userDAO;

    public void setUserDAO(UserDAO userDAO) {
        this.userDAO = userDAO;
    }

    @Override
    @Transactional
    public void updateUserName(String userId, String userName) {

        if(userId !=null && userName != null) {
                int result = this.userDAO.updateUser(userId, userName);

           if(result==0) //userid not available {
                //if userid is NOT available, what to do? check your requirement and handle properly
           }

           } 
       } else {
            //throw exception
       }
    }
    implementation of other methods...
}

控制器类：

@Controller
@RestController
@RequestMapping("user")
public class UserController {

  @Autowired
  private UserService userService;

  @RequestMapping(value = "/updateUserName", method = RequestMethod.POST)
  public StandardResponse updateUserName(HttpServletRequest request, HttpServletResponse reponses, Model model) {
    StandardResponse srObject = userService.updateUserName(request.getparameter("userId"),request.getParameter("userName"));
    return srObject;
  }
}