Hilfe! Command Pattern an easy ImageViewer?

[geneticZ]

Hallo allerseits,
ich lerne und übe derzeit "Design Patterns" und habe mir bezüglich dessen die Aufgabe gestellt, das Command Pattern an einem einfachen ImageViewer auszuprobieren. Hierzu entstehen bei mir ein paar Fragen die ich hoffe mit Hilfe des Forums klären zu können.

Das ganze habe ich auch schon gecoded und es funktioniert auch, ich bin mir aber nicht sicher ob mein Lösungsansatz eher umständlich, richtig oder gar falsch ist. Derzeit beschrenke ich mich ausschließlich auf den OpenFile - Befehl als JMenuItem...

Es gibt ja, bitte verbessert mich wenn ich da falsch liege, mehrere Möglichkeiten das Commandpattern in Swing und Events etc einzusetzen: Als simplen ActionListenerEvent, mittels AbstractAction und dann eben so wie ich das jetzt probiert habe über ein Command Interface. Das Command Interface habe ich auch wegen einer Undo-Methode verwendet, die ich aber erst später einbauen will.

Ich würde mich freuen wenn jemand meinen, dann doch sehr langen Post, lesen, wenn nötig verbessern oder mir ein paar Tips geben könnte. Vielen Dank schonmal für eure Mühe.
Gruß
geneticZ

//Das Command Interface
public interface Command {
	public void execute();
}

//Mein ConcreteCommand
public class OpenImageCmd implements Command{
	private ImageComponent viewComponent;
	
	public OpenImageCmd(ImageComponent im){
		this.viewComponent = im;
	}
	
	public void execute() {
		JFileChooser fc = new JFileChooser(System.getenv("User"));
		fc.setFileFilter(new FileNameExtensionFilter("*.jpg;*.gif", "jpg", "gif"));
		fc.showOpenDialog(viewComponent);
		final File file = fc.getSelectedFile();
		
		if(file != null){
			new SwingWorker<BufferedImage, Void>(){
				@Override
				protected BufferedImage doInBackground() throws IOException {
					return ImageIO.read(file);
				}
				@Override
				protected void done(){
					try{
						viewComponent.setImage(get());
					}
					catch(Exception e){}
				}
			}.execute();
		}
	}
}

//Mein Invoker
public class OpenImage extends JMenuItem implements ActionListener{
	Command cmd;
	
	public OpenImage(Command c){
		super("Open");
		this.cmd = c;
		this.addActionListener(this);
	}
	
	public void actionPerformed(ActionEvent e) {
		cmd.execute();
	}
}

//Mein Client
import java.awt.event.*;
import javax.swing.*;

public class win extends JFrame{
	JMenuBar menuBar;
	JMenu menu;
	JMenuItem menuItem;
	JPanel contentPane;
	
	Command openImage;
	
	public win(){
		contentPane = (JPanel) getContentPane();
		contentPane.setLayout(new BoxLayout(contentPane, BoxLayout.Y_AXIS));
		
		ImageComponent im = new ImageComponent();
		Command openImage = new OpenImageCmd(im);
				
		menuBar = new JMenuBar();
		
		menu = new JMenu("File");
		menuBar.add(menu);
		
		menuItem = new OpenImage(openImage);
		menu.add(menuItem);
		
		setJMenuBar(menuBar);
		
		contentPane.add(new JScrollPane(im));
		
	}
	
	public static void main(String[] args) {
		JFrame frame = new win();
		frame.addWindowListener(new WindowAdapter() {
			public void windowClosing(WindowEvent e) {
				System.exit(0);				
			}
		});
		frame.pack();
		frame.setVisible(true);
	}
}

//Mehr oder weniger mein Reciever bzw die Image Paint Methode
public class ImageComponent extends JComponent {   
	private BufferedImage image; 
	
	public void setImage( BufferedImage image ) {     
		this.image = image;  
		
		setPreferredSize( new Dimension(image.getWidth(), image.getHeight()));
		repaint();    
		invalidate();   
	}   
	
	@Override   
	protected void paintComponent( Graphics g ){
		
		if ( image != null )
			g.drawImage( image,
					((getWidth()/2) - (image.getWidth()/2)),
					((getHeight()/2) - (image.getHeight()/2)),
					this); 			
	} 
}

 

[Gast2]

Jop ich glaub das passt so...
Jetzt kannst du noch Methoden für undo/redo einführen..

Ich würde den FileChooser nur einmal im Kontruktor erzeugen nicht jedesmal neu.
Dann springt der FileChooser auch dort hin wo der User ihn zuletzt geöffnet hat.

 

[geneticZ]

Hey super! Danke dass du mal drüber gesehen hast!
Das mit dem FileChooser bau ich gleich um...
Kannst du mir sagen warum die JScrollPane im contentPane erst nach einmaligen klick auf's Bild aufpopt?

 

[Gast2]

2 Sachen ab Java 1.5 brauchst du die Componenten nicht mehr auf getContentPane adden es reicht ein normales add...
Der WindowListener ist auch überflüssig es reicht die Methode setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE)...
Debug mal durch eventuell stimmt deine PreffredSize nicht...

Des weiteren würde ich so zeichnen

protected void paintComponent(Graphics g) {
  super.paintComponent(g);
  final Insets insets = getInsets();
  final int x = insets.left;
  final int y = insets.top;
  final int width = getWidth() - x - insets.right;
  final int height = getHeight() - y - insets.bottom;
  g.drawImage(img, x, y, width, height, this);
}

 

[geneticZ]

HI,
was für Vorteile bringt mir das ein wenn ich das Bild so zeichne?
Ok, ich sehe gerade dass das mein Bild nicht zentriert sondern so groß zeichnet wie der Container ist.
Das will ich aber gar nicht unbedingt...

Das ScrollPane Problem hab ich leider immernoch. Aus irgend nem Grund wird setPrefferedSize erst aufgerufen wenn ich auf den Rahmen meines Fensters klicke.

 

[Gast2]

Ja kannst ja trotzdem das /2 einmachen dann ist es wieder zentriert...

Ja ist logisch wenn du auf dein Fenster klickt macht er ein repiant vom frame...
wenn du setImage machst musst du dein JFrame repainten weil dieser regelt die Größe des ImageComponent nicht repaint auf ImageComponent aufrufen...


Könntest eventuell gleich das nächste Pattern lernen ...
ObserverPattern...
das JFrame registriert sich als Listener an deinem Panel und das Pnael schmeißt bei setImage ein Event worauf sich das JFrame repainted...

 

[geneticZ]

Könntest eventuell gleich das nächste Pattern lernen ...
ObserverPattern...

Klingt jetzt vielleicht blöd, aber genau das hatte ich vor...
Observer Pattern wäre mein nächstes gewesen.
Schonmal vielen Dank! Du bist mir echt ne große Hilfe!

Edit: Ok, hier stoß ich jetzt auf ein Problem. Ich kapier absolut nicht wie ich das umsetzen kann?

Gruß
geneticZ

 

[geneticZ]

Muss ich dann erstmal den Panel abkapseln?
[Java]
public class ObsPane extends JPanel{
Observable notifier = new ObsBox();
}
[/Java]
[Java]
public class ObsBox extends Observable {
public void notifyObservers(Object b) {
setChanged();
super.notifyObservers(b);
}
}
[/Java]
Oder wie soll das laufen?

 

[Gast2]

Muss ich dann erstmal den Panel abkapseln?
[Java]
public class ObsPane extends JPanel{
Observable notifier = new ObsBox();
}
[/Java]
[Java]
public class ObsBox extends Observable {
public void notifyObservers(Object b) {
setChanged();
super.notifyObservers(b);
}
}
[/Java]
Oder wie soll das laufen?

Jop...
aber das setChanged musst du in deiner setImage Methode aufrufen

Später kannst du dir auch eigene Listener schreiben

 

[geneticZ]

Sorry ich checks nicht.
Wie bekomme ich die setChanged() denn in die setImage Methode?
.... und wie add ich die JFrame - Klasse als Observer?
Ich blick überhaupt nicht durch

 

[Gast2]

Mal ein Denkansatz ungefähr

public class ObsPane extends JPanel{
Observable notifier = new ObsBox();

void setImage(Image image){
this.image = image;
notifier .setChanged();
notifier.notify();
}
}

beim Frame machst implements Obsverable

und machst macht panel.addObserver

 

[geneticZ]

Also ich komm hier leider nicht weiter...

Ich sehe momentan zwei Möglichkeiten wie ich das Observer Pattern implementieren kann, leider hänge ich immer an einer Stelle.
Zum einen könnte ich quasi in der Main-Methode als erstes ein Objekt von der Observable Klasse erstellen welche dann über this.addObserver(new win()); die JFrame Klasse als Observer added.
[Java]
public class ObsBox extends Observable {

public ObsBox(){
this.addObserver(new win());
}

public synchronized void setChanged() {
super.setChanged();
}

public void notifyObserver(){
super.notify();
}

public static void main(String[] args){
new ObsBox();
}
}
[/code]
Da komm ich dann aber mit der ImageComponent-Methode nicht klar, weil dort ja auch ein "neues" Objekt von der Observable Klasse erstellt werden muss?

public class ImageComponent extends JComponent{   
	private BufferedImage image;
	ObsBox notifier = new ObsBox();
	
	public void setImage( BufferedImage image ) {     
		this.image = image;  
		setPreferredSize( new Dimension(image.getWidth(), image.getHeight()));
		notifier.setChanged();
		notifier.notifyObserver();
	}   
	
	protected void paintComponent( Graphics g ){
		
		if ( image != null )
			g.drawImage( image,
					((getWidth()/2) - (image.getWidth()/2)),
					((getHeight()/2) - (image.getHeight()/2)),
					this); 			
	} 
}

Die andere Möglichkeit wäre über die Main als erstes das JFrame Objekt zu erstellen und dann eben wie weiter oben die Observable zu erstellen. Dann check ich aber nicht wie ich das JFrame Window als Observer adden kann, weil da ja kein Observable Objekt existiert? Ich hoffe mein Verständnisproblem wird etwas klar.
Danke für die Hilfe
geneticZ

 

[Gast2]

Klasse Frame nur die wichtigsten Sachen

public class ObserverTest extends JFrame implements Observer{
		
		private MyPanel myPanel;
	
	
		public ObserverTest() {
			myPanel = new myPanel();
			myPanel.addObserver(this);
		}

		@Override
		public void update(Observable arg0, Object arg1) {
			// TODO Auto-generated method stub

		}

}

Panel Klasse

public class ObsPane extends JPanel{
private Observable notifier = new Observable ();
 
public void setImage(Image image){
 this.image = image;
 notifier .setChanged();
 notifier.notify();
 }

public void addObserver(Observer obs){
notifier.addObserver();
}
}

 

[geneticZ]

ah! Es scheint zu funktionieren! Danke!
Die Scrollpane wird leider trotzdem noch nicht aufgerufen, bzw weiterhin erst nach klick auf Frame.
Hier mein Frame Code:
[Java]
import java.awt.Dimension;
import java.awt.event.*;
import java.awt.image.BufferedImage;
import java.util.Observable;
import java.util.Observer;

import javax.swing.*;

public class win extends JFrame implements Observer{
JMenuBar menuBar;
JMenu menu;
JMenuItem menuItem;
JPanel contentPane;

ObsBox obs;
Command openImage;

public win(){
setTitle("ImageViewer");
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setSize(500, 300);


contentPane = (JPanel) getContentPane();
contentPane.setLayout(new BoxLayout(contentPane, BoxLayout.Y_AXIS));

ImageComponent im = new ImageComponent();
im.addObserver(this);

Command openImage = new OpenImageCmd(im);
JScrollPane pane = new JScrollPane(im);

menuBar = new JMenuBar();

menu = new JMenu("File");
menuBar.add(menu);

menuItem = new OpenImage(openImage);
menu.add(menuItem);

setJMenuBar(menuBar);

contentPane.add(pane);

setVisible(true);
}

public void update(Observable o, Object arg) {
repaint();
}
}
[/code]