Einfache MVC-Uhr. MVC richtig umgesetzt? [Überarbeitet]

[razor]

Hallo All,

das wird mein erster Beitrag hier im Forum.
Habe noch nicht viel Erfahrung mit Java, bis jetzt nur ein paar kleine Programme geschrieben.
Ich befasse mich zur Zeit mit Entwurfmustern und wollte mein kleines MVC Programm mal zur Einsicht hier reinstellen.
Habe mir bereits viele Beispiele im Netz angeschaut und versucht sie nachzuvollziehen.
Meine Frage wäre, ob ich ModellViewController richtig "verstanden" habe?
Vielleicht gibt es etwas zu verbessern.

Das Programm an sich läuft, mir geht es Hauptsächlich ums MVC Paradigma, aber falls es auch Verbesserungsvorschläge bzgl. Code gibt, immer her damit.

Vielen Dank in vorraus!

PS:
war mir nicht sicher, ob es vielleicht doch in den OOP Bereich gehört, falls ja Bitte verschieben.

MainKlasse

package pckg;

import java.awt.BorderLayout;
import java.awt.Component;
import java.awt.Toolkit;
import javax.swing.JFrame;

public class TimeMain 
{

	public static void main(String[] args)
	{
		TimeModel model = new TimeModel();
		TimeView view = new TimeView(model);
		TimeController controller = new TimeController(model, view);

		init(view, controller);
	}
	/**
	 * Init erzeugt ein Frame und belegt es mit der Ausgabekomponente und Kontrollkomponente
	 * @param view = Ausgabekomponente
	 * @param controller = Kontrollkomponente
	 */
	public static void init(Component view, Component controller)
	{
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setBounds(200, 200, 300, 100);
		CentrFrame(frame);
		frame.setVisible(true);
		frame.getContentPane().add(view, BorderLayout.CENTER);
		frame.getContentPane().add(controller, BorderLayout.SOUTH);
	}
	/**
	 * Zentrierung des Fensters
	 * @param f = Frame (Fenster, das zentriert werden soll)
	 */
	public static void CentrFrame(JFrame f)
	{
		double height=Toolkit.getDefaultToolkit().getScreenSize().height;
		double width=Toolkit.getDefaultToolkit().getScreenSize().width;
		f.setBounds( (int)width/2-f.getWidth()/2,(int)height/2-f.getHeight()/2, f.getWidth(),f.getHeight());
	}
}

ModellKlasse

package pckg;

import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.Observable;
import java.util.Timer;


public class TimeModel extends Observable 
{
	private String time = new String();
	private boolean tick;
	private Timer timer = new Timer();

	public TimeModel() 
	{
		DateFormat formatter = new SimpleDateFormat("HH.mm.ss / dd.MM.yyyy");
		this.time = formatter.format(new Date());
		this.tick=false;
	}
	/**
	 * Zeit setzen (wird erst mal nicht gebraucht)
	 * @param s
	 */
	public void setTime(String s)
	{
		this.time =s;
		setChanged();
		notifyObservers();
	}
	/**
	 * Zeit ausgeben (wird erst mal nicht gebraucht)
	 * @return
	 */
	public String getTime()
	{
		return time;
	}
	/**
	 * Zeit vom System hollen
	 *
	 */
	public void updateTime()
	{
		DateFormat formatter = new SimpleDateFormat("HH.mm.ss / dd.MM.yyyy");
		this.time = formatter.format(new Date());
		setChanged();
		notifyObservers();
	}
	/**
	 * Aktualisierung der Zeit (on/off)
	 * mit Hilfe der TimeTick KLasse wird die Zeit in TimeModel 
	 * mit der Systemzeit synchronisiert (Interval 1sek)
	 *
	 */
	public void tickTime()
	{
		if(!tick)
		{
			this.tick=true;
			Timer t = new Timer();
			t.schedule(new TimeTick(this), 0, 1000);
			this.timer=t;
		}
		else 
		{
			this.tick=false;
			timer.cancel();
		}
	}
}

TimeTick Klasse (Wird von der TimeModell KLasse benötigt)

package pckg;

import java.util.TimerTask;

public class TimeTick extends TimerTask
{
	private TimeModel model = new TimeModel();
	
	public void run() 
	{
		model.updateTime();
	}
	public TimeTick(TimeModel m)
	{
		this.model = m;
	}
}

ViewKlasse

package pckg;

import java.awt.Color;
import java.awt.Font;
import java.util.Observable;
import java.util.Observer;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class TimeView extends JPanel implements Observer
{
	/**
	 * 
	 */
	private static final long serialVersionUID = 830996912263397813L;
	private JLabel label = new JLabel();
	private TimeModel model = new TimeModel();
	
	/**
	 * Erzeugt eine Ausgabekomponente der Klasse JPanel mit einem Label in der Mitte
	 * @param m = das Modell
	 */
	public TimeView(final TimeModel m)
	{
		super();
		this.model = m;

		label.setHorizontalAlignment(JLabel.CENTER);		
		label.setForeground(Color.BLUE);
		label.setFont(new Font("Georgia", 1, 18));
		
		this.add(label);
	}
	/**
	 * Updatemethode
	 */
	public void update(Observable obs, Object obj) 
	{
		label.setText(model.getTime());
	}
}

ControllerKlasse

package pckg;

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JPanel;

public class TimeController extends JPanel 
{
	/**
	 * 
	 */
	private static final long serialVersionUID = 5216809802813019642L;
	private JButton button = new JButton("Start / Stop");
	
	/**
	 * Kontroller
	 * @param model = das Modell
	 * @param view = die Ausgabekomponente
	 */
	public TimeController(final TimeModel model, final TimeView view)
	{
		super();
		// Layout setzten und Button einfügen
		this.setLayout(new BorderLayout());
		this.add(button, BorderLayout.CENTER);
		
		// Observer registrieren
		model.addObserver(view);
		
		// ButtonListener erzeugen
		button.addActionListener(new ActionListener()
		{
			public void actionPerformed(ActionEvent e)
			{
				// Starte oder Stoppe die System-Zeitanzeige
				model.tickTime();
			}
		});
	}
}

 

[HLX]

Ich habe bislang noch nicht mit dem Observer-Pattern gearbeitet. Ich sehe nur das eine enge Bindung zwischen Model und View besteht. Dein Model kennt deine View und die View das Model. Von daher bin ich mir nicht sicher, ob man das noch im Sinne von MVC getrennt ansehen kann.

2 Dinge würde ich ändern:
1.) die Main-Methode von GUI-Einstellungen befreien:

      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
      frame.setBounds(200, 200, 300, 100); 
      CentrFrame(frame); 
      frame.setVisible(true); 
      frame.getContentPane().add(view, BorderLayout.CENTER); 
      frame.getContentPane().add(controller, BorderLayout.SOUTH);

Das alles in den Konstruktor oder in eine Initialisierungsmethode von deinem Frame.

2.) Den Kontroller nicht von einem Panel sondern von einem Listener ableiten. Der Listener ist in deinem Falle der eigentliche Controller.

 

[razor]

@ HLX
Vielen Dank für deine schnelle Antwort.
ZU den Observer, habe gestern irgendwo gelesen, dass man bei Java mit Observer eine Art eigenes (vereinfachtes) MVC Konzept hat. Der unterscheidet sich darin, dass sie Viewer und Controller zusammenlegen !? Sprich, falls ich MVC richtig umsetzten will muss ich wohl auf Interfaces ? umsteigen.

Pkt 1) den Frame und die Parameter verlege ich die TimeView Klasse, oder vielleicht doch in eine eigene KLasse.

Pkt 2) ok

 

[razor]

Mein zweiter Versuch:

Ich hoffe jemand hat etwas Zeit um es sich anzugucken, ich habe es möglichst einfach programmiert und es ist nicht soviel wie auf dem ersten Blick ausschaut.
Ich will wieder wissen ob ich MVC Prinzip umgesetzt habe.

TimeModell hat einen String time, dieses wird jede Sekunde an die Systemzeit angepasst, aktualisiert.
TimeView hat zwei Componenten JLabel (zur Ausgabe der Zeit) und JButton (damit Starte und Stoppe ich die Zeitanzeige durch den Label).
TimeController verwaltet View und Model.

Verbesserungen oder Vorschläge ?

THX in vorraus!



Main-Klasse

public class TimeMainTwo {
	public static void main(String[] args) {
		TimeModel tmodel = new TimeModel();
		TimeView tview = new TimeView();
		TimeController ctrl = new TimeController(tmodel,tview);
		tmodel.timeTick();
	}
}

Model-Klasse

import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.Observable;
import java.util.Timer;




public class TimeModel extends Observable {
	
	public String time;	
	
	public void setTime(String t){
		time= t;
		setChanged();
		notifyObservers();
	}
	
	public String getTime(){
		return time;
	}

	public void timeTick(){
		Timer t = new Timer();
		t.schedule(new TimeTick(this), 0, 1000);
	}
	public void updateTime(){
		DateFormat formatter = new SimpleDateFormat("HH.mm.ss / dd.MM.yyyy");
		setTime(formatter.format(new Date()));
	}

}

TimeTick-Klasse (benötigt von TimeModel)

import java.util.TimerTask;


public class TimeTick extends TimerTask
{
	private TimeModel model = new TimeModel();
	
	public void run() 
	{
		model.updateTime();
	}
	public TimeTick(TimeModel m)
	{
		this.model = m;
	}
}

Viewer-Klasse

import java.awt.BorderLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;

public class TimeView extends JFrame{

	/**
	 * 
	 */
	private static final long serialVersionUID = -2910645591812558004L;
	private JLabel label = new JLabel("Klicken Sie auf Start");
	private JButton button = new JButton("Start / Stop");

	
	public TimeView(){
		super();
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setBounds(200, 200, 300, 150);
		this.setVisible(true);
		this.getContentPane().add(label, BorderLayout.CENTER);
		this.getContentPane().add(button, BorderLayout.SOUTH);
		label.setHorizontalAlignment(JLabel.CENTER);
	}
	public void setActionListener(TimeController ctrl){
		button.addActionListener(ctrl);
	}
	public void unsetActionListener(TimeController ctrl){
		button.removeActionListener(ctrl);
	}
	public void setOutput(String s){
		label.setText(s);
	}
}

Controller-Klasse

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Observable;
import java.util.Observer;


public class TimeController implements ActionListener, Observer 
{
	private TimeView tview;
	private TimeModel tmodel;
	
	public TimeController(TimeModel model, TimeView view){
		
		this.tmodel=model;
		this.tview=view;
		
		view.setActionListener(this);		
	}

	public void actionPerformed(ActionEvent arg0) {
		if (tmodel.countObservers()==1) 
			tmodel.deleteObservers();
		else 
			tmodel.addObserver(this);
	}

	public void update(Observable arg0, Object arg1) {
		tview.setOutput(tmodel.getTime());
	}
}

 

[HLX]

Gefällt mir auf den ersten Blick sehr gut. Schöne MVC Implementierung.

Eine kleinigkeit würde ich noch verändern:

public void setActionListener(ActionListener al){ 
      button.addActionListener(al); 
   } 
   public void unsetActionListener(ActionListener al){ 
      button.removeActionListener(al); 
   }

Damit bist du flexibler.

 

[razor]

Gefällt mir auf den ersten Blick sehr gut. Schöne MVC Implementierung.

Eine kleinigkeit würde ich noch verändern:

public void setActionListener(ActionListener al){ 
      button.addActionListener(al); 
   } 
   public void unsetActionListener(ActionListener al){ 
      button.removeActionListener(al); 
   }

Damit bist du flexibler.

Vielen Dank!

Die Änderung macht es natürlich deutlich flexibler.