Aufgabe zur Vererbung

[stevarino]

Hallo

Ich hab diese Aufgabe
Ein Vektor ist im n-dim. Raum mit den kartesischen Koordinaten durch n
Gleitkommazahlen, etwa ein Array gegeben.
Erstellen Sie eine Klasse Vektor, welche das Arbeiten mit Vektoren ermöglicht:
Die Methoden (Funktionen) für die Addition void add(Vektor V), die Skalarmultiplikation
scalar(double d) und den Betrag double length() sind zu realisieren.
Erzeugen und Initialisieren Sie diese Klasse mit geeigneten Konstruktoren.
Erweitern Sie Ihre Klasse um die Funktionen boolean equals(Vektor V) zur Prüfung auf
gleichen Inhalt, toString() zur Ausgabe eines Vektors sowie Vektor clone() zum
duplizieren der Werte eines Vektors.

Hab es auch schon so weit fast fertig bis auf die toString, clone, equals methode Heißt das in der Angabe das ich die Klasse erweitere indem ich die Klasse Tupel vererbe und die 3 anderen Methoden hinzufüge?????

Hier mal mein code

class Tupel {

	double []a;

	public Tupel(double[] a)	{
  								this.a = a;
								}

	double length(Tupel v) 	{
					double sum=0;
					for(int i=0;i<=a.length-1;i++)	{
												sum = sum+(a[i]*a[i]);
												}
					return Math.sqrt(sum);
							}

	void add(Tupel v) 	{
						for(int i=0;i<=a.length-1;i++) 	{
													this.a[i]=a[i]+v.a[i];
													}

						}

	void multiply(double d) 	{
								for(int i=0;i<=a.length-1;i++)	{
															this.a[i]=a[i]*d;
															}
								}

	double scalarProduct(Tupel v) 	{
									double skalar=0;
									for(int i=0;i<=a.length-1;i++)	{
																skalar=skalar+a[i]*v.a[i];
																}
									return skalar;
									}
				}

class Tupel1 extends Tupel {
	
	public String toString() 	{
								String s="("+a[0];
								for(int i=1;i<=a.length-1;i++) {
																s =s+", "+a[i];
																}
								return s+")";
								}
				}
	public boolean equals(Tupel v)	{
									for(int i=0;i<=a.length-1;++) {
																	if (a[i]==b[i])	{
																					return true;
																					}
																	else 	{
																			return false;
																			}	
																  }
									}


	public Tupel clone(){
						Tupel clone = new Tupel();
    					clone.a = a;
    					return clone;

						}
		   					}
public class Vector{

	public static void main(String args[]) {
											double []a={1,2,3,4};
											Tupel v = new Tupel(a);
											v.add(new Tupel(a));
											Out.println("v = " +v.toString());
											double []b={3,3,3,3};
											Out.println("scalar product = " + v.scalarProduct(new Tupel(b)));
											}
					}

Ich weiß das ihr keine Aufgaben lösen wollt aber ich bitte euch trotzdem meine Fehler im Program zu verbessern

Danke

lg Stevo

 

[Beni]

Also ich lese in der Aufgabe überhaupt nichts von Tupel... daher hätte ich gesagt, nix mit Vererbung.
Praktisch ist es egal: du kannst Vector neu schreiben oder von Tupel erben lassen. Dann schreibst du in Vector einfach noch die zusätzlich benötigten Methoden. Es gibt keinen speziellen Syntax, nur weil du die Methoden der Oberklasse überschreibst.

P.S. Clone implementiert man korrekterweise so:

public class X{
  public Object clone(){
     return super.clone();
  }
}

Da ist ein bisschen "Magie" im clone() von Object dabei, jedenfalls muss man clone nicht in jeder Unterklasse neu implementieren.

 

[bygones]

warum hast du 3 Klassen ?

du sollst doch nur eine schreiben ?

die hat einen array (die koordinaten).... dann die Methoden und fertig ?!

zur clone hat beni schon was gesagt... toString würde ich einfach mit Arrays.toString() lösen und equals mit Arrays.equals()....

btw - ansich eignet sich die Vektor klasse perfekt für eine Immutable Klasse *g (was aber wahr. nicht teil der aufgabe ist...)

 

[SlaterB]

tja, Interpretation der Aufgabenstellung, das können andere auch kaum besser,
hast du denn keinen Aufgabensteller den du fragen kannst?

klingt für mich eigentlich nicht nach Vererbung, das Wort wird nirgendwo erwähnt außer in der Topicüberschrift,
heißt vielleicht so auch die Aufgabe? ansonsten könnte Erweiterung auch einfach Hinzufügung weiterer Operationen bedeuten,

macht ja keinen rechten Sinn solle elementaren Operationen erst in einer Unterklasse zu definieren,

außerdem heißen die Operation in der Aufgabe add(Vektor V) und boolean equals(Vektor V) ,
das klingt nach nur einer Klasse V

-------

zu deinen Fehlern: die läßt du drin und erwähnst sie mit keinem Wort?
naja da sind so elementare Sachen dabei, da weißt du vielleicht auch nicht weiter


wieso in equals a==b? das macht keinen Sinn, b ist unbekannt, das heißt a==v.a,
die Operation macht übrigens den Fehler dass nach Vergleich eines Wertepaares schon true oder false
zurückgegeben wird, du musst doch erst alle 3 testen!


eine Operation Vector clone() bzw. Tupel clone() geht nicht, selbst wenn es in der Aufgabe so steht,
jedes Objekt hat die Operation Object clone(), die kann zwar überschrieben werden, aber man darf den Rückgabetyp nicht ändern,

wenn eine Oberklasse einen bestimmten Konstruktor definiert und keinen Standardkonstruktor,
dann muss die Unterklasse diesen auch rufen (in einem ebenfalls anzugebenen Konstruktror)

--------

naja und so Kleinigkeiten


wieso heißen die Klassen Tupel und Tupel1 und nicht Vector?, aber die Testklasse heißt dann Vector??

was spricht gegen eine lesbare Formatierung?

i <= a.length -1 habe ich mal durch i < a.length ersetzt, das ist schöner

class Tupel {

	double[] a;

	public Tupel(double[] a) {
		this.a = a;
	}

	double length(Tupel v) {
		double sum = 0;
		for (int i = 0; i < a.length; i++) {
			sum = sum + (a[i] * a[i]);
		}
		return Math.sqrt(sum);
	}

	void add(Tupel v) {
		for (int i = 0; i < a.length; i++) {
			this.a[i] = a[i] + v.a[i];
		}

	}

	void multiply(double d) {
		for (int i = 0; i < a.length; i++) {
			this.a[i] = a[i] * d;
		}
	}

	double scalarProduct(Tupel v) {
		double skalar = 0;
		for (int i = 0; i < a.length; i++) {
			skalar = skalar + a[i] * v.a[i];
		}
		return skalar;
	}
}

class Tupel1 extends Tupel {

	public Tupel1(double[] a) {
		super(a);
	}

	public String toString() {
		String s = "(" + a[0];
		for (int i = 1; i < a.length; i++) {
			s = s + ", " + a[i];
		}
		return s + ")";
	}

	public boolean equals(Tupel v) {
		for (int i = 0; i < a.length; i++) {
			if (a[i] != v.a[i]) {
				return false;
			}
		}
		return true;
	}

	public Object clone() {
		Tupel clone = new Tupel(a);

		return clone;

	}
}

class Vector {

	public static void main(String args[]) {
		double[] a = { 1, 2, 3, 4 };
		Tupel v = new Tupel(a);
		v.add(new Tupel(a));
		System.out.println("v = " + v.toString());
		double[] b = { 3, 3, 3, 3 };
		System.out.println("scalar product = " + v.scalarProduct(new Tupel(b)));
	}
}

 

[Beni]

eine Operation Vector clone() bzw. Tupel clone() geht nicht, selbst wenn es in der Aufgabe so steht,
jedes Objekt hat die Operation Object clone(), die kann zwar überschrieben werden, aber man darf den Rückgabetyp nicht ändern.

Einspruch :!: Ab Java 1.5 absolut legitim :bae:

 

[SlaterB]

das passiert ja jetzt ständig