Implementierung nach MVC

[Dutch_OnE]

Hallo,

ich muss eine Hausarbeit nach MVC Design durchführen und bin mir nicht sicher, wo ich welche Methoden hineinschreiben muss.

Vorweg habe ich 3 Pakete (model, controller, view) erstellt.

In meinem View Paket ist eine Klasse, die den Startbildschirm der Anwendung darstellt.

Im Paket model habe ich eine Klasse Util die Methoden beinhaltet, um bestimmte Datenbankabfragen durchzuführen.

Nun habe ich auf der Startmaske eine Checkbox eingefügt, mit der ich auf diese Klasse Uitl direkt zugreife.

Nach meinem Dafürhalten verletze ich doch damit das MCV Prinzip, da ich direkt von der View aufs Model zugreife oder ?

Ich bin der Meinung das die View auf den Controller zugreift, der dann auch die DB Zugriff hat.

Wie muss denn nun meine Methode im Controller aussehen ?

So sieht meine Methode im Model aus:

	public static Vector<Integer> getAlleAngebotsNr() throws Exception {
		Vector<Integer> angbt = new Vector<Integer>();
		OracleDB db = getOracleDB();
		ResultSet rs1 = db.doQuery("select * from Artikel");
		while (rs1.next()){
			angbt.add(rs1.getInt("artikelnr"));
		}
		rs1.close();
		return angbt;
	}

Und so der bisherige Aufruf aus der View:

JComboBox cmbBoxA = new JComboBox (Util.getAlleAngebotsNr());

Wie gesagt, direkter Zugriff ist halt nicht so toll, aber wie kann man das besser machen?

gruß
dutch

 

[lohr]

Für mich gehört sowas:

JComboBox cmbBoxA = new JComboBox (Util.getAlleAngebotsNr());

Ganz klar in den Controller...Um dir jedoch weiterzuhelfen wären nen paar mehr Informationen notwendig...

PS: Doppelthreads aufzumachen steigert die unbeliebtheit ^^

 

[moormaster]

Für mich gehört sowas:

JComboBox cmbBoxA = new JComboBox (Util.getAlleAngebotsNr());

Ganz klar in den Controller...Um dir jedoch weiterzuhelfen wären nen paar mehr Informationen notwendig...

PS: Doppelthreads aufzumachen steigert die unbeliebtheit ^^

Ich würde für das View eher eine Schnittstelle definieren, wo eine Methode existiert, die nur diese AngebotsNr-Daten entgegen nimmt.
Ob das ganze dann in eine JComboBox kommt oder nach stdout ausgegeben wird ist Sache des Views und geht den Controller gar nix an.

 

[lohr]

Wieso mit diesem Befehl:

new JComboBox (Util.getAlleAngebotsNr());

Geht es um das erstellen eines Objektes welches zum einlesen verwendet wird und nicht zum ausgeben von Information...ist natürlich nur meine ansicht und kann auch falsch sein ^^
das mit dem interface ist natürlich richtig, ich würde ein interface in beide richtung erstellen mit get und set methode und dann jeweils diese in den veschiedenen klassen implementieren somit erhält man eine schöne Architektur.

Model(Interface)---------Controller(Klasse)-----------View(Interface)

 

[Dutch_OnE]

Also das mit dem Doppenthread war keine Absicht.

Was ich eigentlich möchte ist folgendes:

Ich benötige die Checkbox um z.B. alle vorhandenen Angebote aus der Datenbank auszulesen.
Diese Daten sollen allerdings NICHT geändert werden.

Nun ist meine Frage, ob ich der Checkbox direkt als Instanzierungsparameter einen Zugriff auf die oben beschriebene Methode gebe, oder ob ich eine neue Methode im Controller machen muss, die mir das gleiche zurückgibt, was die Methode aus der Uitl eh schon tut ?

Meine Idee ist, wenn ich zum Beispiel im View mir Daten anzeige, die ich erst aufbereiten, zusammenfassen oder verändern möchte, gehör dieses in eine Klasse/Methode des Controllers.

Aber was benötige ich für meine oben beschriebenen Fall ?-

 

[lohr]

Hier mal ein kleines Beispiel dazu:

public class Controller
{ 

	private Read r;
	
	public Controller()
	{
		r = new readConsole() // Hier wird erst bestimmt von wo du lesen willst und nur noch hier geändert
		System.out.println(r.readLine());
	}
	
	public static void main(String[] args) 
	{  
		Controller c = new Controller();
	}

}

public interface Read 
{

	public String readLine();

}

public class ReadConsole implements Read 
{

	private String line = "dies ist ein test";
	
	public String readLine()
	{
		return line;
	}

}

Du könntest nun noch eine Klasse schreiben die das einlesen von nem JText... erlaubt von sonst irgendetwas und diese müssten nur das Interface Read implentieren und dadurch müsstest nur an der kommentierten Stelle eine andere klasse instanziieren und sonst nichts am quellcode ändern (außer halt eine neue klasse schreiben).

 

[Dutch_OnE]

So ganz komme ich mit der Sache noch nicht zurecht. Bisher sieht mein Aufbau wie folgt auf.

Pakete -
view - Klassen und Listener für die GUI Ausgabe
model - Verbindung zur Datenbank, Util Klasse mit verschiedenen ResultSets
controller - leer

In der GUI ist eine Methode in einer Klasse

                JComboBox cmbBoxA = new JComboBox (Util.getAlleAngebotsNr())

mit der ich direkt auf die Datenbank zugreife und mir die Daten hole.

Anbei nochmal die Methode aus der Util Klasse (model Paket):

	public static Vector<Integer> getAlleAngebotsNr() throws Exception {
		Vector<Integer> angbt = new Vector<Integer>();
		OracleDB db = getOracleDB();
		ResultSet rs1 = db.doQuery("select * from Artikel");
		while (rs1.next()){
			angbt.add(rs1.getInt("artikelnr"));
		}
		rs1.close();
		return angbt;
	}

Meine 2 Fragen:

1) Kann ich die Klasse Util nicht in die controller Schicht packen ?
2) Falls die Klasse Util definitiv zum model gehört, muss ich dann im controller eine Klasse / Methode erstellen, die auf den Rückgabetyp aus Util zugreift und ihn dann nochmal zusätzlich im view zur Verfügung stellt ?