OOP Objekt in IF erzeugen - funktioniert nicht so richtig

[Blah123]

Hallo zusammen,

ich programmiere gerade ein kleines Spiel.

Der Nutzer soll eingeben können, ob er Barbar oder Heiler spielt.
Dann wird in der Bedingung einer if überprüft ob Barbar oder Heiler eingegeben wurde.
Abhängig davon wird Heiler h1 = new Heiler() oder Barbar h1 = new Barbar() erstellt.
D.h. das Objekt soll in beiden fällen gleich heissen, aber von unterschiedlichen Klassen abgeleitet sein. Grund: Vererbungshierarchie und im Spiel will ich den Helden immer über h1 ansprechen (egal ob Barbar oder Held).

Problem:
Wenn ich ausserhalb der IF bzw while h1 ansprechen will, gibt der Compiler den Fehler "Cannot find Symbol". (Bei h1.name=...) Komischerweise kann ich h1.XXX ansprechen. D.h. es erscheinen die Attribute und Methoden....


Habe euch auch noch die Dateien Angehängt.

Wäre nett, wenn jemand was weiß.

/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 11.11.2017
  * @author
  */
import java.util.Scanner;
public class Startklasse {
 
  public static void main(String[] args) {
    boolean nochmal=true;
    Scanner tastatur = new Scanner(System.in);
 
 
    Monster m1 = new Monster();
    m1.name="Mooonster";
    m1.lebensenergie=10;
 
 
    //Abfrage, welche Klasser der Spieler spielen will
    while (nochmal) {
   
      System.out.print("Welche Klasse wollen Sie spielen (Heiler oder Barbar)? : ");
      String heldenklasse=tastatur.next();
   
   
      //Heldenobjekt wird erstellt. Entweder Heiler oder Barbar
      if (heldenklasse.equals("Heiler")) {
        Heiler h1 = new Heiler();
     
        h1.lebensenergie=10;
        h1.schlachtruf();
     
     
        nochmal=false;
      } // end of if
      else if (heldenklasse.equals("Barbar")){
        Barbar h1 = new Barbar();
     
        h1.lebensenergie=10;
        h1.schlachtruf();
        nochmal=false;
      } // end of if-else
      else {
        System.out.println("Gebe als Heldenklasse Heiler oder Barbar ein!!!");
      } // end of if-else
   
    } // end of while
 
    System.out.print("Gebe den Namen deines Helden ein: ");
    h1.name=tastatur.nextLine();
 
    System.out.println("Die Lebenenergie des Helden und des Monsters betragen jeweils 10!");
    System.out.println("------------------------------------------");
 
     
    while (nochmal) {
      System.out.println("");
   
      //Held greif Monster an
      int schadenheld=h1.angriff();
   
      //Sonderfähigkeit des Heldes wird abgefragt
      if (heldenklasse.equals("Heiler")) {
        h1.heilen();
      } // end of if
      else if (heldenklasse.equals("Barbar")){
     
      } // end of if-else
   
      //Nach Angriff des Helden wird der Gesundheitscheck für das Monster ausgeführt
      m1.gesundheitsCheck(schadenheld);
   
      //Monster greift Held an
      int schadenmonster=m1.angriff();
   
      //Gesundheitscheck für den Held
      h1.gesundheitsCheck(schadenmonster);
   
      //      if (m1.lebensenergie<=0 || h1.lebensenergie<=0) {
      //        weiterspielen=false;
      //      } // end of if
   
   
    } // end of while
 
     
 
 
 
 
 
 
 
  } // end of main

} // end of class Startklasse

 

[mrBrown]

Du musst die Variable mit einem passenden Typ vor dem if deklarieren - ansonsten ist die Variable nur innerhalb des if-Blocks bekannt.

in dem if musst du die Variable dann nur noch initialisieren.

 

[SchwarzWeiß]

Die passende Klasse wäre in dem Fall Held.

Dann kannst du auch anstatt

if (heldenklasse.equals("Heiler")) {
        h1.heilen();
      } // end of if
else if (heldenklasse.equals("Barbar")){
  
      }

schreiben:

if (h1 instanceof Heiler) {
        h1.heilen();
      } // end of if
else if (h1 instanceof Barbar){
   
      }

Dadurch könntest du am Anfang z.B. auch Zahlen abfragen, damit der User die Heldenklassen nicht ausschreiben muss. (z.B. 1 für Heiler, 2 für Barbar oder so)

Mit welchem Tool hast du das Klassendiagramm gemacht?

 

[Blah123]

Du musst die Variable mit einem passenden Typ vor dem if deklarieren - ansonsten ist die Variable nur innerhalb des if-Blocks bekannt.

in dem if musst du die Variable dann nur noch initialisieren.

Sowas habe ich mir schon gedacht. Dann muss ich aber auch schon vor dem IF je ein Objekt für Barbar und ein Objekt für den Heiler erstellen. Die kann ich dann aber nicht gleich bennenen. Was dazu führt, dass ich im Spiel bei jeder Aktion des Spielers abfragen muss, ob es sich um einen Barbaren oder Heiler handelt und dann die Aktion auf das passende Objekt ausführen muss. zB Wenn die Heldenklasse=Heiler, dann führe heilen() auf Heilerobjekt aus, wenn Heldenklasse=Barbar, dann rage() auf Barbarbenobjekt. Genauso bei dem Gesundheitscheck. Da muss ich immer abfragen was für eine Heldenklasse vorliegt und dann den Gesundheitscheck auf das entsprechende Objekt ausführen?!
--> Gibt es da keine elegantere Lösung?

Die passende Klasse wäre in dem Fall Held.

Dann kannst du auch anstatt

if (heldenklasse.equals("Heiler")) {
        h1.heilen();
      } // end of if
else if (heldenklasse.equals("Barbar")){
 
      }

schreiben:

if (h1 instanceof Heiler) {
        h1.heilen();
      } // end of if
else if (h1 instanceof Barbar){
  
      }

Dadurch könntest du am Anfang z.B. auch Zahlen abfragen, damit der User die Heldenklassen nicht ausschreiben muss. (z.B. 1 für Heiler, 2 für Barbar oder so)

Mit welchem Tool hast du das Klassendiagramm gemacht?

Das mit dem Instanceof würde doch nur was bringen, wenn ich mehrere Objekte mit Namen h1 verschiedener Klassen erzeugen könnte. Die Objekte dürfen doch nicht gleich heissen?!

Habe das Klassendiagramm mit dem JavaEditor erstellt

 

[SchwarzWeiß]

mrBrown meinte

Held h1;

vor der if-Abfrage schreiben. (Deklaration der Variable)
Und dann z.B. im if-Körper die Instanz erzeugen:

h1 = new Heiler();

Dann dürften sich auch die restlichen Fragen geklärt haben.

 

[Blah123]

mrBrown meinte

Held h1;

vor der if-Abfrage schreiben. (Deklaration der Variable)
Und dann z.B. im if-Körper die Instanz erzeugen:

h1 = new Heiler();

Dann dürften sich auch die restlichen Fragen geklärt haben.

Ah. Wusste garnicht, dass man ein Objekt deklarieren kann, ohne den Konstruktor direkt aufzurufen (Held h1 ohne new...). Und dann danach den Konstruktor einer Subklasse drauf anwenden...

 

[SchwarzWeiß]

Ah. Wusste garnicht, dass man ein Objekt deklarieren kann, ohne den Konstruktor direkt aufzurufen (Held h1 ohne new...). Und dann danach den Konstruktor einer Subklasse drauf anwenden...

Doch das geht und hat, wie man an deinem Beispiel wunderbar sehen kann, große Vorteile.

 

[Blah123]

Ich habs aber grad so gemacht, dann findet er ausserhalb der if aber nur die Methoden von Held sowie von Spielfigur, aber vom Heiler und dem Barbaren nicht. Hier zB bringt er einen Fehler bei h1.heilen() bei den Sonderfähigkeiten:

/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 11.11.2017
  * @author
  */
import java.util.Scanner;
public class Startklasse {
 
  public static void main(String[] args) {
    boolean nochmal=true;
    Scanner tastatur = new Scanner(System.in);
   
   
    Monster m1 = new Monster();
    m1.name="Mooonster";
    m1.lebensenergie=10;
   
    Held h1;
   
    //Abfrage, welche Klasser der Spieler spielen will
    while (nochmal) {
     
      System.out.print("Welche Klasse wollen Sie spielen (Heiler oder Barbar)? : ");
      String heldenklasse=tastatur.next();
     
     
      //Heldenobjekt wird erstellt. Entweder Heiler oder Barbar
      if (heldenklasse.equals("Heiler")) {
        h1 = new Heiler();
       
        h1.lebensenergie=10;
        h1.schlachtruf();
       
       
        nochmal=false;
      } // end of if
      else if (heldenklasse.equals("Barbar")){
        h1 = new Barbar(); 
       
        h1.lebensenergie=10;
        h1.schlachtruf();
        nochmal=false;
      } // end of if-else
      else {
        System.out.println("Gebe als Heldenklasse Heiler oder Barbar ein!!!");
      } // end of if-else
     
    } // end of while
   
    System.out.print("Gebe den Namen deines Helden ein: ");
    h1.name=tastatur.nextLine();
   
    System.out.println("Die Lebenenergie des Helden und des Monsters betragen jeweils 10!");
    System.out.println("------------------------------------------");
   
       
    while (nochmal) {
      System.out.println("");
     
      //Held greif Monster an
      int schadenheld=h1.angriff();
        
      //Sonderfähigkeit des Heldes wird abgefragt
      if (h1 instanceof Heiler) {
        h1.heilen();
      } // end of if
      else if (h1 instanceof Barbar){
             
      } // end of if-else
     
     
      //Nach Angriff des Helden wird der Gesundheitscheck für das Monster ausgeführt
      m1.gesundheitsCheck(schadenheld);
     
      //Monster greift Held an
      int schadenmonster=m1.angriff();
     
      //Gesundheitscheck für den Held
      h1.gesundheitsCheck(schadenmonster);
     
      //      if (m1.lebensenergie<=0 || h1.lebensenergie<=0) {
      //        weiterspielen=false;
      //      } // end of if
     
     
    } // end of while
   
       
   
   
   
   
   
   
   
  } // end of main

} // end of class Startklasse

 

[SchwarzWeiß]

Du musst einen Cast machen.

(Heiler) h1.heilen();

 

[mrBrown]

Du müsstest das Objekt nach dem instanceof-Check noch zu dem entsprechendem Typ casten, wenn du spezifische Methoden aufrufen willst.

 

[mrBrown]

Du musst einen Cast machen.

(Heiler) h1.heilen();

Aber dann so:

((Heiler) h1).heilen();

 

[SchwarzWeiß]

Du hast recht. Eclipse schlägt die spezifischen Methoden innerhalb einer instanceof-if-Bedingung immer vor und castet dann automatisch, daher hab ich das schon länger nicht mehr von Hand gemacht