OOP Wieder mal Zugriffsprobleme... (Lösung am Ende)

[Firephoenix]

Folgender Programmaufbau:

2 Klassen:
Fenster und Aufzug

fenster erbt von Frame und macht folgendes:

	public static void main(String[] args) 
	{
		Aufzug aufzug1 = new Aufzug(0, true, 0);
		Fenster window = new Fenster(aufzug1);
		aufzug1.setFenster(window);
		window.setVisible(true);
		
	}

Erzeugt eine Instanz von Aufzug, erstellt ein Fenster und übergibt aufzug1 (warum, dazu komme ich gleich), und übergibt das erstellte Fenster an das erstellte Aufzugobjekt.

Mein Problem ist folgendes: ich arbeite gerne Modular, und ich kriege die Krise wenn ich alles in eine Datei schreiben muss.

Folgende Aufgabe soll das Programm leisten:

Ein einfaches Fenster erstellen (Buttons und Labels - dieser werden im Konstruktor erzeugt)
Einen Aufzug erstellen (kann hoch und runterfahren etc).

Der Aufzug soll über Fenster gesteuert werden (daher wird Aufzug übergeben, in Aufzug ist das Interface ActionListener implementiert und eine entsprechende Methode um eingaben der Buttons abzufangen)

Soweit einfach.

Jetzt geht es aber rückwärts: der Aufzug soll auch im Fenster änderungen vornehmen können.

Dazu wird das erzeugte Fenster an das erzeugte Aufzugobjekt übergeben.

Nun kann ich von Aufzug aus zwar auf das Fenster zugreifen (Hintergrundfarbe etc), aber keine Elemente manipulieren.

Als Beispiel: Der Text eines Labels ltuerstatusausgabe soll geändert werden.
In meiner Klasse Aufzug sieht das so aus (in Aufzug heißt das Fenster window, wird über eine setmethode eingegeben)

	fenster.ltuerstatusausgabe.setText(getTuerstatus());

leider erkennt aufzug das Label nicht.

Die einzige Lösung die mir jetzt einfallen würde wäre alle Elemente in Fenster die ich mit Aufzug manipulieren will als Variablen über Setmethoden an Aufzug zu übergeben, dort zu ändern und zurückzuschieben, was sehr hässlich wäre.

Fällt jemandem dazu ein besserer Ansatz an, wie ich aus meiner Klasse Aufzug heraus Elemente in Fenster ändern kann?

 

[w0ddes]

Nicht wirklich besser, aber problemlösend:

Die zu ändernden Komponenten

 public static

machen ..

 

[function]

du könntest auch in der Fensterklasse getter/setter methoden für die komponenten anbieten...

 

[@x.l]

Überprüfe mal die Sichtbarkeit deiner Felder. (obwohl es kein guter Stil ist Member public zu machen)

 

[Firephoenix]

du könntest auch in der Fensterklasse getter/setter methoden für die komponenten anbieten...

Bingo ^^ war zu sehr auf den Konstruktor fixiert. Sekunde wird gleich getestet (elende Stringumwandlungen :rtfm

 

[Michael...]

Das ist ein klassisches Modellierungsproblem. Mögliche Lösung MVC
Fenster ist die View, Aufzug das Model, dann brauchst Du nur noch einen Controler

 

[w0ddes]

Hm. Da hab ich wohl was falsch verstanden Dachte er will keine getter/setter .. ^^

 

[ARadauer]

Nicht wirklich besser, aber problemlösend:

Die zu ändernden Komponenten

 public static

machen ..

auf keinen Fall! :wuerg:

Jetzt geht es aber rückwärts: der Aufzug soll auch im Fenster änderungen vornehmen können.

.... ich finde der Aufzug sollte das Fenster nicht kennen. Ich würde eher einen zentralen Controller machen der Fenster und Aufzüge kennt und der aufgrund von Aktionen des Aufzugs das Fenster manipulieren kann.

 

[w0ddes]

auf keinen Fall! :wuerg:


.... .

Deswegen steht obendrüber: "Nicht wirklich besser" .. aber hatte das Problem ja falsch aufgefasst

 

[Firephoenix]

An so einem Controller saß ich schon - kläglich versagt.

Hatte da
Fenster
FensterAktionen
und Aufzug

FensterAktionen sollte die Form des Controllers übernehmen und zwischen den Klassen vermitteln.

Also FensterAktionen als Listener an Fenster und die Objekte dort übergeben,das erzeugte Fenster als variable in FensterAktionen damit FensterAktionen dort Änderungen vornehmen kann - hat auch geklappt

bis der Aufzug kam... konnte zwar Aufzug von FensterAktionen aus steuern (war über setter auch kein Ding) nur Rückwärts *grusel* hats nicht geklappt, hatte ja schon FensterAktionen übergeben... wobei wärend ich gerade schreibe fällt mir glaube ich die Lösung ein, an der ich 2 Tage lang rumüberlegt habe

Fenster erzeugen
Controller erzeugen
Aufzug erzeugen

Controller als Listener an Fenster und Elemente hängen
Fenster als Variable in Controller
Aufzug über Setter aus Controller steuern
Fenster über Setter aus Controller steuern die Aufzug ansprechen kann...

Hm muss mal einiges umschreiben, dann melde ich mich wieder ^^ danke aber auf alle Fälle schonmal für die vielen Antworten

/EDIT:
ihr könnt mir aber trotzdem nochmal helfen, ist aber eher simpel.

ich erzeuge in Fenster im Kontruktor ein Label "ltuerstatusausgabe"

			Label ltuerstatusausgabe = new Label("Türstatusausgabe");
			ltuerstatusausgabe.setBounds(300, 180, 150, 60);
			add(ltuerstatusausgabe);

Und in einer Setmethode

	public void setTuerstatus(String text)
	{
		ltuerstatusausgabe.setText(text);
	}

in der Setmethode gibt mir Eclipse aber aus: ltuerstatusausgabe cannot be resolved - warum kann ich nicht darauf zugreifen?

Lg,
Firephoenix

 

[Michael...]

in der Setmethode gibt mir Eclipse aber aus: ltuerstatusausgabe cannot be resolved - warum kann ich nicht darauf zugreifen?

Weil Du ltuerstatusausgabe als lokale Variable im Konstruktor definiert hast. Mach eine Instanzvariable draus.

 

[Firephoenix]

So Controllerstruktur steht und Aufzug fährt *freu*
jetzt kann ich ihn so nach und nach ausbauen
Hat vom lernen her einiges gebracht mein Projekt, endlich wieder halbwegs in OOP drinne

todo:
-Grafikausgabe dazu
-Layout (Grid) zur Verwaltung von Fenster
-mehrere Knöpfe drückbar, die der Aufzug abarbeitet
-Außenknöpfe der Stockwerke um Aufzug zu rufen
-Aufzug als eigener Thread damit Fenster nicht hängt (wichtig)
-Knopf um Türen zu öffnen
-Codeoptimierung

Hier mal der Quellcode, wird natürlich noch optimiert und kommentiert, als Anfänger in Java ist das so eh noch nicht optimal

Klasse Fenster:

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

public class Fenster
extends Frame
{

	private static final long serialVersionUID = 1L;
	
	private Label ltuerstatusausgabe = new Label("Türstatusausgabe");
	private Label ltuerstatus = new Label("Tuerstatus");
	private Label lstockwerkausgabe = new Label("Stockwerkausgabe");
	private Label lstockwerk = new Label("Aktuelles Stockwerk:");
	private Label lstatusausgabe = new Label("Stockwerkausgabe");
	private Label lstatus = new Label("Aktueller Status:");
	
	public static void main(String[] args) 
	{

		
		Controller ca = new Controller();
		Aufzug aufzug1 = new Aufzug(0, true, 0); //Instanz fa zum Abfangen von FensterAktionen
		ca.setAufzug(aufzug1);
		aufzug1.setController(ca);
		Fenster window = new Fenster(ca); //Anzeigefenster window
		ca.setFenster(window);
		window.setVisible(true); //Fenster window anzeigen
		


		
		
	}

	
	public void setStatus(String status)
	{
		lstatusausgabe.setText(status);
	}
	
	public void setStockwerk(String stockwerk)
	{
		lstockwerkausgabe.setText(stockwerk);
	}
	
	public void setTuerStatus(String tstatus)
	{
		ltuerstatusausgabe.setText(tstatus);
	}
	
	public Fenster(Controller ca) //erstellt das Fenster
	{
		super("Aufzug"); //Fenstertitel
		addWindowListener(new WindowClosingAdapter(true)); //WCA zum Schließen
		setBackground(Color.ORANGE); //Hintergrund Orange
		setSize(500,600); // Größeneinstellung
		setLayout(null); //Layout null um
		
			//Buttons erstellen
			Button b0 = new Button("Erdgeschoss");
			b0.setBounds(20, 60, 100, 60);
			b0.addActionListener(ca);
			add(b0);		
			
			Button b1 = new Button("Stockwerk 1");
			b1.setBounds(20, 120, 100, 60);
			b1.addActionListener(ca);
			add(b1);

			Button b2 = new Button("Stockwerk 2");
			b2.setBounds(20, 180, 100, 60);
			b2.addActionListener(ca);
			add(b2);
			
			Button b3 = new Button("Stockwerk 3");
			b3.setBounds(20, 240, 100, 60);
			b3.addActionListener(ca);
			add(b3);
			
			Button b4 = new Button("Stockwerk 4");
			b4.setBounds(20, 300, 100, 60);
			b4.addActionListener(ca);
			add(b4);
			
			Button b5 = new Button("Stockwerk 5");
			b5.setBounds(20, 360, 100, 60);
			b5.addActionListener(ca);
			add(b5);
			//Buttons erstellen Ende
			
			//Labels erstellen
			
			lstatus.setBounds(150, 60, 150, 60);
			add(lstatus);
			
			
			lstatusausgabe.setBounds(300, 60, 150, 60);
			lstatusausgabe.setText(ca.getAufzugStatus());
			add(lstatusausgabe);

			
			lstockwerk.setBounds(150, 120, 150, 60);
			add(lstockwerk);
			
			
			lstockwerkausgabe.setBounds(300, 120, 150, 60);
			lstockwerkausgabe.setText(ca.getAufzugStockwerk());
			add(lstockwerkausgabe);
			

			ltuerstatus.setBounds(150, 180, 150, 60);
			add(ltuerstatus);
			
			
			ltuerstatusausgabe.setBounds(300, 180, 150, 60);
			add(ltuerstatusausgabe);
			ltuerstatusausgabe.setText(ca.getAufzugTuerstatus());
			//Labels ende
	}

	public void setTuerstatus(String text)
	{
		ltuerstatusausgabe.setText(text);
	}
	
}

Klasse Controller:

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


public class Controller
implements ActionListener{


	
	private Aufzug aufzug;
	private Fenster fenster;
	
	
	public void setAufzug(Aufzug aufzug)
	{
		this.aufzug = aufzug;
	}
	
	public void setFenster(Fenster fenster)
	{
		this.fenster = fenster;
	}
	
	
	
	
	//Verbindungen nach Fenster
	public void setFensterStatus(String status)
	{
		fenster.setStatus(status);
	}
	
	public void setFensterStockwerk(String stockwerk)
	{
		fenster.setStockwerk(stockwerk);
	}
	
	public void setFensterTuerStatus(String tstatus)
	{
		fenster.setTuerstatus(tstatus);
	}
	//Verbinungen nach Fenster Ende
	
	//Verbindungen nach Aufzug
	public String getAufzugStatus()
	{
		return aufzug.getStatus();
	}
	
	public String getAufzugStockwerk()
	{
		return aufzug.getPosition();
	}
	
	public String getAufzugTuerstatus()
	{
		return aufzug.getTuerstatus();
	}
	
	//verbindungen nach Aufzug Ende
	
	public void actionPerformed(ActionEvent event)
	{
		String cmd = event.getActionCommand();
		if (cmd.equals("Erdgeschoss"))
		{
			
			aufzug.setZiel(0);
		}
			else
			{
				//nichts
			}
		if (cmd.equals("Stockwerk 1"))
		{
			
			aufzug.setZiel(1);
		}
			else
			{
				//nichts
			}
		if (cmd.equals("Stockwerk 2"))
		{
			
			aufzug.setZiel(2);
		}
			else
			{
				//nichts
			}
		if (cmd.equals("Stockwerk 3"))
		{
			
			aufzug.setZiel(3);
		}
			else
			{
				//nichts
			}
		if (cmd.equals("Stockwerk 4"))
		{
			
			aufzug.setZiel(4);
		}
			else
			{
				//nichts
			}
		if (cmd.equals("Stockwerk 5"))
		{	
			aufzug.setZiel(5);
		}
			else
			{
				//nichts
			}
	}
	
	
	
	
	
	
	
	
	
	
	
	
}

Klasse Aufzug

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


public class Aufzug 
{
	
	private Controller ca;
	
	private int position;
	private int ziel;
	private int status; //0 = stehen 1 = hoch 2 = runter 3 = Türen öffnen 4 = Türen Schließen
	private String statusausgabe;
	private boolean tueroffen;
	
	public Aufzug (int position, boolean tuerOffen, int status)
	{
	this.position = position;
	this.tueroffen = tuerOffen;
	this.status = status;
	}
	
	public Aufzug ()
	{
	this.position = 0;
	this.tueroffen = true;
	this.status = 0;
	}
	
	public void setController(Controller ca)
	{
		this.ca = ca;
	}

	public String getTuerstatus()
	{
		if(tueroffen == true)
			statusausgabe = new String ("Offen");
		else
		;
		if(tueroffen == false)
			statusausgabe = new String ("Geschlossen");
		else
		;
		return statusausgabe;
			
	}

	public String getStatus()
	

	{
		if(this.status == 0)
			statusausgabe = new String("steht");
		else
		;
		if(this.status == 1)
			statusausgabe = new String("fährt hoch");
		else
		;
		if(this.status == 2)
			statusausgabe = new String("fährt runter");
		else
		;
		if(this.status == 3)
			statusausgabe = new String("Türen Öffnen");
		else
		;
		if(this.status == 4)
			statusausgabe = new String("Türen Schließen");
		else
		;
		return statusausgabe;
		
	}
	
	public String getPosition ()
	{
		statusausgabe = String.valueOf(position);
		return statusausgabe;
	}
	
	

	public void setZiel(int Ziel) //Knopf drücken
	{
		this.ziel = Ziel;
		if (this.ziel == this.position)
		{
			tueroeffnen(0);
		}
		else
			;
		if (this.ziel < this.position)
		{
			runterfahren();
		}
		else
			;
		if (this.ziel > this.position)
		{
				hochfahren();
		}
		else
			;
	}
	
	private void hochfahren()
	{
		tuerschliessen(1);
		status = 1; //hochfahren	
		ca.setFensterStatus(getStatus());
		do
		{
			ca.setFensterStockwerk(getPosition());
			position++;
			ca.setFensterStockwerk(getPosition());
			try 
			{
				Thread.sleep(2000);
			} 
			catch(InterruptedException e) 
			{
			}
		}
		while(this.position != this.ziel);
		status = 0; //stehenbleiben
		ca.setFensterStatus(getStatus());
		tueroeffnen(0);
	}
	
	private void runterfahren()
	{
		tuerschliessen(2);
		status = 2; //runterfahren
		ca.setFensterStatus(getStatus());
		do
		{
			ca.setFensterStockwerk(getPosition());
			position--;
			ca.setFensterStockwerk(getPosition());
			try 
			{
				Thread.sleep(2000);
			} 
			catch(InterruptedException e) 
			{
			;
			}
		}
		while(this.position != this.ziel);
		status = 0; //stehenbleiben
		ca.setFensterStatus(getStatus());
		tueroeffnen(0);
	}
	
	public void tueroeffnen(int folgeStatus)
	{
	if (tueroffen == true)
		//nichts
		;
		else
			ca.setFensterTuerStatus(getTuerstatus());
			status = 3; //Türen Öffnen
			ca.setFensterStatus(getStatus());
			try 
			{
				Thread.sleep(2000); //Pause beim Öffnen
			} 
			catch(InterruptedException e) 
			{
			;
			}
			tueroffen = true;
			status = folgeStatus;
			ca.setFensterTuerStatus(getTuerstatus());
			ca.setFensterStatus(getStatus());
			try 
			{
				Thread.sleep(1000); //Pause nach Öffnen
			} 
			catch(InterruptedException e) 
			{
			;
			}
	}
	
	public void tuerschliessen(int folgeStatus)
	{
	if (tueroffen == false)
		//Fehlt noch
		;
		else
			ca.setFensterTuerStatus(getTuerstatus());
			status = 4;
			ca.setFensterStatus(getStatus());
			try 
			{
				Thread.sleep(2000); //Pause beim Schließen
			} 
			catch(InterruptedException e) 
			{
			;
			}
			tueroffen = false;
			status = folgeStatus;
			ca.setFensterTuerStatus(getTuerstatus());
			ca.setFensterStatus(getStatus());
			try 
			{
				Thread.sleep(1000); //Pause nach Schließen
			} 
			catch(InterruptedException e) 
			{
			;
			}
	}
	

	
}

und hier der fertige Aufzug1.0 für die die testen wollen:

File-Upload.net - Aufzug.jar

Lg,
Firephoenix