Abfrage mit instanceof führt zu unerwarteten Wert

[NeoLexx]

Hej, ich kann folgendes nicht nachvollziehen, vielleicht kann mir da einer von euch helfen:


In Zeile 7 erzeuge ich ein Array vom DatenTyp ObjektUniversum.
In Zeile 8 ein Array vom Typ SonnenSystem.
ObjektUniversum ist die Oberklasse von SonnenSystem.



In Zeile 302 mache ich eine Abfrage mit instanceof zu Testzwecken, dir mir ein "false" liefert, was ich jedoch nicht ganz nachvollziehen kann:

Wie man sehen kann, teile ich in Zeile 301 dem Objekt galaxie [50] [50] das Objekt sonne [1499] zu, wie bereits oben erwähnt handelt es sich um Objekte, die aus Oberklasse und Unterklasse bestehen. Soweit mir bekannt ist, soll instanceof bei dem vergleich von zwei Datentypen, die in einem Vererbungsverhältnis stehen, also Oberklasse und Unterklasse, ein true ausgeben.

 

[mihe7]

Wo ist denn die Zuweisung zu sonnen[1499]?

 

[NeoLexx]

Hätte jetzt gedacht, dass ich es in Zeile 301 zugewiesen habe.

Die Idee ist folgende, ich erzeuge ein Feld von 51 x 51 Feldern. Jedes davon ist ein Objekt vom Typ ObjektUniversum. In einer Methode erzeuge ich dann einen Spiralarm aus Sternen, welches sich in diesem 51x51 Feld wieder findet (siehe Bild). Überall da wo die Methode einen Stern erzeugt möchte ich den Typ des Feldes von ObjektUniversum in den Typ SonnenSystem abändern.

 

[mihe7]

Das ist die Zuweisung zu galaxy[50][50], aber irgendwo musst Du ja sonnen[1499] auch einen Wert zuweisen.

 

[NeoLexx]

Hmm, also irgendwie weiß ich gerade noch nicht was ich dem DatenTypen von Typ SonnenSystem überhaupt für ein Wert zuweisen soll. Die Klasse SonnenSystem selbst, ist noch nicht fertig. Soll aber Später ein ähnliches Feld von 51 x 51 Feldern enthalten die dann aus Objekten vom Typ SonnenSystemObjekt bestehen, die wiederum Oberklasse von Objekten wie Planet usw. sind..



Ich habe aber gerade die Vermutung, dass das Ganze mit Arrays anzugehen, nicht meiner Modellvorstellung (siehe UML) entspricht und es dadurch zu einer Verwirrung meinerseits gekommen ist. Na ja, heute Nachmittag, werde ich mich mit dem Code weiter auseinander setzen, bis dahin bin ich für alle Tipps dankbar.

 

[mrBrown]

Du kannst das durchaus mit Arrays lösen, der Modellierung in UML widerspricht das nicht. Das was du im Code machst passt nur nicht ganz zu dem Diagramm bzw ist einfach auf Java-Ebene falsch.

Nach deinem Diagramm enthält eine Galaxie ein (sollten eher mehrere sein) "ObjektUniversum" (potentiell solltest du dir einen besseren Namen überlegen 😉 ), und ein "ObjektUniversum" kann dann ein Sonnensystem sein – klingt soweit nicht falsch.

In deinem Code enthält eine Galaxie aber sowohl viele "ObjektUniversum", als auch zusätzlich 1500 Sonnensysteme. Letzteres solltest du an der Stelle einfach streichen, dann entspricht es wieder deinem Diagramm. In dem ObjektUniversum-Array solltest du stattdessen direkt Sonnensystem-Objekte speichern, also etwas in die Richtung von galaxie[50][50] = new Sonnensystem();.

 

[NeoLexx]

Nach deinem Diagramm enthält eine Galaxie ein (sollten eher mehrere sein) "ObjektUniversum" (potentiell solltest du dir einen besseren Namen überlegen 😉 ), und ein "ObjektUniversum" kann dann ein Sonnensystem sein – klingt soweit nicht falsch.

Ja, das soll eigentlich ObjektGalaxie heißen. Werde ich noch abändern.

In deinem Code enthält eine Galaxie aber sowohl viele "ObjektUniversum", als auch zusätzlich 1500 Sonnensysteme. Letzteres solltest du an der Stelle einfach streichen, dann entspricht es wieder deinem Diagramm. In dem ObjektUniversum-Array solltest du stattdessen direkt Sonnensystem-Objekte speichern, also etwas in die Richtung von galaxie[50][50] = new Sonnensystem();.

Okay, das hört sich gut an. Mit new SonnenSystem(); spreche ich ja den Standart-Konstruktor der Klasse SonnenSystem an. Damit sollte auch die Abfrage isinstanceof funktionieren.

Nach deinem Diagramm enthält eine Galaxie ein (sollten eher mehrere sein) "ObjektUniversum"

Die Multiplizitäten muss ich noch nachtragen, in der Beziehung Galaxie zur ObjektGalxie(noch ObjektUniversum) besteht die Multiplizität 1 ------> 0..*

Danke für die Tipps. 🙂

 

[mrBrown]

Ja, das soll eigentlich ObjektGalaxie heißen. Werde ich noch abändern.

ObjektGalaxie klingt allerdings genauso merkwürdig 😉 wenn „Objekt“ im Namen vorkommt, ist das meist ein Hinweis, das es besser ginge – ich wüsste aber um ehrlich zu sein nicht, wie man das sonst nennen könnte, vielleicht sogar einfach nur „Raum“ (wird allerdings ziemlich OT...)

Mit new SonnenSystem(); spreche ich ja den Standart-Konstruktor der Klasse SonnenSystem an.

Wobei du gut überlegen solltest, ob du überhaupt einen Standard-Konstrukor haben willst, war ja nur ein Beispiel von mir 😉

Die Multiplizitäten muss ich noch nachtragen, in der Beziehung Galaxie zur ObjektGalxie(noch ObjektUniversum) besteht die Multiplizität 1 ------> 0..*

Ach so, die hab ich da nicht mal vermisst, ich finde nur das „hat ein“ sehr unschön (also generell, nicht bezogen auf dein Diagramm). Diese „Standard-Beschriftungen“ würde ich immer weglassen, wenn man nichts passendes findet. Das gleiche drückt man schon durch die Pfeile aus, die nichts-sagende Beschriftung ist da nur „Bloat“ (genauso wie die meisten Multiplizitäten) — aber ist auch wieder ziemlich OT, also ruhig ignorieren was ich alter Mann hier so vor mich hinplappere 😛

 

[NeoLexx]

Danek für die Hilfe! Das Programm macht im genau das, was es in seinem frühen Zustand machen soll:



Es kann nun "Galaxien" per Zufall generieren.

 

[temi]

Meine laienhafte Meinung zur Astronomie ist ja, dass das Universum das große Ganze ist und viele Galaxien enthält. Die Galaxien enthalten wiederum viele Sonnen (-systeme) und andere Dinge, wobei das Universum neben Galaxien auch einzelne Sonnen und andere Dinge enthalten kann (die z.B. irgendwann mal von "ihrer" Galaxie getrennt wurden).

"Andere Dinge" meint z.B. Nebel, schwarze Löcher, Quasare, dunkle Materie, ...

Aber das nur am Rande... 😉