Objektorientierte Programmierung in der Schule

Zeus schrieb:

Ich weiss nicht, was du hast, das Modell selbst ist doch in Ordnung?

Nein, es hat nichts mit sinnvoller Objektorientierung gemein.
Beispiel:
Du hast eine Procedure die ein Quadrat erwartet, du kannst ihr aber alles geben, ein Rechteck, ein Würfel oder ein Quader, weil das ja alles Quadrate sind...

Zeus

Ein Objekt kapselt seine Daten und seine Methoden.

d.h. das Objekt selbst muss sich selbst berechnen.

gibst ne Methode Fläche in Quadrat.

kann man sie Überlagern in Rechteck, Quader, Würfel (bzw sollte man auch).

Vererbung ist nicht dazu da um Coderedudanzen zu vermeiden.

Wenn du B von A erben läßt, dann ist B ein A. Ein Quader ist kein Quadrat

Zeus

Vererbung ist nicht dazu da um Coderedudanzen zu vermeiden.

Hab ich das gesagt?

Ja hast du, du hast gesagt die Art der Vererbung wäre ok, weil sie ja alle die Flächenberechnung gemein haben.

Grad auch dadurch das erste Problem aufgetreten.
Das Quadrat hatte noch eine Prozedur setArea, man konnt ja von der Fläche auf die Seite zurückschließen. Beim Rechteck gehts nun nicht mehr. Aber was soll man nun mit der virtuellen Prozedur anfangen? Da sieht man schon, dass das absolut nciht sinnvoll ist.

Zeus

Leider kann ich fast kein Stück Delphi!

class Quadrat{
	int a;

	Quadrat(int a){
		this.a = a;
	}

	int flache(){
		return a*a;
	}
}

class Wuerfel extends Quadrat{
	Wuerfel(int a){
		super(a);
	}

	int flache(){
		return 6*a*a;
	}
}

class Rechteck extends Quadrat{
	int b;

	Rechteck(int a, int b){
		super(a);
		this.b = b;
	}

	int flache(){
		return a*b;
	}
}

class Quader extends Rechteck{
	int c;

	Quader(int a, int b, int c){
		super(a, b);
		this.c = c;
	}

	int flache(){
		return 2*a*b+2*a*c+2*b*c;
	}

}

public class vererbung {
	public static void main(String[] args) {
		System.out.println(new Quadrat(2).flache());
		System.out.println(new Wuerfel(2).flache());
		System.out.println(new Rechteck(2,3).flache());
		System.out.println(new Quader(2,3,4).flache());
	}
}

@Zeus: *LOL* berechne mal die Fläche von 'nem Würfel. Erst denken, dann schreiben!

CGI-BIN

Warum soll man nicht die (Ober-)Fläche von nem Würfel berechnen können, du Held?

Zeus

ich hab den fehler schon berichtigt das stand a*a*a anstelle von 6*a*a
und da Volumen != Fläche ist - naja ich darf wohl auch paar Bugs Programmieren.

HumeSikkins

Wenn jemand mal einen Link zum Thema Vererbung hat der ihm nahe bringt, das Vererbung eine ist ein Beziehung ist, dann her damit...

Also das sollte in *jedem* Buch über OOD/OOP drinstehen.

Vielleicht hilft dir Barbara Liskovs "Substitution Principle". Schön erklärt von Uncle Bob: http://www.objectmentor.com/resources/articles/lsp.pdf

Bashar

Zeus, das verletzt das Liskov-Prinzip:
http://fara.cs.uni-potsdam.de/~kaufmann/?page=GenCppFaqs&faq=Liskov#Answ
http://www.objectmentor.com/resources/articles/lsp.pdf

Zeus

Vielleicht sollte ich schreiben, dass das Java Code ist !

Bashar

Wozu, das sieht man doch. Auch in Java gelten grundlegende OO-Prinzipien

Zeus

Erfüllt eine Funktion die einen Pointer oder eine Referenz auf eine Basisklasse erwartet eine bestimmte Aufgabe, so muss diese Funktion die selbe Aufgabe auch mit einem Objekt einer abgeleiteten Klasse erfüllen können.

Tut mein Code das nicht ? Also ich sehe die Verletzung des Prinzip nicht!

Walli

Dann schau doch bitte hin.

Bashar

Bei dir gilt schon die IST-EIN-Beziehung nicht. Ein Rechteck ist eben im Allgemeinen kein Quadrat. Wenn du also eine Methode hast, die mit Quadraten arbeitet und von typischen Quadrat-Eigenschaften ausgeht (z.B. dass es 4 Seiten hat, die alle gleich lang sind), wird sie kolossal durcheinanderkommen, wenn du ihr ein Rechteck oder gar einen Quader unterschiebst. Deine Vererbung lässt also keine Polymorphie zu.

Zeus

Dann frage ich mich Warum das hier klappt?

class Quadrat{
	int a;

	Quadrat(int a){
		this.a = a;
	}

	int flache(){
		return a*a;
	}
}

class Wuerfel extends Quadrat{
	Wuerfel(int a){
		super(a);
	}

	int flache(){
		return 6*a*a;
	}
}

class Rechteck extends Quadrat{
	int b;

	Rechteck(int a, int b){
		super(a);
		this.b = b;
	}

	int flache(){
		return a*b;
	}
}

class Quader extends Rechteck{
	int c;

	Quader(int a, int b, int c){
		super(a, b);
		this.c = c;
	}

	int flache(){
		return 2*a*b+2*a*c+2*b*c;
	}

}

public class vererbung {
	public static int summe(Quadrat[] q){
		int sum = 0;
		for(int i = 0; i < q.length; ++i)
		{
			sum += q[i].flache();
		}

		return sum; 
	}

	public static void main(String[] args) {
		Quadrat[] q = new Quadrat[4];

		q[0] = new Quadrat(2);
		q[1] = new Wuerfel(2);
		q[2] = new Rechteck(2,3);
		q[3] = new Quader(2,3,4);

		System.out.println(q[0].flache());
		System.out.println(q[1].flache());
		System.out.println(q[2].flache());
		System.out.println(q[3].flache());
		System.out.println(summe(q));
	}
}

Bashar

Weil deine Funktion nicht davon ausgeht ein Quadrat zu bekommen, sondern lediglich ein Ding, was eine Fläche hat. Eine dem angemessene Vererbungsstruktur dafür wär also, dass Quadrat, Rechteck etc. von der abstrakten Klasse/Interface Flaeche abgeleitet sind oder so.
Es sagt ja keiner, dass deine Struktur nie funktioniert. Nur dass sie etwas kommuniziert, was sie nicht leisten kann, und daher schlechter OO-Stil ist.