Java Swing跨类获取JLabel值失败致SQL数据迁移失效问题排查

咱们一步步拆解你代码里的问题，这些都是导致无法获取ID、迁移逻辑失效的核心原因：

1. 类实例化的低级错误（最直接的问题） #

你在按钮点击事件里实例化的是Repeat_student类，但你的迁移逻辑明明写在transfer_person类里啊！这就导致你调用的insertID()和makeTransfer()根本不是处理迁移的方法，自然拿不到目标ID，也执行不了数据迁移。

修复：把按钮事件里的类改成transfer_person：

private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) {
    transfer_person rsp = new transfer_person();
    rsp.insertID();
    rsp.makeTransfer();
}

2. 非静态组件的访问错误 #

你的insertID()方法里直接用Transfer.displayID.getText()，如果displayID不是Transfer类的静态成员，这种写法要么编译报错，要么运行时抛空指针。正确的做法是获取Transfer窗口的实例，而不是直接通过类名访问组件。

推荐修复思路：
不要直接依赖静态组件，而是在transfer_person类里添加带参数的构造方法，直接传入ID值：

// 修改transfer_person类，新增带ID参数的构造方法
private String targetID;
public transfer_person(String personId) {
    con = myKIDS.connectKids();
    this.targetID = personId;
}

// 同时删除原来的insertID()方法，按钮事件里先获取ID再传入
private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) {
    // 确保这里的Transfer是当前打开的窗口实例，而不是类名
    String personId = yourTransferWindowInstance.displayID.getText();
    transfer_person rsp = new transfer_person(personId);
    rsp.makeTransfer();
}

3. SQL注入风险与语法错误隐患 #

你用字符串拼接的方式写SQL语句：where ID='"+getID+"'，如果ID里包含特殊字符（比如单引号）会直接导致SQL语法错误，还存在严重的SQL注入风险。

修复：使用PreparedStatement的参数绑定功能：

// 插入table2的语句修改为
String sql = "Insert into table2(ID,NAME,MIDDLENAME,SURNAME,OLD_ARREARS,DATE,CONTACT) " +
             "Select ID,NAME,MIDDLENAME,SURNAME,OLD_ARREARS,DATE,CONTACT from table1 where ID=?";
pst = con.prepareStatement(sql);
pst.setString(1, targetID); // 绑定参数，避免注入和语法错误
pst.executeUpdate(); // 执行更新操作要用executeUpdate()，不是execute()

// 删除table1的语句同理
sql = "delete from table1 where ID=?";
pst = con.prepareStatement(sql);
pst.setString(1, targetID);
pst.executeUpdate();

4. 资源关闭的空指针问题 #

你的finally块里直接关闭rs和pst，但这些对象可能根本没被初始化（比如SQL执行出错时），会抛出空指针异常。

修复：关闭前先判断对象是否为null：

finally {
    try {
        if (rs != null) rs.close();
        if (pst != null) pst.close();
    } catch(Exception e) {
        e.printStackTrace(); // 打印异常信息，方便调试
    }
}

5. 事务一致性问题 #

迁移操作是「插入table2 + 删除table1」，这两个操作应该是原子性的：要么都成功，要么都失败，不然会出现数据不一致（比如插入成功但删除失败，导致table1和table2都有同一条数据）。

修复：开启事务，手动控制提交或回滚：

public void makeTransfer(){
    try {
        con.setAutoCommit(false); // 关闭自动提交，开启手动事务
        
        // 插入table2的逻辑（用参数绑定）
        String sql = "Insert into table2(ID,NAME,MIDDLENAME,SURNAME,OLD_ARREARS,DATE,CONTACT) " +
                     "Select ID,NAME,MIDDLENAME,SURNAME,OLD_ARREARS,DATE,CONTACT from table1 where ID=?";
        pst = con.prepareStatement(sql);
        pst.setString(1, targetID);
        pst.executeUpdate();
        
        // 删除table1的逻辑（用参数绑定）
        sql = "delete from table1 where ID=?";
        pst = con.prepareStatement(sql);
        pst.setString(1, targetID);
        pst.executeUpdate();
        
        con.commit(); // 两个操作都成功，提交事务
        JOptionPane.showMessageDialog(null,"student / pupil with id :"+targetID+" repeated to Form 2 Successfully.");
    }catch(SQLException | HeadlessException e){
        try {
            con.rollback(); // 出错就回滚，恢复到操作前状态
        } catch (SQLException ex) {
            ex.printStackTrace();
        }
        JOptionPane.showMessageDialog(null,e);
    } finally{
        try{
            if (rs != null) rs.close();
            if (pst != null) pst.close();
            con.setAutoCommit(true); // 恢复自动提交
        } catch(Exception e){
            e.printStackTrace();
        }
    }
}

6. 变量命名不规范 #

public String getID;这个变量名不符合Java命名规范，容易和getter方法混淆，建议改成小驼峰命名，比如public String personId;。

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