Swing:函数的执行顺序

时间:2014-03-22 17:53:57

标签: java mysql swing concurrency

在我使用 Java Swing MySQL 实现的一个DBMS项目中,我遇到了一个非常有趣的问题。

考虑以下代码:

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

预期的内容:

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

当用户想要插入记录时,他单击该按钮并调用insertRecord()函数。此插入函数继承自抽象类。

插入getAllRecords()函数后,执行“SELECT”查询并将结果存储到全局ResultSet中。

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

明显发生的事情:

在执行insertRecord()函数时,还会执行另外两个函数getAllRecords()displayAllRecords()

这意味着插入后不显示我的数据,而是显示插入前的数据。

问题:

这似乎是与Swing和线程中的并发相关的问题,但有没有任何简单的解决方案可以让代码以我想要的方式工作,这不会对我的程序产生负面影响?

(如果您想查看其他代码段,请在评论中告诉我,我会将其作为编辑发布。原始代码很长,因此我没有发布。)< / em>的


编辑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();
}

用于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();
}

CODE FOR 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());

}

(对不起,代码太长而且很复杂。我试图在任何地方发表评论。我会感谢您提供的任何帮助。)

1 个答案:

答案 0 :(得分:1)

您的insertRecord()方法不会在数据库中插入记录。它只是显示一个新的框架。但是,您希望在显示此新帧后立即插入记录并准备好读取和显示。事实并非如此。

要插入,用户必须盯着框架,喝完一杯咖啡,去洗手间,回来,最后决定点击提交按钮。然后,只有这样,数据才会被插入数据库中。但是getAllRecords()displayAllRecords()方法已经执行了很长时间,因为只要显示帧,就会继续执行调用方法。

所以,你有两个解决方案:

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

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