多次将相同的值添加到db

时间:2016-09-07 12:02:09

标签: java mysql sql spring hibernate

// LoginDAO,用于将UI值保存到DB中的逻辑...添加新值并替换旧值

System.out.println("In Vendor registration i.e LoginDAO class::::::::::"+vendorRegistration.getVid()+""+""+vendorRegistration.getFirstName());
                Session session = getSession();                               
                session.beginTransaction();                                                //begin transaction for the session.
                Query query = session.createQuery("UPDATE VendorRegistration set firstName =:firstname,lastName =:lastname,Email =:email,password =:password,Vid =:vid "); 

                query.setParameter("firstname",vendorRegistration.getFirstName());
                query.setParameter("lastname",vendorRegistration.getLastName());
                query.setParameter("email",vendorRegistration.getEmail());
                query.setParameter("password",vendorRegistration.getPassword());
                query.setParameter("vid",vendorRegistration.getVid());              

                int user = query.executeUpdate();

                 session.save(vendorRegistration);
                 System.out.println("user values are ::::::::::::::::::"+user);
                session.getTransaction().commit();                            //here transaction complete with commit the data in db. 
                session.close();



// Controller class

@RequestMapping(value = {"/signup"}, method = RequestMethod.POST)
    public String saveRegAction(@ModelAttribute("signup") @Validated VendorRegistration vendorRegistration, Model model,HttpSession session,BindingResult bindingResult) throws IOException
    {
        System.out.println("the object is:" + model);
        if(bindingResult.hasErrors())
        {
            logger.info("user details===========" +vendorRegistration.getFirstName()+""+vendorRegistration.getLastName()+""+vendorRegistration.getVid()+""+vendorRegistration.getEmail()+""+vendorRegistration.getPassword());
            logger.info("Returning home.jsp page"); 

            model.addAttribute("vendor", new VendorRegistration()); 
            return "signup";
        }
        loginService.saveNewUser(vendorRegistration);
        session.setAttribute("vendorRegistration", vendorRegistration);

        logger.info("in registration page........... save register action");
        return "vendorLogin";

//值保存多次保留旧值并将新值保存到DB

![这是DB的问题] [1]] [1]

// pojo class

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


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


    @Column(name = "VID")
    private String vid;

    @NotBlank
    @Column(name = "FIRSTNAME")
    private String firstName;

    @Column(name = "LASTNAME")
    private String lastName;

    @Column(name = "EMAIL")
    @Email(message="Please Enter Valid Email ID")
    private String email;

    @Column(name ="PASSWORD")
    private String password;
//getters and setters

2 个答案:

答案 0 :(得分:2)

您的代码中存在一些错误/缺陷。首先是你的控制器,它有一个错误的方法签名。

首先修复方法签名,BindingResult 必须直接遵循它适用的方法参数,在您的情况下是@ModelAttribute带注释的元素。

@RequestMapping(value = {"/signup"}, method = RequestMethod.POST)
public String saveRegAction(@ModelAttribute("signup") @Validated VendorRegistration vendorRegistration, BindingResult bindingResult, Model model,HttpSession session,) throws IOException { ... }

您的数据库问题是由于您的service / dao方法的实现。您首先执行更新查询,最后插入它。所以会发生什么,首先更新所有内容,然后插入新记录。你的方法应该只有session.save而已。

Session session = getSession();                               
session.beginTransaction();                                                 

session.save(vendorRegistration);
session.getTransaction().commit();                           
session.close();

但是,您的其余代码仍然存在缺陷,因为如果出现错误/异常,可能会导致连接池不足。您应该在代码中使用try/catch/finally。 (或者更好的是让Spring通过使用Spring托管事务为您做到这一点)。

final Session session = getSession();
try {
    session.beginTransaction();
    session.save(vendorRegistration);
    session.getTransaction().commit();                           
} catch (Exception e) {
    session.getTransaction().rollback();
}  finally {       
    session.close();
}

对于每个与会话一起使用的方法,您基本上需要类似的东西。如果您不这样做,您的Session可能会闲逛并使用连接池中的连接。如果发生这种情况,您的应用程序将会死亡或开始爬行。

最好是使用Spring托管交易。

@Transactional
public saveNewUser(VendorRegistration vendorRegistration) {
    sessionFactory.getCurrentSession().save(vendorRegistration);
}

Spring现在为您管理会话和事务。

专业提示:不要使用普通的休眠,而是使用JPA并使用Spring Data JPA。那你只需要这个。

interface VendorRegistrationRepository extends JpaRepository<Long, VendorRegistration> {}

没有实现,只有一个界面,一切都是为您管理的。

答案 1 :(得分:0)

你的问题在

Query query = session.createQuery("UPDATE VendorRegistration set firstName =:firstname,lastName =:lastname,Email =:email,password =:password,Vid =:vid ");

您尚未添加where子句。所以所有记录都在更新。