2014-11-14 35 views
0

任何人都可以帮助我实现我的无响应jbutton线程?我想通过使用pepraredStatement插入到数据库。然而,无论何时添加数据库部分(connection和pepraredStatement),'UPLOAD'按钮都不响应。但是当我删除任何与数据库相关的东西时,我所有的按钮都在运行。你可以在这里找到代码http://pastebin.com/euKdWhr2无响应的jbutton请求并发

我会很感激任何帮助或建议。我有关于线程的解释,但我仍然无法将其包含到我的代码中。

public void actionPerformed(ActionEvent ev) 
     { 
      String file = fileField.getText(); 
      SetGetQuestionFileName pattern = new SetGetQuestionFileName(file); 
       ConnectToDatabase database = new ConnectToDatabase(); 
      try 
      { 

      ///////// check whether textfile is empty or not 

      if(ev.getActionCommand().equals("UPLOAD")) 
      { 
       if(fileField.getText().isEmpty()) 
       { 
        JOptionPane.showMessageDialog(null,"File field can not be empty!!! Please try again.","ALERT", JOptionPane.ERROR_MESSAGE); 
       } 
       else 
        { 
         File fi = new File(fileField.getText()); 
         //////////////// perform upload 

         try 
          { 

        String sql = "INSERT INTO testsystem.questionnaire (category_questions, questions, correct_answer)" + "VALUES (?, ?, ?)"; 

        PreparedStatement st = null; 

        Connection dbconnection = database.getConnection(); 

        st = dbconnection.prepareStatement(sql); 

            if(fi.getAbsoluteFile().exists()) 
            { 
             List<String> lines = Files.readAllLines(Paths.get(fileField.getText()), Charset.defaultCharset()); 


             for (int i = 0; i < lines.size(); i+=10) 
              { 
                String category = lines.get(i); 
                System.out.println(category); 
                String question = lines.get(i+1); 
                System.out.println(question); 

                String answers = 
                    lines.get(i+2)+System.lineSeparator() 
                    +lines.get(i+3)+System.lineSeparator() 
                    +lines.get(i+4)+System.lineSeparator() 
                    +lines.get(i+5); 
                System.out.println(answers); 

                String correct = lines.get(i+7); 
                System.out.println("correct answer is: "+correct); 
                System.out.println("----------------"); 


            st.setString(1, category); 
            st.setString(2, answers); 
            st.setString(3, correct); 
            st.executeUpdate(); 

             } 

             JOptionPane.showMessageDialog(null,"File has been successfully uploaded in the database.","NOTIFCATION",JOptionPane.INFORMATION_MESSAGE); 
            } 
       else 

         JOptionPane.showMessageDialog(null,"File could not be found. Please try again","ALERT",JOptionPane.ERROR_MESSAGE); 
       } 

         catch(SQLException ex) 
        { 

        } 

        catch(Exception ex) 
        { 

        } 
+0

认沽'e.printStackTrace()'到你的'catch'块。 – 2014-11-14 23:10:46

+0

您的问题:您在GUI线程中提供耗时的操作(调用EDT - 事件分派器线程)。只需将数据库代码移动到另一个线程中,例如使用[SwingWorker](https://docs.oracle.com/javase/8/docs/api/javax/swing/SwingWorker.html)。阅读[这里](https://docs.oracle.com/javase/tutorial/uiswing/concurrency/index.html),你会更好地理解你的问题。 – 2014-11-14 23:15:49

+0

这就是我现在得到的错误java.lang.NullPointerException at Lecturer $ MyAction.actionPerformed(Lecturer.java:183)and the line 183 is this one st = dbconnection.prepareStatement(sql); – mkwilfreid 2014-11-14 23:18:58

回答

2

当你拥有它强烈建议您将封装代码实际执行工作的行为。在你的情况下,它是数据库(称之为DBThread)。什么会为你工作,所以按钮不会没有响应是分为3部分(起初听起来太多,但它是有道理的)。所以你必须

  1. 行动(即调用DBThreadProgress)
  2. DBThreadProgress(这是一个线程,该线程将在年底加入,将调用DBThread)
  3. DBThread(这是作业)

因此,前面提到的DBThread是您的业务逻辑。该Action调用一个进度线程,您可以在面板上放置一个gif图像,以显示最终用户的背景信息。 progressThread允许您等待DBthread完成,以便您可以告知最终用户有关结果。我通常在开始时将按钮设置为setEnable(false),并且setEnable(true),以便用户知道他不能点击两次。

希望这有助于:-) enter image description here

+0

感谢您的建议,我设法使它工作 – mkwilfreid 2014-11-15 01:15:24