Можно ли использовать индикатор выполнения в классе вне main? - PullRequest
Новая
       46

Можно ли использовать индикатор выполнения в классе вне main?

31 голосов
/ 09 января 2011

Прямо сейчас мой основной просто вызывает графический интерфейс с 10 строками.Исходя из того, сколько из этих строк имеют текст, вызывается 1 из 9 классов (две строки должны иметь текст).Вызываемый класс выполняет вычисления, к которым я хотел бы привязать индикатор выполнения.Вот пример одного из названных классов (каждый класс похож, но достаточно различен, чтобы оправдать новый класс.) Я считаю, что проблема является нарушением правил EDT, но все примеры, которые я видел на них, связаны с основнымаргумент.Кадр появляется при запуске кода, но индикатор выполнения не обновляется, пока не будут выполнены все вычисления. 

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class twoLoan extends JFrame {

    static JFrame progressFrame;
    static JProgressBar progressBar;
    static Container pane;
    double amountSaved = 0;
    int i = 0;

    public void runCalcs(Double MP, Double StepAmt,
        Double L1, Double L2, Double C1, Double C2,
        Double IM1, Double IM2, Double M1Start, Double M2Start) {

        try {
            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        } catch (Exception e) {
        }

        int iterations = (int) (MP - (M1Start * M2Start));

        //Create all components
        progressFrame = new JFrame("Calculation Progress");
        progressFrame.setSize(300, 100);
        pane = progressFrame.getContentPane();
        pane.setLayout(null);
        progressFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        progressBar = new JProgressBar(0, iterations);

        //Add components to pane
        pane.add(progressBar);

        //Position controls (X, Y, width, height)
        progressBar.setBounds(10, 10, 280, 20);

        //Make frame visible
        progressFrame.setResizable(false); //No resize
        progressFrame.setVisible(true);

        double M1 = M1Start;
        double M2 = M2Start;

        // Set MinLoop as maximum to start
        // Loan 1
        double N1 = (Math.log10(1 - IM1 * L1 / M1) * -1) / Math.log10(1 + IM1);
        double M1Sum = M1 * N1;
        // Loan 2
        double N2 = (Math.log10(1 - IM2 * L2 / M2) * -1) / Math.log10(1 + IM2);
        double M2Sum = M2 * N2;
        double minLoop = M1Sum + M2Sum;
        double MTotal = 0;


        // Define variables for mins
        double MP1 = 0;
        double MP2 = 0;
        double NP1 = 0;
        double NP2 = 0;
        double MP1Sum = 0;
        double MP2Sum = 0;

        while (M1 <= MP - M2Start && M2 >= M2Start) {
            N1 = (Math.log10(1 - IM1 * L1 / M1) * -1) / Math.log10(1 + IM1);
            M1Sum = N1 * M1;
            N2 = (Math.log10(1 - IM2 * L2 / M2) * -1) / Math.log10(1 + IM2);
            M2Sum = N2 * M2;
            MTotal = M1Sum + M2Sum;
            if (MTotal < minLoop) {
                minLoop = MTotal;
                MP1 = M1;
                MP2 = M2;
                NP1 = N1;
                NP2 = N2;
                MP1Sum = M1Sum;
                MP2Sum = M2Sum;
            } // end if
            M1 = M1 + StepAmt;
            M2 = MP - M1;
            // Reset monthly sums
            M1Sum = 0;
            M2Sum = 0;
            i++;
            progressBar.setValue(i);
            progressBar.repaint();
            if (i >= iterations) {
                progressFrame.dispose();
            }
        } // end while

        // if there's a value for current payments, calculate amount saved
        if (C1 > 0) {
            double CN1 = (Math.log10(1 - IM1 * L1 / C1) * -1) / Math.log10(1 + IM1);
            double CT1 = CN1 * C1;

            double CN2 = (Math.log10(1 - IM2 * L2 / C2) * -1) / Math.log10(1 + IM2);
            double CT2 = CN2 * C2;

            double CTotal = CT1 + CT2;
            amountSaved = CTotal - minLoop;
        }

    } // end method runCalcs

    //Workbook wb = new HSSFWorkbook();
    public double savedReturn() {
        return amountSaved;
    }
} // end class twoLoans

Ответы [ 3 ]

30 голосов
/ 09 января 2011

SwingWorker идеально подходит для этого.Приведенный ниже пример выполняет простую итерацию в фоновом режиме, сообщая о ходе и промежуточных результатах в окне.Вы можете передать любые необходимые параметры в подходящем конструкторе SwingWorker. 

import java.awt.EventQueue;
import java.awt.GridLayout;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.text.DecimalFormat;
import java.util.List;
import javax.swing.*;

/** @see http://stackoverflow.com/questions/4637215 */
public class TwoRoot extends JFrame {

    private static final String s = "0.000000000000000";
    private JProgressBar progressBar = new JProgressBar(0, 100);
    private JLabel label = new JLabel(s, JLabel.CENTER);

    public TwoRoot() {
        this.setLayout(new GridLayout(0, 1));
        this.setTitle("√2");
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.add(progressBar);
        this.add(label);
        this.setSize(161, 100);
        this.setLocationRelativeTo(null);
        this.setVisible(true);
    }

    public void runCalc() {
        progressBar.setIndeterminate(true);
        TwoWorker task = new TwoWorker();
        task.addPropertyChangeListener(new PropertyChangeListener() {

            @Override
            public void propertyChange(PropertyChangeEvent e) {
                if ("progress".equals(e.getPropertyName())) {
                    progressBar.setIndeterminate(false);
                    progressBar.setValue((Integer) e.getNewValue());
                }
            }
        });
        task.execute();
    }

    private class TwoWorker extends SwingWorker<Double, Double> {

        private static final int N = 5;
        private final DecimalFormat df = new DecimalFormat(s);
        double x = 1;

        @Override
        protected Double doInBackground() throws Exception {
            for (int i = 1; i <= N; i++) {
                x = x - (((x * x - 2) / (2 * x)));
                setProgress(i * (100 / N));
                publish(Double.valueOf(x));
                Thread.sleep(1000); // simulate latency
            }
            return Double.valueOf(x);
        }

        @Override
        protected void process(List<Double> chunks) {
            for (double d : chunks) {
                label.setText(df.format(d));
            }
        }
    }

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {

            @Override
            public void run() {
                TwoRoot t = new TwoRoot();
                t.runCalc();
            }
        });
    }
}
4 голосов
/ 09 января 2011

Я думаю, что вы правы, вы должны придерживаться правил потоков Swing.

Так что же делать?

Во-первых, я не уверен, как именно разработано ваше приложение. Вы говорите, что у вас есть основной фрейм с кучей строк, и потенциально каждый из них может потенциально вызвать один из 9 классов, и все они выглядят так, как показано выше. Кажется, что эти классы будут генерировать свои собственные JFrame. Я думаю, что этот новый кадр используется исключительно для индикатора выполнения. Я предположу, что это дизайн и предложу соответственно.

Я предлагаю вам выполнить пару действий в экземплярах Runnable, и вы перетащите эти Runnable экземпляры в SwingUtilities.invokeLater, чтобы они запускались на EDT. В то же время, если бы вы читали, я бы потратил время на реорганизацию вашего кода.

  1. переместите создание ваших битов GUI в метод: 
public void createComponents () {
      SwingUtilities.invokeLater(new Runnable() {
        public void run() {
          //Create all components
          progressFrame = new JFrame("Calculation Progress");
          progressFrame.setSize(300, 100);
          pane = progressFrame.getContentPane();
          pane.setLayout(null);
          progressFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          progressBar = new JProgressBar(0, iterations);
          //Add components to pane
          pane.add(progressBar);

          //Position controls (X, Y, width, height)
          progressBar.setBounds(10, 10, 280, 20);

          //Make frame visible
          progressFrame.setResizable(false); //No resize
          progressFrame.setVisible(true);
       }
      });

    }
  1. Тогда я бы систематизировал два действия с графическим интерфейсом, которые у вас есть в Calc: 
     private void updateProgressBar(final int i) {
           SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                   progressBar.setValue(i);
                   //no need for the following
                   //progressBar.repaint(); 

                }
           });
    }

    private void killDialog() {
           SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    progressFrame.setVisible(false);
                }
            });
    }
  1. Наконец, замените, где код, содержащийся в этих новых методах, на вызовы методов.
3 голосов
/ 15 января 2011

Спасибо за помощь. Я начал с попытки использовать первый ответ, но не смог запустить панель одновременно, и она запустилась после завершения программы. Я уверен, что это сработает, но я не смог понять это. Используя ответ trashgod и некоторые другие примеры, я смог заставить его работать, используя SwingWorker. К сожалению, я не совсем понимаю, как это работает, но сейчас я возьму это.

GUI и метод для выполнения вычислений сначала вызываются в другом классе: 

iterations = (int) (MPay - (M1Start + M2Start));
        twoLoan myLoan = new twoLoan();
        myLoan.createGui(iterations);
        myLoan.runCalcs(MPay, Step, L1, L2, C1, C2, IM1, IM2, M1Start, M2Start);

Тогда он работает следующим образом: 

public class twoLoan extends JFrame {

    JFrame progressFrame;
    JProgressBar progressBar;
    JLabel label = new JLabel("Calculating...");;
    Container pane;

    double amountSaved = 0;
    int i = 0;
    int iterations;

    public void createGui(int iterations) {
           //Create all components
          progressFrame = new JFrame("Calculation Progress");
          progressFrame.setSize(300, 100);
          pane = progressFrame.getContentPane();
          pane.setLayout(null);
          label = new JLabel("Calculating...");
          label.setBounds(115, 35, 200, 25);
          progressBar = new JProgressBar(0, iterations);
          progressBar.setBounds(10, 10, 280, 20);
          progressBar.setStringPainted(true);
          //Add components to pane
          pane.add(progressBar);
          pane.add(label);
          //Make frame visible
          progressFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          progressFrame.setResizable(false); //No resize
          progressFrame.setLocationRelativeTo(null);
          progressFrame.setVisible(true);
    }

    public void runCalcs (double MP, double StepAmt, double L1, double L2,
            double C1, double C2, double IM1, double IM2, double M1Start, double M2Start) {

        progressBar.setIndeterminate(false);
        TwoWorker task = new TwoWorker(MP, StepAmt, L1, L2, C1, C2, IM1, IM2, M1Start, M2Start);
        task.addPropertyChangeListener(new PropertyChangeListener() {

            @Override
            public void propertyChange(PropertyChangeEvent e) {
                if ("progress".equals(e.getPropertyName())) {
                    progressBar.setIndeterminate(false);
                    progressBar.setValue((Integer) e.getNewValue());
                }
            }
        });
        task.execute();
    } //end method runCalcs

    public class TwoWorker extends SwingWorker<Double, Double> {

        private final double MP, StepAmt,L1, L2,
            C1, C2, IM1, IM2, M1Start, M2Start;

        public TwoWorker(double MPa, double StepAmta, double L1a, double L2a,
            double C1a, double C2a, double IM1a, double IM2a, double M1Starta, double M2Starta) {

            MP = MPa;
            StepAmt = StepAmta;
            L1 = L1a;
            L2 = L2a;
            C1 = C1a;
            C2 = C2a;
            IM1 = IM1a;
            IM2 = IM2a;
            M1Start = M1Starta;
            M2Start = M2Starta;
           }
        @Override
        protected Double doInBackground() {

            double M1 = M1Start;
            double M2 = M2Start;

        // Set MinLoop as maximum to start
        // Loan 1
        double N1 = (Math.log10(1 - IM1 * L1 / M1) * -1)/Math.log10(1 + IM1);
    double M1Sum = M1 * N1;
    // Loan 2
    double N2 = (Math.log10(1 - IM2 * L2 / M2) * -1)/Math.log10(1 + IM2);
    double M2Sum = M2 * N2;
    double minLoop = M1Sum + M2Sum;
    double MTotal = 0;

        // Define variables for mins
    double MP1 = 0;
    double MP2 = 0;
    double NP1 = 0;
    double NP2 = 0;
    double MP1Sum = 0;
    double MP2Sum = 0;

        while ( M1 <= MP - M2Start && M2 >= M2Start ) {
            N1 = (Math.log10(1 - IM1 * L1 / M1) * -1)/Math.log10(1 + IM1);
            M1Sum = N1 * M1;
            N2 = (Math.log10(1 - IM2 * L2 / M2) * -1)/Math.log10(1 + IM2);
            M2Sum = N2 * M2;
            MTotal = M1Sum + M2Sum;
            if (MTotal < minLoop) {
                minLoop = MTotal;
                MP1 = M1;
                MP2 = M2;
                NP1 = N1;
                NP2 = N2;
                MP1Sum = M1Sum;
                MP2Sum = M2Sum;
            } // end if
                        i++;
                        progressBar.setValue(i);
                    M1 = M1 + StepAmt;
            M2 = MP - M1;
            // Reset monthly sums
            M1Sum = 0;
            M2Sum = 0;
        } // end while

        System.out.printf("MP1 = %.2f\n", MP1);
        System.out.printf("MP2 = %.2f\n", MP2);
        System.out.printf("NP1 = %.2f\n", NP1);
        System.out.printf("NP2 = %.2f\n", NP2);
        System.out.printf("MP1Sum = %.2f\n", MP1Sum);
        System.out.printf("MP2Sum = %.2f\n", MP2Sum);
                System.out.printf("MTotal = %.2f\n", minLoop);
                System.out.printf("i = %d\n",i);
                System.out.printf("M1Start = %.2f\n", M1Start);
        System.out.printf("M2Start = %.2f\n", M2Start);
                System.out.printf("MP= %.2f\n",MP);

    // if there's a value for current payments, calculate amount saved
    if( C1 > 0 ) {
        double CN1 = (Math.log10(1 - IM1 * L1 / C1) * -1)/Math.log10(1 + IM1);
        double CT1 = CN1 * C1;

        double CN2 = (Math.log10(1 - IM2 * L2 / C2) * -1)/Math.log10(1 + IM2);
        double CT2 = CN2 * C2;

        double CTotal = CT1 + CT2;
        amountSaved = CTotal - minLoop;
        } // end if

        return null;

    } // end doInBackGround

        @Override
        protected void done() {
            label.setBounds(133, 35, 200, 25);
            label.setText("Done!");
        }
    } // end TwoWorker


    public double savedReturn() {
        return amountSaved;
    }

} // end class twoLoans
...