java Swing：函数的执行顺序

Question

在我的一个使用JavaSwing和MySQL实现的DBMS项目中，我遇到了一个非常有趣的问题

考虑下面的代码：

if(source == insert){
    insertRecord(columnNames, numOfColumns);
    getAllRecords();
    displayAllRecords();
}

预期结果：

此代码是事件处理程序的一部分，它决定单击按钮时要执行的操作

当用户想要插入一条记录时，他点击按钮，调用insertRecord()函数。这个insert函数是从抽象类继承的

插入getAllRecords()函数后，执行“选择”查询，并将结果存储到全局结果集中

然后displayAllRecords()函数在屏幕上的表格中显示记录

显然发生了什么：

在执行insertRecord()函数的同时，其他两个函数getAllRecords()和displayAllRecords()也会执行

这意味着，它不显示插入后的数据，而是显示插入前存在的数据

问题：

这似乎是一个与Swing和线程中的并发性相关的问题，但有没有什么简单的解决方案可以让代码按照我希望的方式工作，而不会对我的程序产生负面影响

（如果你想查看其他代码，请在评论中告诉我，我会将它们作为编辑发布。原始代码很长，因此我没有发布。）

---

编辑1：为上述方法添加代码：

getAllRecords（）的代码。

public void getAllRecords(){
    // res is a global ResultSet field  
    try{
        res = sqlConn.getStatement().executeQuery("SELECT * FROM " + tableName);
    }catch(Exception e){
        System.out.println("Error obtaining data!");
    }
}

代码：displayAllRecords（）

public void displayAllRecords() {

    try{
        table = new JTable(buildTableModel());      // code provided at bottom
    }
    catch(SQLException e){
        e.printStackTrace();
        JOptionPane.showMessageDialog(null, "Unexpected error!");
    }
    displayResult();
}

显示结果（）的代码。

public void displayResult(){
    //updates the view port of the JScrollPane that is used for the table
    tablePane.getViewport().remove(table);
    table.revalidate();
    tablePane.getViewport().add(table);
    custFrame.repaint();
}

buildTableModel（）的代码。

public DefaultTableModel buildTableModel() throws SQLException{
    //res is a global ResultSet field
    Vector<String> columnNames = new Vector<String>();
    Vector<Vector<Object>> data = new Vector<Vector<Object>>();
    ResultSetMetaData metadata = res.getMetaData();

    //getting names of columns
    int columnCount = metadata.getColumnCount();
    for(int i = 0; i < columnCount; i++){
        columnNames.add(metadata.getColumnName(i+1));
    }

    //getting all data
    while(res.next()){
        Vector<Object> vec = new Vector<Object>();
        for(int i = 0; i < columnCount; i++){
            vec.add(res.getObject(i+1));
        }
        data.add(vec);
    }
    return new DefaultTableModel(data, columnNames);        
}

insertRecord（）的代码

public void insertRecord(ArrayList<String> columnNames, int numOfColumns){

    //creation of the GUI... don't know if it matters so kept it
    formFrame = new JFrame("INSERT DATA IN TABLE");
    labelList = new ArrayList<JLabel>();
    textList  = new ArrayList<JTextField>();
    submit = new JButton("SUBMIT");      
    prompt = new JLabel("Enter all the details:");

    formFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    formFrame.setAlwaysOnTop(true);
    formFrame.setResizable(false);
    formFrame.setAutoRequestFocus(true);

    formFrame.setLayout(new GridBagLayout());
    GridBagConstraints gbc = new GridBagConstraints();

    gbc.gridx = 0;
    gbc.gridy = 0;
    gbc.insets = new Insets(10, 10, 10, 10);
    gbc.fill = GridBagConstraints.BOTH;

    formFrame.add(prompt, gbc);
    gbc.gridy++;

    for(int i = 0; i < numOfColumns; i++, gbc.gridy++){

        labelList.add(new JLabel());
        labelList.get(i).setText(columnNames.get(i).toUpperCase());
        formFrame.add(labelList.get(i), gbc);

    }

    gbc.gridx = 1;
    gbc.gridy = 1;

    for(int i = 0; i < numOfColumns; i++, gbc.gridy++){

        textList.add(new JTextField(30));
        formFrame.add(textList.get(i), gbc);

    }

    gbc.gridx = 0;
    gbc.gridwidth = 2;
    formFrame.add(submit, gbc);

    formFrame.pack();
    formFrame.setVisible(true); 

    // creation and processing of the insert query

    class InsertHandler implements ActionListener{

        @Override
        public void actionPerformed(ActionEvent e){

            String query = "INSERT INTO " + tableName
                    + " VALUES( "; 

            int numOfColumns = textList.size();
            for(int i = 0; i < numOfColumns - 1; ++i){
                query +=  "'" + textList.get(i).getText() + "', ";
            }

            query += "'" + textList.get(numOfColumns-1).getText() + "')";

            try {
                sqlConn.getStatement().executeUpdate(query);
            } catch (SQLException e1) {
                JOptionPane.showMessageDialog(null, "Insertion failed!");
            }

        }

    };

    submit.addActionListener(new InsertHandler());

}

（抱歉，代码太长太复杂了。我已经尽可能地对其进行了注释。我将感谢您提供的任何帮助。）

Answer 1

你的insertRecord()方法不会在数据库中插入记录。它所做的只是显示一个新的帧。然而，您希望在显示了这个新帧之后，立即插入记录，并准备好读取和显示。事实并非如此

要插入，用户必须盯着镜框，喝完咖啡，去卫生间，回来，最后决定点击提交按钮。然后，也只有到那时，数据才会被插入数据库。但是getAllRecords()和displayAllRecords()方法将被执行很长时间，因为一旦显示帧，调用方法的执行就会继续

这里有两种解决方案：

1. 用模态JDialog替换insertRecord()显示的帧。模态JDialog将阻止执行，直到对话框关闭。然后，您可以检查用户是否确实单击了提交按钮，并刷新显示的数据

2. 成功插入数据后，调用InsertHandler.actionPerformed()方法中刷新数据的方法