Java，在执行的jButton动作中读取输入流

Question

I am trying to read an input stream of a process that I create inside an actionPerformed function of a JButton for my interface. For that purpose, I have implemented a runnable class. The problem is that I get an output stream in quantas, meaning that, lets say I get 50 lines, than a big pause, and 50 lines more and such. The bigger problem is that the order of lines are not consistent. Here is what my code looks like...

private void jButton2ActionPerformed(java.awt.event.ActionEvent evt)
{
  try{
    String sCommand = "cmd /c \"myenvsetup.bat && myprogram.exe\"";
    Runtime rt = Runtime.getRuntime();
    pr = rt.exec(sCommand);
    java.awt.EventQueue.invokeLater(new Runnable()
    {
      public void run()
      {
        try{
          BufferedReader input = new BufferedReader(new InputStreamReader(pr.getInputStream()));
          String line = null;
          while((line = input.readLine()) != null){
            jTextArea1.append(line + "\n");
            jTextArea1.scrollRectToVisible(new Rectangle(0, jTextArea1.getHeight(), 0, 0));
            jTextArea1.update(jTextArea1.getGraphics());
          }
          pr.waitFor();
        } catch (Exception e) {
        }
      }
    });
  } catch (Exception e){
  }
}

Answer 1

In short you shouldn't do any blocking I/O in the event thread. Use another thread,

Answer 2

You should use a swing worker. I have written an example using an executor to do the IO.

ExecutorService executor = newSingleThreadExecutor()

The purpose of this example is to show, you do your work on the executor thread, and then you post your changes to the EDT.

try {
    String sCommand = "cmd /c \"myenvsetup.bat && myprogram.exe\"";
    Runtime rt = Runtime.getRuntime();
    pr = rt.exec(sCommand);
    executor.submit(new Runnable() {
      public void run(){
        try{
            BufferedReader input = new BufferedReader(
              new InputStreamReader(pr.getInputStream())
            );
            String line = null;
            while((line = input.readLine()) != null){
                final String l = line;
                EventQueue.invokeLater(new Runnable(){
                  @Override
                  public void run(){
                     jTextArea1.append(l + "\n");
                     jTextArea1.scrollRectToVisible(
                       new Rectangle(0, jTextArea1.getHeight(), 0, 0)
                     );
                     jTextArea1.repaint();
                 }
                });
              }
              pr.waitFor();
        } catch (Exception e) {}
      }
   });
} catch (Exception e){
}

The SwingWorker class takes care of doing work off of the EDT and allowing you to publish the work. It is more appropriate for this situation.

And you shouldn't use .update(Graphics g).