JLabel während einer methode aktualisieren

[Jannik123]

Hallo , ich habe folgendes Problem : ich habe für die Schule ein programm geschrieben welches überprüfen soll ob eine eingegebene zahl eine primzahl ist, das funktioniert, nun habe ich das programm um die funktion erweitert dass es mir die primzahlen zwischen 2 zahlen berechnen soll ( zb 0 und 10 = 2,3,5,7), das funktioniert auch, nur scheitert es am detail, und zwar will ich mit den aktuellen bestand an primzahlen angucken -> wenn das programm die zahl 3 in diesem beispiel überprüft soll dort stehen 1 von 2 zahlen [0 und 1] sind primzahlen) ... das funktioniert bedingt -> am ende der gesammten operationen steht das finale ergebnis(auch richtig) auf dem label, nur möchte ich, dass man quasi live verfolgen kann bei welcher zahl sich der computer ungefähr befindet, also dass das label nach jeder rechenoperation erneuert wird ...
Quellcode :

import java.awt.Color;
import java.awt.List;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JCheckBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.event.DocumentEvent;
import javax.swing.event.DocumentListener;


public class Prime extends JFrame {
	
	JPanel colorPanel;
	JTextField input, start, end;
	JButton ok;
	JLabel total;
	List list;
	JCheckBox cBox;

	public Prime() {
		setTitle("Prime");
		setSize(500,800);
		setVisible(true);
		setLayout(null);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setResizable(false);
		setLocation(100,100);
		
		colorPanel = new JPanel();
		input = new JTextField();
		start = new JTextField();
		end = new JTextField();
		list = new List();
		cBox = new JCheckBox("numbers between");
		ok = new JButton("Calculate!");
		total = new JLabel("total numbers :");
		
		
		total.setBounds(10, 0, 480, 50);
		input.setBounds(10,50,200,50);
		cBox.setBounds(330,50,150,50);
		start.setBounds(10,110,150,50);
		end.setBounds(170, 110, 150, 50);
		ok.setBounds(330, 110, 100, 50);
		list.setBounds(10, 170, 480, 595);
		colorPanel.setBounds(220,50,100,50);
		colorPanel.setBackground(Color.gray);
		
		start.setEditable(false);
		end.setEditable(false);
		ok.setEnabled(false);
		
		ok.addActionListener(new ActionListener() {
			
		
			
			public void actionPerformed(ActionEvent e) {
				int pr = 0, nr = 0;
				
				long st = Long.parseLong(start.getText());
				long en = Long.parseLong(end.getText());
				if(st < en) {
					list.removeAll();
					

					for(; st <= en ; st ++) {
						if(isPrime(st)) {
							list.add(Long.toString(st));
							pr ++;
						}
						nr ++;
					
						total.setText("total primes :" + pr + " of " + nr +" numbers");
						
					
					}
					
				}
			}
		});
		
		cBox.addActionListener(new ActionListener() {
			
			public void actionPerformed(ActionEvent arg0) {
				if(cBox.isSelected()) {
					start.setEditable(true);
					end.setEditable(true);
					ok.setEnabled(true);
				}
				else {
					start.setEditable(false);
					end.setEditable(false);
					ok.setEnabled(false);
				}
				
			}
		});
		
		
		input.getDocument().addDocumentListener(new DocumentListener() {
			
			@Override
			public void removeUpdate(DocumentEvent arg0) {
				// TODO Auto-generated method stub
				
			}
			
			@Override
			public void insertUpdate(DocumentEvent arg0) {
				if(isPrime(Float.parseFloat(input.getText()))) {
					colorPanel.setBackground(Color.green);
				}
				else
					colorPanel.setBackground(Color.red);
				
			}
			
			@Override
			public void changedUpdate(DocumentEvent arg0) {
			
				
			}
		});
	
		add(cBox);
		add(list);
		add(start);
		add(end);
		add(input);
		add(colorPanel);
		add(ok);
		add(total);
		repaint();
	}
	
	public void setLabelText(String s) {
		total.setText(s);
	}
	
	
	public boolean isPrime(float p) {
		
		if(p > 1) {
			for(long i = 2; i != p;i++) {
				if(p % i == 0)
					return false;
			}
			
			return true;
		}
		
		else
			return false;
		
		
	}
	
	
	public static void main(String[] args) {
		new Prime();
	}

}

 

[Gucky]

Und wo ist das Problem? Du schreibst in die Schleife, dass in dem Label dein Text zu stehen hat.

PS: Dein Code ist...gewöhnungsbedürftig. Guck dir bitte noch mal das Konzept von Methoden und Objekten an.

 

[Jannik123]

erstmal danke für die schnelle antwort, du kannst das programm ja einmal ausführen und wirst sehen, dass das label nur einmal am ende aktualisiert wird.(in der konsole würde das live aktualisiert werden, im label jedoch nicht)
zu meinem programmierstil : ich weiß, dass ich sehr merkwürdig / schlecht schreibe, ich bin ja noch am anfang , könntest du mir vllt genauer sagen was du meinst / mir empfehlungen oder links zur verbesserung geben ?
mfg

 

[Gucky]

Das kommt daher, dass du total.setText(...) nicht IN der Schleife hast, sondern DANACH.

Google einfach mal nach METHODEN und OBJEKTEN und dem Sinn danach.

 

[Jannik123]

ne das ist in der schleife, wenn ich statt total.settext System.out.println schreibe schreibt er jeden schritt in die konsole, dh es muss etwas mit der klasse JLabel zu tun haben

 

[Gucky]

Meinst du, du hättest einen Bug gefunden?
In dem Code, den du gepostet hast ist keine Lableänderung innerhalb einer Schleife zu sehen.

 

[Hogli]

Hallo,
die Methode "actionPerformed" wird in swing im Eventthread ausgeführt. Methoden die länger dauern blockieren den Eventthread und nichts in dem UI kann während der Ausführung aktualisiert werden, auch das Label nicht.

Soll das UI trotzdem "responsive" sein, sollte der langwierige Programmteil am besten im Hintergrund-Thread ausgeführt werden und über Events dann der Fortschritt angezeigt werden. Swing hat dazu eine nützliche Klasse den SwingWorker. Zu beachten ist jedoch auch hier, dass jetzt während der langwierigen Bearbeitung weiterhin Eingaben möglich sind und deshalb alle Komponenten, dessen Bedienung während der Bearbeitung keinen Sinn mehr machen, auszuschalten sind (z.B: button.setEnabled(false)). Oftmals kann es sinnvoll sein, einen "Abbrechen"-Knopf aktiv zu lassen und dann ggf. den Hintergrundthread zu stoppen (swingWorker .cancel(true));

Leider entsteht bei der Nutzung hier eine Menge "boilerplate code", hier mal ein brauchbarer Code-Schnipsel dazu.

  private void action() {
    // ui.setEnabled(false);
    SwingWorker<String, Integer> swingWorker = new SwingWorker<String, Integer>() {
      @Override
      protected final String doInBackground() throws Exception {
        for (int i = 0; i < 100; i++) {
          // Hier den Inhalt übergeben für die asynchrone process Methode
          publish(i);
        }
        return "Objekt für get()";
      }

      @Override
      protected void process(List<Integer> chunks) {
        // Hier das Label setzen
        // label.setText(chunks.get(0));
      }

      @Override
      protected void done() {
        String erg;
        try {
          erg = get();
          if (erg.isEmpty()) {
            // ggf. Fortschritt wieder zurück setzen
            publish(0);
          }else{
            // eventuell Fehlermeldung
          }
        } catch (InterruptedException e) {
          // TODO Auto-generated catch block
          e.printStackTrace();
        } catch (ExecutionException e) {
          // TODO Auto-generated catch block
          e.printStackTrace();
        }finally{
          // ui.setEnabled(false);
        }
      };
    };
    swingWorker.execute();
  }

Grüße Hogli