Schleife bis "Bedingung" ausführen

[xtina80]

Hallo, immer wenn ich gerade ein Problem gelöst haben, stoße ich leider wieder auf neue Probleme

Ich möchte für mein Spiel eine ganz simple Kampfschleife, die endet sobald einer der kämpfenden Charaktere tot ist.

Durch Recherchen zum Thema "Kampfsystem" bin ich darauf gestoßen, dass man dieses Problem mit

do
"methode";
while("Bedingung trifft nicht mehr zu");

lösen kann.

Ich habe dies nun ausprobiert und es funktioniert nicht, da sich die Spielfiguren nun in Endlosschleife bekämpfen, obwohl sie schon längst tot sind

Mein Code dazu:

Spielfigur held = new Held();
Monster goblin = new Goblin();
      do {
        held.schlagen(goblin);
        goblin.schlagen(held);
       } 
       while(held.leben <= 0 || goblin.leben <= 0);

Was mache ich falsch?

Es tut mir Leid, wenn ich mich sehr doof anstelle aber ich bin totaler Anfänger in Java! Ich habe mir zwar bereits verschiedene Codes zu diesem Thema angeschaut aber die waren alle mit einem zu komplizierten Grundgerüst drum herum, sodass ich keinen Durchblick mehr hatte, wie ich das auf meinen Code anwenden könnte.

Für Lösungsvorschläge wäre ich sehr dankbar!

LG

 

[Joose]

Dein Bedingung ist falsch
Du sagst "solange das Leben von dem Helden oder dem Goblin kleiner gleich 0 ist führe den Code in der Schleife aus"

Eigentlich müsstest du prüfen: "Ist das Leben vom Held und vom Goblin noch größer 0"
Somit würde die Schleife enden sobald einer von beiden weniger oder gleich 0 Leben hat.

 

[stg]

Mit anderen Worten: einfach die Bedingung negieren.

 

[xtina80]

Also müsste das so aussehen?

do {
        held.schlagen(goblin);
        goblin.schlagen(held);
       }
       while(held.leben > 0 && goblin.leben > 0);

Das Problem ist jetzt aber, dass die sich nicht in einer Schleife bekämpfen bis einer von Ihnen stirbt, sondern nur jeweils einmal

 

[Joose]

Das Problem ist jetzt aber, dass die sich nicht in einer Schleife bekämpfen bis einer von Ihnen stirbt, sondern nur jeweils einmal

Naja kann es sein das einer von beiden schon nach dem 1.Schlag stirbt? Dann würde die Schleife ja richtig funktionieren.

Entweder verwendest du den Debugger und kontrollierst was passiert oder du lässt dir ein paar Kontrollausgaben auf der Konsole ausgeben um zu sehen wie sich die Werte von "leben" der beiden ändern.

 

[xtina80]

Naja kann es sein das einer von beiden schon nach dem 1.Schlag stirbt? Dann würde die Schleife ja richtig funktionieren.

Nee, jeder von Ihnen verliert jeweils einmal 10 von 100 Leben und dann hört die Schleife auf

 

[VfL_Freak]

Moin,

prima
Und wir dürfen jetzt raten, wie Dein Code ausschaut ??
Gibt es auch was gewinnen ??

Gruß Klaus

 

[Jardcore]

Kleine Frage nebenbei

do {
held.schlagen(goblin);
goblin.schlagen(held);
}
while(held.leben > 0 && goblin.leben > 0);

Was passiert wenn der Goblin vom Helden getötet wird? Schlägt er dann trotzdem?
Zum besseren Verstäntnis würde ich noch eine Methode definieren, die istAmLeben() heißt.

public boolean istAmLeben() {
    return leben > 0;
}

Dann kannst du wiederrum deine Schleife verbessern;

do{
    held.schlagen(goblin);
    goblin.schlagen(held);
} while(held.istAmLeben() && goblin.istAmleben()); // solange einer von beiden am Leben ist.

In deiner schlagen Methode kannst du dann wohlöglich auch mit istAmLeben() Testen ob der jeweilige Character überhaupt noch schlagen darf.[/COLOR]

 

[xtina80]

Das wäre der Code für die Klasse Spielfigur:

    public int leben;
    private int lebenMax;
    private int lebenMin;
    private String name;

    /**
     * Constructor for objects of class Spielfigur
     */
    public Spielfigur()
    {
       leben = 100;
       lebenMax = 100;
       lebenMin = 0;
       Main.ausgabe("Eine neue Spielfigur erscheint. ");
    }
   
    public void setName(String p_name)
    {
        name = p_name;
    }
   
   
    public Spielfigur (int p_leben)
    {
        leben = p_leben;
        lebenMax = p_leben;
        lebenMin = 0;
    }
   
   
    public void schlagen(Spielfigur ziel) 
    { if (ziel.leben>ziel.lebenMin)
       
       ziel.verliereLeben(10);
       
       else
       
       ziel.verliereLeben(0);
    }
   
    public void beissen(Held ziel)
   
    { if (ziel.leben>0)
       
       ziel.verliereLeben(15);
       
       else
       
       ziel.verliereLeben(0);
    }
   
    public void heilen(Spielfigur ziel) 
   
    {if (ziel.leben>ziel.lebenMax)
       
         ziel.heileLeben(5);
       
       else
       
       ziel.heileLeben(0);
    }
   
    public void heileLeben (int anzahl)
    {
       
        if(leben < anzahl)
        {   leben += anzahl;
            Main.ausgabe(this+"hat "+anzahl+" Leben bekommen :) ");
         
        }
        else
        {  
           leben=lebenMax;
            Main.ausgabe(this+" hat bereits volles Leben! ");
           
        }  
    }
     
    public void verliereLeben (int anzahl)
    {
     
        if(leben >= anzahl)
        {
           
            leben =- anzahl;
            Main.ausgabe(this+" hat "+anzahl+" Leben verloren :( ");
         
        }
        else
        {  
            leben=lebenMin;
        }  
       
    }

}

und das wäre der Code für Main zur Ausführung:

 import java.util.*;

public class Main
{
    public static void main()
   
   {
       Scanner scanner = new Scanner (System.in);
       
       ausgabe("Willkommen zu dem besten Spiel, das Sie jemals gespielt haben: Helden und Monster!\n \n Aber bevor das Abenteuer losgehen kann, müssen Sie zunächst einmal bestimmen, mit wem Sie überhaupt spielen wollen. \n ");
   
       ausgabe("\nBitte wählen Sie für einen männlichen Charakter (m) und für einen weiblichen Charakter (w)!");
       
       
         boolean Geschlecht = true;
         
         while (Geschlecht){
         String input = scanner.next();
         if(input.equalsIgnoreCase("m")) 
         {
            System.out.println("Sie haben sich also für einen Mann entschieden.");
            Geschlecht = false;
         }
       
   
         else if (input.equalsIgnoreCase("w"))
       
         {
            System.out.println("Sie haben sich also für eine Frau entschieden.");
            Geschlecht = false;
         }
       
         else
       
         {
            System.out.println("Mann oder Frau! Sie können nur zwischen diesen beiden wählen!\nBitte wählen Sie für einen männlichen Charakter (m) und für einen weiblichen Charakter (w)!");
       
         }
   
      }
   
     ausgabe("\nBitte wählen Sie (1) für Heiler, (2) für Magier oder (3) für Krieger:");
   
     Spielfigur held = null;
       
   
       
       while(held == null)
       {
           String input2 = scanner.next();
           
           switch (input2)
         
           
          {
     
             case "1":
             held = new Heiler();
             System.out.println("Glückwunsch! Sie haben sich mit dem Heiler für die richtige Klasse entschieden!\n");
             break;
                   
             case "2":
             held = new Magier();
             System.out.println("Glückwunsch! Sie haben sich mit dem Magier für die richtige Klasse entschieden!\n");
             break;
               
             case "3":
             held = new Krieger();
             System.out.println("Glückwunsch! Sie haben sich mit dem Krieger für die richtige Klasse entschieden!\n");
             break;
       
             default:
             System.out.println("Falsche Eingabe! Bitte wählen Sie (1) für Heiler, (2) für Magier oder (3) für Krieger:");
             break;
       
         }
      }
     
      System.out.println("\nSo! Jetzt kann das Abenteuer aber losgehen! \n Sind Sie bereit? Bitte wählen Sie (j) für JA und (n) für NEIN");
     
      boolean Spielen= true;
       
       while (Spielen){
         String input = scanner.next();
         if(input.equalsIgnoreCase("j")) 
         {
            System.out.println("Das war die einzig richtige Entscheidung!\n");
            Spielen = false;
         }
       
   
         else if (input.equalsIgnoreCase("n"))
       
         {
            System.out.println("\nDas glaube ich Ihnen nicht! Am besten wir versuchen es noch einmal: \n Entweder (j) für JA oder (n) für NEIN");
           
         }
       
         else
       
         {
            System.out.println("\nFalsche Eingabe! Entweder (j) für JA oder (n) für NEIN");
       
         }
   
      }
     
     
       Monster goblin = new Goblin();
      do {
        held.schlagen(goblin);
        goblin.schlagen(held);
       }
       while(held.leben > 0 && goblin.leben > 0);
       

     
    }

    public static void ausgabe (String text)
   {
       System.out.println (text);
   }
}

 

[xtina80]

Kleine Frage nebenbei

Was passiert wenn der Goblin vom Helden getötet wird? Schlägt er dann trotzdem?
Zum besseren Verstäntnis würde ich noch eine Methode definieren, die istAmLeben() heißt.

Ja, genau das hatte ich mir auch schon überlegt. Ich hatte das so:

public boolean lebtNoch()
    {
        if(leben > 0)
        {
            return true;
        }
       
        else
       
        {  Main.ausgabe(this+" ist an seinen Verletzungen gestorben ");
           
            return false;
        }  
    }

aber ist das dann in Verbindung mit der schlagen Methode

public void schlagen(Spielfigur ziel) 
    { if (ziel.leben>ziel.lebenMin)
       
       ziel.verliereLeben(10);
       
       else
       
       ziel.verliereLeben(0);
    }

und verliereLeben:

public void verliereLeben (int anzahl)
    {
     
        if(leben >= anzahl)
        {
           
            leben =- anzahl;
            Main.ausgabe(this+" hat "+anzahl+" Leben verloren :( ");
         
        }
        else
        {  
            leben=lebenMin;
        }  
       
    }

nicht "doppelt gemoppelt"?

 

[Joose]

Nein.
Dein Held schlägt den Goblin, die Gesundheit vom Goblin fällt auf <=0.
Laut Code würde der Goblin nun nochmal den Held schlagen, was aber nicht passieren darf.

Deine Schlagen Methode prüft nur ob das Ziel noch lebt, nicht aber die schlagende Figur selbst.

 

[flopalko]

Dein Fehler ist in verliereLeben, da gehört -= und nicht =-

 

[flopalko]

Weiters als Anregung zum Verbessern deines Codes:
Wenn du schon Membervariablen wie lebenMin und lebenMax hast denke darüber nach, wo es sinnvoll ist diese einzusetzen, wo du sie jetzt noch nicht hast.
Kann es in deinem Spiel zB eine Figur geben die unsterblich ist? Ihr lebenMin wäre demnach ja dann zB 2 und nicht 0. Kann es in deinem Code dann vorkommen, dass diese Figur weniger als 2 Leben hat?
Wie gesagt, gehe wenn du fertig bist kritisch deinen Code durch und frage dich, ob du manche Sachen nicht besser lösen kannst.

 

[xtina80]

Vielen vielen Dank für eure Mühe und Hilfestellung! Jetzt funktioniert es so wie ich es mir vorgestellt habe!

Weiters als Anregung zum Verbessern deines Codes:
Wie gesagt, gehe wenn du fertig bist kritisch deinen Code durch und frage dich, ob du manche Sachen nicht besser lösen kannst.

Das versuche ich stets, nur fällt es mir jetzt am Anfang noch schwer einen Überblick zu erhalten, da es noch so einige Baustellen gibt, die ich noch lösen möchte bevor es ans Optimieren geht, wie zB.:

- Namensabfrage mit Eingabe und Weiterbenutzung des Namens im Laufe des Spiels
- die Geschlechter nicht nur als Abfrage sondern ins Spiel einbauen (anstatt Krieger, dann KriegerIN usw.)
- bei verlorenem Kampf ein "Game over" und Spielende
- eventuell ein Zufallsgenerator für den Schaden anstatt festgelegte Parameter

und noch so einige Verbesserungen und Kleinigkeiten, von denen ich momentan noch keine Ahnung habe wie ich diese umsetzen soll.