NetBeans 8.2中Hibernate插入失败：需手动分配ID求助

Hey there! Let's break down why you're hitting this error and how to fix it, plus some code optimizations to make your implementation cleaner.

问题根源 #

Your TblUsers entity uses email as its primary key (marked with @Id), but Hibernate doesn't see any automatic generation strategy for this key. By default, when you use @Id without specifying a generation strategy, Hibernate expects you to manually set the primary key value before calling session.save().

Looking at your registernewUser method, you're creating a TblUsers instance but never setting the email field—this is exactly why the error is thrown.

快速解决方案 #

Simply add a line to set the email value (which you already have as a method parameter) before saving the entity:

public String registerNewUser(String name, String email, String password, String mobileNo, String uType) {
    TblUsers ins = new TblUsers();
    // 必须手动设置主键email的值
    ins.setEmail(email); // 这行是关键！
    ins.setUsername(name);
    ins.setPassword(password);
    ins.setMobileNo(mobileNo);
    ins.setTblUsersType(uType); // 移除重复的setTblUsersType("C")赋值
    ins.setTblUserscol("D");

    try {
        Transaction t = session.beginTransaction();
        session.save(ins);
        t.commit();
        return "Operation executed";
    } catch(Throwable ex) {
        // 异常时回滚事务，避免数据库状态不一致
        if (t != null && t.isActive()) {
            t.rollback();
        }
        System.err.println("Session Issue " + ex.getMessage());
        ex.printStackTrace(); // 打印完整栈轨迹，方便排查问题
        return ex.getMessage();
    }
}

代码优化建议 #

Here are some tweaks to make your code more robust and maintainable:

1. 显式指定主键生成策略
   Since your primary key is a manually assigned string (email), explicitly add the generation strategy to your @Id field to make the code intent clear:

   @Id
   @GeneratedValue(strategy = GenerationType.ASSIGNED)
   @Column(name="email", unique=true, nullable=false)
   public String getEmail() {
       return this.email;
   }
   

2. 自动填充创建时间
   Use the @PrePersist annotation to automatically set the createTime when the entity is first saved, instead of relying on manual assignment:

   @PrePersist
   protected void onCreate() {
       createTime = new Date();
   }
   
   @Temporal(TemporalType.TIMESTAMP)
   @Column(name="create_time", length=26)
   public Date getCreateTime() {
       return this.createTime;
   }
   

3. 遵循Java命名规范

   • Rename registernewUser to registerNewUser (camelCase for method names)
   • Rename MObileNo to mobileNo, Utype to uType (camelCase for parameters)
   • Remove duplicate assignments like ins.setTblUsersType(Utype); followed by ins.setTblUsersType("C")—pick one valid value to set.

4. 完善事务边界
   Ensure transactions are properly rolled back on exceptions (already added in the solution code) to prevent partial data from being persisted.

5. 增强异常调试信息
   Printing just ex.getMessage() limits debugging visibility. Adding ex.printStackTrace() logs the full stack trace, which helps pinpoint exactly where errors originate.

内容的提问来源于stack exchange，提问作者user3879637