Minimalestopuhr: JLabel.setText() durch eigenes Event ändern

[apple-itob]

Guten Morgen,

ich versuche gerade eine kleine Stopuhr zu programmieren und soweit klappt auch alles ganz gut, das einzige, was mir noch nicht gelungen ist, ist die verstrichene Zeit in in

JLabel

zu schreiben (bisher gebe ich sie nur mit

System.out.println

aus). Ich denke, dass ich den Timer dazu bringen müsste, ein Event zu senden, das das Label auffangen kann, allerdings blicke ich auch mit Hilfe von Google nicht, wie das funktionieren sollte … vielleicht gibt es ja auch einen eleganteren Weg?

Die Implementierung orientiert sich am Model-View-Control-Entwurfsmuster (zur Übung), wobei ich Control und View in SwControl.java zusammenfasse. Hier mal der Code

// Datei: SwModel.java
// Das Model implementiert die Fähigkeiten
package de.tweh.sw;

public class SwModel {

	private int time = 0;
	private boolean running = false;

	Runnable timerRun = new Timer();
	Thread timer;

	public void start() {
		time = 0;
		System.out.println("Start");
		running = true;
		timer = new Thread(timerRun);
		timer.start();
	}

	public void stop() {
		running = false;
		timer.interrupt();
		System.out.println("Stoptime = " + time);
	}

	public int getTime() {
		return time;
	}

	public boolean isRunning() {
		return running;
	}

	class Timer implements Runnable {
		public void run() {
			while (running) {
				try {
					Thread.sleep(1000);
					time++;
					System.out.println("curren time = " + time);
				} catch (Exception e) {
				}
			}
		}
	}
}

// Datei: SwControl.java
// DIe Control-Datei verarbeitet Benutzereingaben und stellt Ergebnisse dar
package de.tweh.sw;

import javax.swing.*;
import javax.swing.event.*;

import java.awt.*;
import java.awt.event.*;

public class SwControl {

	JButton startStopButton;
	JLabel time;
	SwModel mod = new SwModel();

	SwControl(SwModel model) {
		this.mod = model;

		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		startStopButton = new JButton();
		startStopButton.setText("Start");
		startStopButton.addActionListener(new SwListener());

		time = new JLabel();
		time.setText("0");

		frame.add(BorderLayout.SOUTH, startStopButton);
		frame.add(BorderLayout.CENTER, time);

		frame.setSize(500, 350);
		frame.setVisible(true);
	}
	
	public void startStop() {
		if (mod.isRunning()) {
			startStopButton.setText("Start");
			mod.stop();
		} else {
			startStopButton.setText("Stop");
			mod.start();
		}
	}

	class SwListener implements ActionListener {
		public void actionPerformed(ActionEvent event) {
			startStop();
		}
	}
	
	class SwTimeListener implements ChangeListener {
		public void stateChanged(ChangeEvent event) {
			time.setText(""+mod.getTime());
		}
	}

}

// Datei: StopWatch.java
// hier werden Model und Control/View vereint
package de.tweh.sw;


public class StopWatch {

	public static void main(String[] args) {
		
		SwModel mod = new SwModel();
		SwControl controler = new SwControl(mod);
		
	}

}

Für allgemeine Anmerkungen zum Code bin ich jederzeit offen, ich bin relativ neu dabei und hab noch nicht so richtig im Blick, wie man was am besten umsetzt. Außerdem dürft ihr mir gerne Hinweise geben, wie ich das Beispiel hier besser posten kann – aus der LaTeX-Welt kenne ich Minimalbeispiele, bei Java weiß ich aber nicht, wie man das anstellt.

Danke euch!
apple-itob

PS: Das ist keine Hausaufgabe sondern mein Privatvergnügen ;-)

 

[KingOfExceptions]

Was willst du genau machen? Dass die bereits verstrichene Zeit angezeigt wird? ->

JLabel verstricheneZeit=new JLabel();
int zeit=0;
verstricheneZeit.setText(String.valueOf(zeit));
frame.validate();
frame.repaint();

 

[Michael...]

Ich hätte den Timer eher im Controller implementiert, ein Model hält ja eigentlich "nur" die Daten.
Damit sich die View aktualisieren kann, muss diese als Listener/Observer am Model registiert sein. Das Model benachrichtigt dann bei Änderung der Daten (in dem Fall time) alle regisiterten Listener/Observer

 

[apple-itob]

Dank euch beiden schonmal!

@KingOfExceptions: Ich will, dass das Label die vergangenen Sekunden anzeigt, also jede Sekunde aktualisiert wird. Wenn ich deinen Code so anschaue, passiert das ja nur einmal, sofern ich [c]frame.repaint();[/c] nicht regelmäßig aufrufe, das kann ich aber num im Model und von dort komme ich ja nicht an die [c]frame[/c]-Referenz. Also müsste ich den ganzen Timer-Kram, wie Michael sagt, mit in den Controler aufnehmen.

@Michael: Das dachte ich mir auch so, nur wie bekomme ich das Modell bzw. [c]timer[/c] dazu ein Event zu senden, dass das Label verstehen kann? Ich hab schon geoogelt und auch hier im Forum gesucht, aber nix gefunden was ich verstehen/übertragen konnte. Was auch damit zusammenhängt dass ich bei einigen Beispielen nicht zuordnen kann, was an welche Stelle bzw. in welche Datei gehört.



Bonusfrage: Wie kann ich das Zeitlabel in dem Fenster zentrieren? Im Moment klebt es ja am linken Rand …

 

[bERt0r]

Wenn du willst dass dein Model dein JLabel per ObserverPattern benachrichtigt bzw. ein ChangeEvent wirft musst du das eben implementieren. Das heißt du musst dir eine Collection<ChangeListener> im Model machen, add und remove ChangeListener Methoden und eine irgendwo die Liste durchgehen und jedem ein ChangeEvent schicken wenn sich was ändert. Du kannst auch die Klasse EventListenerList als Collection nehmen. Wenn du ein Beispiel brauchst: java - When to use EventListenerList instead of a general collection of listeners - Stack Overflow

 

[Michael...]

nur wie bekomme ich das Modell bzw. [c]timer[/c] dazu ein Event zu senden, dass das Label verstehen kann?

nicht das Label muss es verstehen, die View muss es mitbekommen.
Du kannst z.B. ein eigenes Interface definieren:

interface TimeModelListener {
     public void timeChanged();
}

Diese wird dann von Deiner View bzw. ViewController Klasse implementiert.

Dem Model muss ein solcher Listener übergeben werden können. Die Codeabschnitte könnten so aussehen:

public class SwModel {
    private TimeModelListener listener;
...
    // hierüber kann sich z.B. die View als Listener am Model registrieren
    public void addTimeModelListener(TimeModelListener listener) {
        this.listener = listener;
    }
...
    // Die Variable tiem sollte nur über eine Methode veränderbar ...
    public void increaseTime() {
        time++;
        // benachrichtigen des Listeners
        listener.timeChanged();
    }

    // ... und auch nur über eine Methode abgreifbar sein
    public int getTime() {
        return time;
    }
...
}

in der Implementierung der Methode timeChanged() in der View holst Du Dir über getTime() am Model den aktuellen Wert und schreibst diesen ins Label

label.setText(String.valueOf(model.getTime()));

Den Ansatz hattest Du ja bereits in Deinem SwTimeListener, nur hast Du ihn nicht verwendet (und man müsste noch ein Event Objekt erzeugen)

Bonusfrage: Wie kann ich das Zeitlabel in dem Fenster zentrieren? Im Moment klebt es ja am linken Rand …

evtl. reicht ja schon das:

time = new JLabel("", JLabel.CENTER);

 

[apple-itob]

@Michael: Das hat geklappt danke! Ich habe inzwischen übrigens auch noch eine eigene Lösung dem Observer-Pattern folgend gefunden …