Konstante in Objekt definieren

[unarmed_soldier]

Ich hab leider ein Problem und weiß nicht wie ich das programmiertechnisch lösen kann. Vllt. hat ja jemand von euch eine Idee.

Ich habe eine Klasse für ein Spiel programmiert, und habe die Methode buyFacility(int facility), um neue Geräte zu kaufen.
Für Geräte habe ich eine eigene Klasse geschrieben, die beschreibt ob der Spieler dieses Gerät bereits besitzt oder nicht (boolean owned) und wieviel der Preis für dieses Gerät ist (float price).
Der buyFacility Methode übergebe ich eine Konstante die ich momentant noch manuell in der Klasse des Spiels definiert habe (fortlaufende Nummer). Dann bearbeite ich die einzelnen Geräte mit einer switch Anweisung.

public class spiel()
{
   Facility fishing_rod = new Facility(price, true/false);
   Facility hunting_rifle = new Facility(price, true/false);
   public final int FACILITY_FISHING_ROD = 1;
   public final int FACILITY_HUNTING_RIFLE = 2;

   public void buyFacility(int facility) { //Hier wird dann die Koonstante übergeben
   switch(facility) {
   case FACILITY_FISHING_ROD:
      //...
   case FACILITY_HUNTING_RIFLE:
      //... 
   }
}

Facility Class sieht so aus:

public class Facility {
   public Facility(float price, boolean owned) {
   //...
   }
   private float price;
   private boolean owned;
}

Wenn ich allerdings viele Facilities habe, wird das schnell unübersichtlich. Ich möchte also jeder Facility-Instanz eine ID im Konstruktor geben. Ich habe das schon versucht mit einer final int ID. Dann habe ich noch eine static int nextID gemacht und im Konstruktor den Wert der ID mit nextID gesetzt und dann nextID erhöht.
Leider zu früh gefreut, weil switch nimmt diese Konstante nicht weil die Instanz zu der die ID gehört ja nicht existieren muss.

case fishing_rod.ID
   //...

Wie kann man dieses Problem lösen?
Ich hab diesen Thread absichtlich nicht zu den Spielen gegeben, weil das Thema eher allgemein ist und auch Nicht-Spiele-Programmierer interessieren könnte. Wenn das falsch war dann bitte verschieben..

Ich hoffe es gibt jemanden der mir da helfen kann, weil ich weiß echt nicht weiter... ???:L

 

[Andi_CH]

Solche switch -case Konstrukte oder auch lange if then else if statements sind eine Zeichen dass OO mangelhaft umgesetzt wurde -

Ich würde Klasen FishingRod, HuntingRifle u.s.w. erstellen und denen nur das bisschen Code mitgeben das für diese Klasse relevant ist. Die alle müssen eine gemeinsame Vaterklasse haben (Facility) die abstrakt sein kann

 

[unarmed_soldier]

Was sollte sich das bringen wenn ich eine extra Kindklasse von Facility mache? fishing_rod und hunting_rifle haben die selben Attribute und Methoden und müssen sich ja nicht unterscheiden?
Und was würde das an meinem Problem ändern?
Ich versteh leider nicht wie mir das helfen sollte :bahnhof:

EDIT: Ich hätte auch schon die Idee gehabt einfach in die Facility Klasse eine buy() Methode einzubauen, nur leider kann ich dann nicht auf die Variable money von der Klasse spiel zugreifen..

 

[Andi_CH]

Aber der code in den Methoden ist ja anders sonst hättest du nicht so ein irres switch - case Konstrukt.

Wenn sich die nich tunterscheiden dann würde es das nicht brauchen.

Und wenn du meinst ich versuche dir nicht zu helfen .... na ja.

 

[unarmed_soldier]

Sorry das war nicht böse gemeint, ich bin dankbar für deine Hilfe aber versteh wirklich nicht was du mir damit sagen willst. Die buyFacility() methode ist ja von der Klasse spiel.
Wie bereits im Edit oben gesagt, kann ich sie nicht in das Facility Objekt hineinnehmen, weil ich ja auf die money Variable von der Klasse spiel zugreifen muss. Vielleicht hast du auch eine Idee wie ich das Geld von spiel abziehen kann wenn ich eine buy() Methode in die Klasse Facility packe

 

[Andi_CH]

Es ist nicht sicher ob das dahingehört - ich weiss ja wirklich nicht, was du genau willst - viielleicht ist buyFacility ein Factory die ein bestimmtes Objekt liefert - aber was in der realen Welt unterschieldich ist, wird wohl auch je eine eigene Klasse werden.

Ich kann nämlich nicht wirklich glauben, dass eine Fischerrute genau dasselbe ist wie ein Jagdgewehr ;-)

Statt von aussen buyFacilitiy(Jagddgewehr) aufzurufen, würde ich viel lieber einfach new Jagdgewehr(); machen - es brauche echt kein switch case.

Du weisst aussen was du willst, packst das in einen int und drinnen switcht du wieder auseinander ... das meine ich mit OO nicht umgesetzt.

 

[unarmed_soldier]

Ok vllt war das ein Fall von zu wenig Info.

Also bei dem Spiel geht es um das Geld verdienen. Man kann dazu entweder fischen, jagen oder züchten. Das Spiel hat natürlich noch viel mehr Inhalt aber das zu erklären würde zu lange dauern.
Die "Facilities" (Angelrute, Jagdgewehr, Weide) benötigt man um diese Aktionen durchführen zu können. Man muss sie also vorher kaufen. Und festzustellen ob der Spieler diese Geräte besitzt, habe ich eine Klasse Facilities gemacht, von denen ich für jedes Gerät ein Objekt erstelle. Hier wird dann gespeichert wieviel das Objekt zu kaufen kostet, bzw. ob es schon im Besitz ist oder nicht.
Ich hab eine Klasse hunterGame (bis jetzt spiel genannt) gemacht, die alle Sachen für das Spiel übernimmt. Z.B gibt es da auch die Methode buyFacility(), um eines dieser Geräte zu kaufen, und nachher damit arbeiten zu können. Aber natürlich sell() um die Fische, etc. zu verkaufen oder work, etc.
Mit der Hauptklasse hunterGame kann ich also das Spiel auf verschiedenen Wegen umsetzten, z.B. nur im Terminal oder als GUI oder ähnliches. Ich programmiere also nur die Menüfunktionen, den Rest soll dann die fertige Klasse hunterGame übernehmen. Verstehst du jetzt besser was ich meine?