class in neuen Thread auslagern

[PaulK]

Hallo an alle,

erstmal zu Situation: Ich bin dabei, ein Programm zum Überpüfen, ob eine Zahl prim ist, zu schreiben. Das Ganze wird in 2 Dateien (GUI und Algorithmus) und mit der BigInteger-Klasse realisisiert.
Um dem Benutzer eine Abschätzung der Rechenzeit zu ermöglichen, hab ich eine jprogressbar eingebaut. Allerdings schaffe ich es nicht, den value() dieser aus der 2. Datei heraus zu ändern. Mein zweites Problem ist, dass die Progressbar erst nach dem kompletten Schleifendurchlauf aktualisiert würde. Ich habe dazu viel gelesen und halte es für sinnvoll, den Algorithmus in einen extra Thread auszulagern. Allerdings weiß ich nicht, wie :cry:. Ich hab viel rumprobiert, bin aber zu keiner funktionierenden Lösung gekommen.

Ich würde mich freuen, wenn mir jemand einen Ansatz (auch mit Code :wink geben könnte, wie ich mein Programm verändern muss, damit die progressbar richtig funktioniert.

PS: Der Algorithmus an sich ist erstmal nur schnell "hingeschmiert" und noch gar nicht optimiert, aber um den geht es ja nicht...

mfG PaulK

GUI:

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


public class pz_GUI extends JFrame {
  // Anfang Variablen
  private JLabel jLabel1 = new JLabel();
  private JTextField txtZahl = new JTextField();
  private JButton cmdPruefen = new JButton();
  private JScrollPane jScrollPanetxtAusgabe = new JScrollPane();
  private JTextArea txtAusgabe = new JTextArea("");
  private JButton cmdEnde = new JButton();
  private JProgressBar pgbFortschritt = new JProgressBar();
  
  pz_BigInt p = new pz_BigInt();
  // Ende Variablen

  public pz_GUI(String title) {
    // Frame-Initialisierung
    super(title);
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent evt) { System.exit(0); }
    });
    int frameWidth = 843;
    int frameHeight = 267;
    setSize(frameWidth, frameHeight);
    setTitle("Primzahlen");
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);
    Container cp = getContentPane();
    cp.setLayout(null);
    
    // Anfang Komponenten

    jLabel1.setBounds(8, 10, 30, 13);
    jLabel1.setText("Zahl:");
    jLabel1.setFont (new Font("MS Sans Serif", Font.BOLD, 11));
    cp.add(jLabel1);
    txtZahl.setBounds(43, 8, 782, 21);
    txtZahl.setFont (new Font("MS Sans Serif", Font.BOLD, 11));
    txtZahl.setText("");
    txtZahl.setHorizontalAlignment(4);

    cp.add(txtZahl);
    cmdPruefen.setBounds(609, 39, 217, 22);
    cmdPruefen.setFont (new Font("MS Sans Serif", Font.BOLD, 11));
    cmdPruefen.setText("auf Primzahl überprüfen");
    cp.add(cmdPruefen);
    cmdPruefen.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        cmdPruefenActionPerformed(evt);
      }
    });

    cmdEnde.setBounds(610, 199, 217, 23);
    cmdEnde.setFont (new Font("MS Sans Serif", Font.BOLD, 11));
    cmdEnde.setText("Beenden");
    cp.add(cmdEnde);
    cmdEnde.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        cmdEndeActionPerformed(evt);
      }
    });

    pgbFortschritt.setBounds(8, 72, 817, 16);
    cp.add(pgbFortschritt);

    jScrollPanetxtAusgabe.setBounds(8, 97, 818, 91);
    txtAusgabe.setFont (new Font("MS Sans Serif", Font.BOLD, 11));
    txtAusgabe.setText("Copyright © Richard Schwark 2007\n");
    txtAusgabe.setEditable(false);
    jScrollPanetxtAusgabe.setViewportView(txtAusgabe);
    cp.add(jScrollPanetxtAusgabe);
    // Ende Komponenten

    setResizable(false);
    setVisible(true);
  }

  // Anfang Ereignisprozeduren
  public void cmdPruefenActionPerformed(ActionEvent evt) {
    txtAusgabe.setText(txtAusgabe.getText() + "\n" + p.istPrim(txtZahl.getText()));
  }

  public void cmdEndeActionPerformed(ActionEvent evt) {
    System.exit(0);
  }
  // Ende Ereignisprozeduren

  public static void main(String[] args) {
    new pz_GUI("pz_GUI");
  }
}

Algorithmus:

import java.math.BigInteger;

public class pz_BigInt {

  public String istPrim (String strZahl) {
      BigInteger Zahl = new BigInteger(strZahl);
      BigInteger Zwei = new BigInteger("2");
      BigInteger Drei = new BigInteger("3");
      BigInteger Hundert = new BigInteger("100");
      BigInteger Rest = new BigInteger("0");
      BigInteger i = new BigInteger("0");
      BigInteger Haelfte = new BigInteger("0");
      BigInteger Prozent = new BigInteger("0");
      BigInteger pzI = new BigInteger("0");
      int intProzent=0;

      Haelfte=Zahl.add(BigInteger.ONE);     // SchleifenEnde
      Haelfte=Zahl.divide(Zwei);
      Rest=Zahl.mod(Zwei);                  // Rest der Division

      if (Rest.equals(BigInteger.ZERO)) {    // -> gerade Zahlen aussortieren
        if (Zahl.compareTo(Zwei)==0) {       // Zahl=2
          return "Primzahl: "+strZahl+"!";
        } else {
           return "Keine Primzahl, da "+strZahl+" / 2 = "+Zahl.divide(Zwei)+"!";
        }
      } else {
          i=Drei;

          while (i.compareTo(Haelfte) != 1) {
            Rest=Zahl.mod(i);
            if (Rest.equals(BigInteger.ZERO)) {
              return "Keine Primzahl, da "+strZahl+" / "+i+" = "+Zahl.divide(i)+"!";
            }

            pzI=i.multiply(Hundert);

            Prozent=pzI.divide(Haelfte);
            intProzent=Prozent.intValue();

        //    pgbFortschritt.setValue(intProzent);

            i=i.add(Zwei);
          }
          if (Zahl.compareTo(BigInteger.ONE)!=0) { // Zahl != 1 (keine Primzahl)
            return "Primzahl: "+strZahl+"!";
          }else {
            return "Keine Primzahl: "+strZahl+"!";
          }

      }

    }
}

 

[Beni]

Ein ganz kleines Beispiel, wie man so einen Algorithmus in einen Thread auslagern könnte:

package forum;

import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;

public class Beispiel extends JFrame{
    private JProgressBar bar = new JProgressBar();
    private JButton button = new JButton( "Start" );
    private JTextField number = new JTextField();
    private JLabel result = new JLabel();
    
    public static void main( String[] args ) {
        Beispiel beispiel = new Beispiel();
        beispiel.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
        beispiel.pack();
        beispiel.setVisible( true );
    }
    
    public Beispiel(){
        setTitle( "Beispiel" );
        
        JPanel panel = new JPanel( new FlowLayout());
        number.setColumns( 8 );
        number.setText( "599999" );
        result.setText( "Resultat kommt hierhin" );
        panel.add( number );
        panel.add( result );
        panel.add( button );
        
        add( panel, BorderLayout.CENTER );
        add( bar, BorderLayout.SOUTH );
        
        button.addActionListener( new ActionListener(){
            public void actionPerformed( ActionEvent e ) {
                start();
            }
        });
    }
    
    public void start(){
        try{
            // Den Algorithmus starten
            long check = Long.parseLong( number.getText() );
            
            button.setEnabled( false );
            number.setEnabled( false );
            PrimAlgorithmus algo = new PrimAlgorithmus( check, this );
            algo.start();
        }
        catch( NumberFormatException e ){
            e.printStackTrace();
        }
    }
    
    public void setAsynchronProgress( final long max, final long value ){
        // Mit der GUI darf man nur indirekt kommunizieren, ein direkter
        // Zugriff aus dem falschen Thread könnte die GUI durcheinanderbringen
        EventQueue.invokeLater( new Runnable(){
            public void run() {
                double factor = ((double)Integer.MAX_VALUE) / max / 2.0;
                
                bar.setMaximum( (int)(max * factor));
                bar.setValue( (int)(value * factor));
                result.setText( value + " / " + max );
            } 
        });
        
        // Das hier hält den aktuellen Thread an, und lässt andere
        // Threads arbeiten.
        Thread.yield();
    }
    
    public void setAsynchronResult( final String text ){
        EventQueue.invokeLater( new Runnable(){
            public void run() {
                button.setEnabled( true );
                number.setEnabled( true );
                result.setText( text );
                bar.setValue( 0 );
            }
        });
    }
}

class PrimAlgorithmus extends Thread{
    private long check;
    private Beispiel beispiel;
    
    public PrimAlgorithmus( long check, Beispiel beispiel ){
        this.check = check;
        this.beispiel = beispiel;
    }
    
    @Override
    public void run() {
        if( check <= 0 )
            beispiel.setAsynchronResult( "Zahl zu klein" );
        else if( check == 1 )
            beispiel.setAsynchronResult( "Keine Primzahl" );
        else if( check == 2 )
            beispiel.setAsynchronResult( "Primzahl" );
        else{
            // Durch alle Zahlen teilen, und den Fortschritt
            // jeweils der GUI mitteilen.
            long max = check-1;
            for( long i = 2; i <= max; i++ ){
                if( check % i == 0 ){
                    beispiel.setAsynchronResult( "Keine Primzahl" );
                    return;
                }
                
                beispiel.setAsynchronProgress( max, i );
            }
            
            beispiel.setAsynchronResult( "Primzahl" );
        }
    }
}

 

[PaulK]

Danke Beni!

Ich werde gleich mal versuchen, das Beispiel in meinem Programm umzusetzen und melde mich dann später nochmal...

PK

 

[PaulK]

OK, also ich hab das Beispiel jetzt ausprobiert und angepasst. Ich kann es auch leicht auf meinen Algorithmus übertragen + in 2 Dateien aufsplitten.

Vielen Dank nochmal!

PK