Warum wird if-Schleife nicht ausgeführt?

[Kadezzo]

Das was jetzt gebraucht wird (denke ich zumindest) sieht so aus:

    public void add()
    {
        if(Frucht1 == "Apfel") {
        if(meineAepfel.length > 0){
            meinKorb.apfelHinzufuegen();
            Scanner scanner1 = new Scanner (System.in);
            System.out.print("Wähle deine zweite Frucht: ");  
            Frucht2 = scanner1.next();
            Korb2();
        } else {
            System.out.println("Wähle eine andere erste Frucht. Es gibt keine Äpfel mehr!");
            Korb1();
        }
    }
}


    public void Korb1() {
        Scanner scanner = new Scanner (System.in);
        System.out.print("Wähle deine erste Frucht: ");  
        Frucht1 = scanner.next();     
        add();
       
}

Meine Frage lautet nun wieso Java nachdem ich Korb1 ausgeführt habe und scanner.next() die Variable "Apfel" zugewiesen habe, in add() dann nicht die if-Schleife ausführt.

Wenn das an Code nicht reicht, kann ich auch die ganze Klasse posten

 

[Chips199]

Hallo,
du kannst Strings nicht mit == vergleichen sondern nur mit:

String str1 = "Apfel";
if(str1.equals("Apfel")) {
// tue etwas
}

Mfg Chips199
Ps: benutze demnächst bitte[ code=Java][ /code] für den Code.

 

[Kadezzo]

Oh man, danke erstmal. Das funktioniert jetzt auch, stehe nun aber vor einem weiteren Problem. Ich poste wieder erstmal den Code

Klasse Spielbrett

    public void add()
    {
        if(Frucht1.equals("Apfel")) {
        if(meineAepfel.length > 0){
            meinKorb.apfelHinzufuegen();
            Scanner scanner1 = new Scanner (System.in);
            System.out.print("Wähle deine zweite Frucht: "); 
            Frucht2 = scanner1.next(); // Get what the user types.
            Korb2();
        } else {
            System.out.println("Wähle eine andere erste Frucht. Es gibt keine Äpfel mehr!");
            Korb1();
        }
    }
}
    public void Korb1() {
        Scanner scanner = new Scanner (System.in);
        System.out.print("Wähle deine erste Frucht: "); 
        Frucht1 = scanner.next(); // Get what the user types.   
        add();
      
}

Und hier Klasse Korb

public class Korb{
    private Apfel[] meineAepfel;
    private Kirsche[] meineKirschen;
    private Birne[] meineBirnen;
    private Pflaume[] meinePflaumen;
    public int a = 0;
    public int k = 0;
    public int b = 0;
    public int p = 0;
   
    public void apfelHinzufuegen() {
        meineAepfel = new Apfel[a];
        for(a = 0; a<meineAepfel.length; a++){ 
            meineAepfel[a] = new Apfel(); 
        }
    }
    public void kirscheHinzufuegen() {
        meineKirschen = new Kirsche[k];
        for(k = 0; k<meineKirschen.length; k++){ 
            meineKirschen[k] = new Kirsche(); 
        }
    }
    public void birneHinzufuegen() {
        meineBirnen = new Birne[b];
        for(b = 0; b<meineBirnen.length; b++){ 
            meineBirnen[b] = new Birne(); 
        }
    }
    public void pflaumeHinzufuegen() {
        meinePflaumen = new Pflaume[p];
        for(p = 0; p<meinePflaumen.length; p++){ 
            meinePflaumen[p] = new Pflaume(); 
        }
    }
}

Nun die Frage: Wieso bekomme ich eine "ava.lang.NullPointerException" im Bezug auf "meinKorb.apfelHinzufuegen();"?
Bin wieder für jede Hilfe dankbar!

 

[Chips199]

Kannst du bitte mal die ganze Klasse Spielbrett posten?

 

[Kadezzo]

import java.util.Scanner;
public class Spielbrett{
    public Apfel[] meineAepfel;
    private Kirsche[] meineKirschen;
    private Birne[] meineBirnen;
    private Pflaume[] meinePflaumen;
    private Würfel meinWuerfel;
    private Rabe meinRabe;
    private Obst meinObst;
    private Korb meinKorb;
    private int a = 10;
    private int k = 10;
    private int p = 10;
    private int b = 10;
    private String Frucht1;
    private String Frucht2;
    private String Apfel = "Apfel";
    public Spielbrett(Rabe pRabe) {
        meinRabe = pRabe;
        meinWuerfel = new Würfel();
        meineAepfel = new Apfel[a];
        meineKirschen = new Kirsche[k];
        meineBirnen = new Birne[b];
        meinePflaumen = new Pflaume[p];
        for(a = 0; a<meineAepfel.length; a++){
            meineAepfel[a] = new Apfel();
        }
        for( k = 0; k<meineKirschen.length; k++){
            meineKirschen[k] = new Kirsche();
        }
        for(b = 0; b<meineBirnen.length; b++){
            meineBirnen[b] = new Birne();
        }
        for(p = 0; p<meinePflaumen.length; p++){
            meinePflaumen[p] = new Pflaume();
        }
    }
    public Korb getKorb(){
        return meinKorb;
    }
    public Rabe getRabe(){
        return meinRabe;
    }
    public Würfel getWürfel(){
        return meinWuerfel;
    }
    public void würfeln(){
        meinWuerfel.würfeln();
        if(meineBirnen.length == 0 && meineKirschen.length == 0 && meineAepfel.length == 0 && meinePflaumen.length == 0) {
            System.out.println("Die Spieler haben das Spiel gewonnen");
        } else {
            if(meinRabe.puzzleteile < 9) {
                if(getWürfel().seite2 == 1/*Apfel*/) {
                    if(meineAepfel.length > 0) {
                        a = a - 1;
                        meineAepfel = new Apfel[a];
                        System.out.println("Apfel");
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Äpfel mehr!");
                    }
                }
                if(getWürfel().seite2 == 0/*Kirsche*/) {
                    if(meineKirschen.length > 0) {
                        k = k - 1;
                        meineKirschen = new Kirsche[k];
                        System.out.println("Kirsche");
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Kirschen mehr!");
                    }
                }
                if(getWürfel().seite2 == 2/*Pflaume*/) {
                    if(meinePflaumen.length > 0) {
                        p = p - 1;
                        meinePflaumen = new Pflaume[p];
                        System.out.println("Pflaume");
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Pflaumen mehr!");
                    }
                }
                if(getWürfel().seite2 == 3/*Birne*/) {
                    if(meineBirnen.length > 0) {
                        b = b - 1;
                        meineBirnen = new Birne[b];
                        System.out.println("Birne");
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Birnen mehr!");
                    }
                }
                if(getWürfel().seite2 == 4/*Korb*/) {
                    Korb1();
                }
            } else {
                System.out.println("Der Rabe hat gewonnen!");
            }
        }
    }
    public void add()
    {
        if(Frucht1.equals("Apfel")) {
        if(meineAepfel.length > 0){
            meinKorb.apfelHinzufuegen();
            Scanner scanner1 = new Scanner (System.in);
            System.out.print("Wähle deine zweite Frucht: ");
            Frucht2 = scanner1.next();
            Korb2();
        } else {
            System.out.println("Wähle eine andere erste Frucht. Es gibt keine Äpfel mehr!");
            Korb1();
        }
    }
}
    public void Korb1() {
        Scanner scanner = new Scanner (System.in);
        System.out.print("Wähle deine erste Frucht: ");
        Frucht1 = scanner.next(); // Get what the user types. 
        add();
    
}
    public void Korb2() {
        if(Frucht2.equals("Apfel")) {
        if(meineAepfel.length > 0){
            meinKorb.apfelHinzufuegen();
        } else {
            System.out.println("Wähle eine andere zweite Frucht. Es gibt keine Äpfel mehr!");
            add2
        }
    }
}
    public void add2() {
                Scanner scanner = new Scanner (System.in);
        System.out.print("Wähle deine zweite Frucht: ");
        Frucht2 = scanner.next();
        Korb2();
    }
}

Ist etwas lang, sorry dafür. Aber hoffe das das zur Antwort beiträgt

 

[Robat]

Du initialisierst "meinKorb" nirgends.
BTW: http://if-schleife.de

 

[Kadezzo]

Ach man, das mir sowas selber nicht auffällt. Und ja das mit der "if-schleife" merk ich mir

 

[Chips199]

In der Methode Korb2 fehlen bei add2 die beiden Klammern.

 

[Kadezzo]

So mittlerweile ist das Spiel fertig. Danke nochmal für die Hilfe. Es handelt sich um das Kinderspiel "Obstgarten". Wer sich für den fertigen Code interessiert & vielleicht Lust hat etwas daraus zu machen (Grafische Oberfläche zum Beispiel), hier ist er mit allen Klassen:

Spielbrett

import java.util.Scanner;
public class Spielbrett{
    public Apfel[] meineAepfel;
    private Kirsche[] meineKirschen;
    private Birne[] meineBirnen;
    private Pflaume[] meinePflaumen;
    private Würfel meinWuerfel;
    private Rabe meinRabe;
    private Obst meinObst;
    private Korb meinKorb;
    private int a = 10;
    private int k = 10;
    private int p = 10;
    private int b = 10;
    private String Frucht1;
    private String Frucht2;
    public Spielbrett(Rabe pRabe, Korb pKorb) {
        meinRabe = pRabe;
        meinKorb = pKorb;
        meinWuerfel = new Würfel();
        meineAepfel = new Apfel[a];
        meineKirschen = new Kirsche[k];
        meineBirnen = new Birne[b];
        meinePflaumen = new Pflaume[p];
        for(a = 0; a<meineAepfel.length; a++){
            meineAepfel[a] = new Apfel();
        }
        for( k = 0; k<meineKirschen.length; k++){
            meineKirschen[k] = new Kirsche();
        }
        for(b = 0; b<meineBirnen.length; b++){
            meineBirnen[b] = new Birne();
        }
        for(p = 0; p<meinePflaumen.length; p++){
            meinePflaumen[p] = new Pflaume();
        }
    }
    public Korb getKorb(){
        return meinKorb;
    }
    public Rabe getRabe(){
        return meinRabe;
    }
    public Würfel getWürfel(){
        return meinWuerfel;
    }
    public void würfeln(){
        meinWuerfel.würfeln();
        if(meineBirnen.length == 0 && meineKirschen.length == 0 && meineAepfel.length == 0 && meinePflaumen.length == 0) {
            System.out.println("Die Spieler haben das Spiel gewonnen");
        } else {
            if(meinRabe.puzzleteile < 9) {
                if(getWürfel().seite2 == 1/*Apfel*/) {
                    if(meineAepfel.length > 0) {
                        a = a - 1;
                        meineAepfel = new Apfel[a];
                        System.out.println("Apfel gewürfelt. Übrige Äpfel: " + meineAepfel.length);
                    }  
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Äpfel mehr!");
                    }
                }
                if(getWürfel().seite2 == 0/*Kirsche*/) {
                    if(meineKirschen.length > 0) {
                        k = k - 1;
                        meineKirschen = new Kirsche[k];
                        System.out.println("Kirsche gewürfelt. Übrige Kirschen: " + meineKirschen.length);
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Kirschen mehr!");
                    }
                }
                if(getWürfel().seite2 == 2/*Pflaume*/) {
                    if(meinePflaumen.length > 0) {
                        p = p - 1;
                        meinePflaumen = new Pflaume[p];
                        System.out.println("Pflaume gewürfelt. Übrige Pflaumen: " + meinePflaumen.length);
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Pflaumen mehr!");
                    }
                }
                if(getWürfel().seite2 == 3/*Birne*/) {
                    if(meineBirnen.length > 0) {
                        b = b - 1;
                        meineBirnen = new Birne[b];
                        System.out.println("Birne gewürfelt. Übrige Birnen: " + meineBirnen.length);
                    }
                    else {
                        System.out.println("Bitte erneut würfeln, es gibt keine Birnen mehr!");
                    }
                }
                if(getWürfel().seite2 == 4/*Korb*/) {
                    Korb1();
                }
            } else {
                System.out.println("Der Rabe hat gewonnen!");
            }
        }
    }
    public void add()
    {
        if(Frucht1.equals("Apfel")) {
            if(meineAepfel.length > 0){
                meinKorb.apfelHinzufuegen();
                a = a - 1;
                meineAepfel = new Apfel[a];
                System.out.println("Apfel gewählt. Übrige Äpfel: " + meineAepfel.length);
                Scanner scanner1 = new Scanner (System.in);
                System.out.print("Wähle deine zweite Frucht: "); 
                Frucht2 = scanner1.next();
                Korb2();
            } else {
                System.out.println("Wähle eine andere erste Frucht. Es gibt keine Äpfel mehr!");
                Korb1();
            }
        }
        else if(Frucht1.equals("Kirsche")) {
            if(meineKirschen.length > 0){
                meinKorb.kirscheHinzufuegen();
                k = k - 1;
                meineKirschen = new Kirsche[k];
                System.out.println("Kirsche gewählt. Übrige Kirschen: " + meineKirschen.length);
                Scanner scanner1 = new Scanner (System.in);
                System.out.print("Wähle deine zweite Frucht: "); 
                Frucht2 = scanner1.next();
                Korb2();
            } else {
                System.out.println("Wähle eine andere erste Frucht. Es gibt keine Kirschen mehr!");
                Korb1();
            }
        }
        else if(Frucht1.equals("Birne")) {
            if(meineBirnen.length > 0){
                meinKorb.birneHinzufuegen();
                b = b - 1;
                meineBirnen = new Birne[b];
                System.out.println("Birne gewählt. Übrige Birnen: " + meineBirnen.length);
                Scanner scanner1 = new Scanner (System.in);
                System.out.print("Wähle deine zweite Frucht: "); 
                Frucht2 = scanner1.next();
                Korb2();
            } else {
                System.out.println("Wähle eine andere erste Frucht. Es gibt keine Birnen mehr!");
                Korb1();
            }
        }
        else if(Frucht1.equals("Pflaume")) {
            if(meinePflaumen.length > 0){
                meinKorb.pflaumeHinzufuegen();
                p = p - 1;
                meinePflaumen = new Pflaume[p];
                System.out.println("Pflaume gewählt. Übrige Pflaumen: " + meinePflaumen.length);
                Scanner scanner1 = new Scanner (System.in);
                System.out.print("Wähle deine zweite Frucht: "); 
                Frucht2 = scanner1.next();
                Korb2();
            } else {
                System.out.println("Wähle eine andere erste Frucht. Es gibt keine Pflaumen mehr!");
                Korb1();
            }
        }
        else {
            System.out.println("Dies ist keine gültige Frucht. Wähle zwischen Apfel, Birne, Pflaume & Kirsche!");
            Korb1();
        }
    }
    public void Korb1() {
        Scanner scanner = new Scanner (System.in);
        System.out.print("Wähle deine erste Frucht: "); 
        Frucht1 = scanner.next();  
        add();
    }
    public void Korb2() {
        if(Frucht2.equals("Apfel")) {
            if(meineAepfel.length > 0){
                meinKorb.apfelHinzufuegen();
                a = a - 1;
                meineAepfel = new Apfel[a];
                System.out.println("Apfel gewählt. Übrige Äpfel: " + meineAepfel.length);
            } else {
                System.out.println("Wähle eine andere zweite Frucht. Es gibt keine Äpfel mehr!");
                Korb1();
            }
        }
        else if(Frucht2.equals("Kirsche")) {
            if(meineKirschen.length > 0){
                meinKorb.kirscheHinzufuegen();
                k = k - 1;
                meineKirschen = new Kirsche[k];
                System.out.println("Kirsche gewählt. Übrige Kirschen: " + meineKirschen.length);
            } else {
                System.out.println("Wähle eine andere zweite Frucht. Es gibt keine Kirschen mehr!");
                add2();
            }
        }
        else if(Frucht2.equals("Birne")) {
            if(meineBirnen.length > 0){
                meinKorb.birneHinzufuegen();
                b = b - 1;
                meineBirnen = new Birne[b];
                System.out.println("Birne gewählt. Übrige Birnen: " + meineBirnen.length);
            } else {
                System.out.println("Wähle eine andere zweite Frucht. Es gibt keine Birnen mehr!");
                add2();
            }
        }
        else if(Frucht2.equals("Pflaume")) {
            if(meinePflaumen.length > 0){
                meinKorb.pflaumeHinzufuegen();
                p = p - 1;
                meinePflaumen = new Pflaume[p];
                System.out.println("Pflaume gewählt. Übrige Pflaumen: " + meinePflaumen.length);
            } else {
                System.out.println("Wähle eine andere zweite Frucht. Es gibt keine Pflaumen mehr!");
                add2();
            }
        }
        else {
            System.out.println("Dies ist keine gültige Frucht. Wähle zwischen Apfel, Birne, Pflaume & Kirsche!");
            add2();
        }
    }
    public void add2() {
        Scanner scanner = new Scanner (System.in);
        System.out.print("Wähle deine zweite Frucht: "); 
        Frucht2 = scanner.next();
        Korb2();
    }
}

Obst

public class Obst{
    public String farbe;
public Obst(String pFarbe){
    farbe = pFarbe;
    }
  
}

Kirsche, Apfel, Birne, Pflaume (Am Beispiel Kirsche)

public class Kirsche extends Obst{
    public Kirsche(){
        super("rot");
    }
}

Korb

public class Korb{
    private Apfel[] meineAepfel;
    private Kirsche[] meineKirschen;
    private Birne[] meineBirnen;
    private Pflaume[] meinePflaumen;
    public int a = 0;
    public int k = 0;
    public int b = 0;
    public int p = 0;
  
    public void apfelHinzufuegen() {
        a = a + 1;
        meineAepfel = new Apfel[a];
       
        for(a = 0; a<meineAepfel.length; a++){
            meineAepfel[a] = new Apfel();
        }
    }
    public void kirscheHinzufuegen() {
        k = k + 1;
        meineKirschen = new Kirsche[k];
        for(k = 0; k<meineKirschen.length; k++){
            meineKirschen[k] = new Kirsche();
        }
    }
    public void birneHinzufuegen() {
        b = b + 1;
        meineBirnen = new Birne[b];
        for(b = 0; b<meineBirnen.length; b++){
            meineBirnen[b] = new Birne();
        }
    }
    public void pflaumeHinzufuegen() {
        p = p + 1;
        meinePflaumen = new Pflaume[p];
        for(p = 0; p<meinePflaumen.length; p++){
            meinePflaumen[p] = new Pflaume();
        }
    }
 
  
   
}

Spieler

public class Spieler{
    private Spielbrett meinSpielbrett;
    private Korb meinKorb;
    public Spieler(Spielbrett pSpielbrett, Korb pKorb){
        meinSpielbrett = pSpielbrett;
        meinKorb = pKorb;
    }
    public void würfeln(){
        meinSpielbrett.würfeln();
        if (meinSpielbrett.getWürfel().seite2==5/*Rabe*/) {
            meinSpielbrett.getRabe().teilHinzufuegen();
        }
       
        else if (meinSpielbrett.getWürfel().seite2==0/*Kirsche*/) {
            //meinKorb.k = meinKorb.k + 1;
            meinKorb.kirscheHinzufuegen();
           
        }
       
        else if (meinSpielbrett.getWürfel().seite2==1/*Apfel*/) {
            //meinKorb.a = meinKorb.a + 1;
            meinKorb.apfelHinzufuegen();
        }
       
        else if (meinSpielbrett.getWürfel().seite2==2/*Pflaume*/) {
            //meinKorb.p = meinKorb.p + 1;
            meinKorb.pflaumeHinzufuegen();
        }
       
        else if (meinSpielbrett.getWürfel().seite2==3/*Birne*/) {
            //meinKorb.b = meinKorb.b + 1;
            meinKorb.birneHinzufuegen();
        }
        else if (meinSpielbrett.getWürfel().seite2==6/*Korb*/) {
            meinSpielbrett.add();
   }
}
}

Rabe

public class Rabe{
    public int puzzleteile;
    private String farbe;
    public Rabe(){
        farbe = "Schwarz";
        puzzleteile = 0;
    }
    public void teilHinzufuegen(){
        if (puzzleteile<9){
            puzzleteile++;
            System.out.println("Der Rabe hat aktuell " + puzzleteile + " Teile");
            if(puzzleteile==9){
                System.out.println("Der Rabe hat gewonnen!");
            }
        }
      
    }
    public int getPuzzleteile(){
        return puzzleteile;
    }
}

Würfel

import java.util.Random;
public class Würfel
{
    private String s1;
    private String s2;
    private String s3;
    private String s4;
    private String s5;
    private String s6;
    public int seite;
    public int seite2;
    public Würfel()
    {
      s5 = "Korb gewürfelt.";
      s6 = "Rabe gewürfelt.";
       
    }
    public void würfeln(){
        Random zufallsgenerator = new Random();
        int seite = zufallsgenerator.nextInt(6);
       
        seite2 = seite;
       
       
      
      
      if (seite==4){
            System.out.println(s5);
        }
        else if (seite==5){
            System.out.println(s6);
        }
      
      
       
    }
   
   
 
}

 

[MoxxiManagarm]

Huhu,

ich hoffe du hast nichts dagegen, wenn ich dir noch ein paar Tipps mit auf den Weg gebe:

1.

public class Würfel
public void würfeln()

Umlaute können Probleme bereiten. Bitte versuche von vornherein darauf zu verzichten.

2.

if(meineAepfel.length > 0){

Ich frage mich, wie die Länge deines Arrays 0 wird

3.

if(getWürfel().seite2 == 0/*Kirsche*/) {

- für mich ist total unklar was seite1 und seite2 sind. bzw ich denke, dass seite2 die gewürfelte Augenzahl ist, aber warum heißt das nicht sprechender? Es ist irritierend.
- anstatt integers mit Kommentaren zu versehen ist es ratsam sprechende static Werte beispielsweise zu verwenden oder mit der gewürfelten Zahl auf ein 2D-Array zugreifen.

4. Du hast generell viel duplizierten Code. Versuche diesen für alle Objekte zu generalisieren.

5. Mir erschließt sich nicht ganz wozu du Arrays von den Früchten für dein Spiel hast. Würde nicht ein Zähler alleine ausreichen?

6.

public void Korb1() {

Bitte schreibe Methoden in Java immer klein

7. Bitte verwenden hier im Forum für Java Code immer entsprechende Tags
[code=java][/code]


LG Moxxi

 

[Javinner]

[SPOILER="Open"][code=Java]Dein Code[/code][/SPOILER]
Wenn du noch Spoiler-Funktion benutzt, dann wird dein Post wesentlich kürzer.

 

[Javinner]

Mal ganz nebenbei: wie ist es eigentlich, wenn man das Spiel, nehmen wir als Beispiel gerade das hier, hier samt Code usw. veröffentlicht. Gibt es da irgendwelche Rechte, Paragraphen zu beachten? Wo erfahre ich, ob ein Spiel unter Copyright steht? Wo kann man sich da einlesen?

Sorry für solche Fragen, aber ich weiß es leider nicht besser.

 

[MoxxiManagarm]

Mein Wissensstand wär folgender, man darf mich korrigieren:
1. Eingetragene Namen dürfen nicht verwendet werden, es dürfte also ggf. nicht "Obstgarten" genannt werden
2. Die GUI darf nicht dem Original nicht nachempfunden werden, das Design ist also geschützt. Das Spielbrett darf also keine direkte Ähnlichkeit mit dem Original haben.
3. Auch sollte die Beschreibung oder Spielanleitungen nicht kopiert werden. Die Texte sind wie das Design geschützt.

Das heißt um wirklich Distanz in diesem Fall zum Original zu bekommen könnte man z.B. folgende Dinge tun:
- Benenne es anders und verwende andere Früchte. Wie wäre es z.B . mit Gemüse?
- Das Puzzle ist nicht wirklich grundlegend, das könnte alles Mögliche sein. Nimm statt dem Raben z.B. den bösen Max und fülle sein Sparschwein
- Im Original sind die 4 Bäume in einem Kreis angeordnet. Ordne sie in der Reihe an, mach aus den Bäumen Beete
- Das Original hat 4 Körbe, also quasi 4 Spieler. Belasse es bei einem.
- ...
--> Je mehr Abweichungen umso besser

Ansonsten wird sicherlich kein Vertreiber solche "Übungen" nachverfolgen. Wenn du dein Remake nicht kommerziell vetreibst wird es selten einen Kläger geben

 

[MoxxiManagarm]

Und hier mal eine Variante, wie es auf Modellebene zusammengekürzt aussehen könnte:

Spoiler

package javaforum.org.obstgarten;

import java.util.Random;
import java.util.Scanner;

public class Spiel {
    private Obstsammlung baeume;
    private Obstsammlung spieler;
   
    private int rabe;
   
    public Spiel() {
        baeume = new Obstsammlung(10, 10, 10, 10);
        spieler = new Obstsammlung(0, 0, 0, 0);
       
        rabe = 0;
    }
   
    public void spiele() {
        print();
       
        do {
            switch ((new Random()).nextInt(6)) {
            case 0:
                baeume.transferApfel(spieler);
                break;
            case 1:
                baeume.transferBirne(spieler);
                break;
            case 2:
                baeume.transferKirsche(spieler);
                break;
            case 3:
                baeume.transferPflaume(spieler);
                break;
            case 4:
                entscheide();
                print();
               
                entscheide();
                break;
            case 5:
                rabe++;               
            }
           
            print();
        } while(baeume.zaehle() > 0 && rabe < 9);
       
        System.out.println("Spiel vorbei");
    }
   
    private void print() {
        System.out.println("Spielfeld: " + baeume);
        System.out.println("Korb: " + spieler);
        System.out.println("Rabe: " + rabe);
        System.out.println("-----------------------------------");
    }
   
    private void entscheide() {
        Scanner scanner = new Scanner(System.in);
        System.out.print("Wähle 0-3: ");
        while(true) {
            switch(scanner.nextInt()) {
            case 0:
                baeume.transferApfel(spieler);
                return;
            case 1:
                baeume.transferBirne(spieler);
                return;
            case 2:
                baeume.transferKirsche(spieler);
                return;
            case 3:
                baeume.transferPflaume(spieler);
                return;
            default:
                System.out.print("Wähle nochmal: ");
            }
        }
    }
   
    public static void main(String[] args) {
        (new Spiel()).spiele();
    }
}

package javaforum.org.obstgarten;

public class Obstsammlung {   
    private int apfel;
    private int birne;
    private int kirsche;
    private int pflaume;
   
    public Obstsammlung(int apfel, int birne, int kirsche, int pflaume) {
        this.apfel = apfel;
        this.birne = birne;
        this.kirsche = kirsche;
        this.pflaume = pflaume;
    }
   
    public int zaehle() {
        return apfel + birne + kirsche + pflaume;
    }
   
    public void transferApfel(Obstsammlung sammlung) {
        if(apfel > 0) {
            apfel--;
            sammlung.apfel++;
        }
    }
   
    public void transferBirne(Obstsammlung sammlung) {
        if(birne > 0) {
            birne--;
            sammlung.birne++;
        }
    }
   
    public void transferKirsche(Obstsammlung sammlung) {
        if(kirsche > 0) {
            kirsche--;
            sammlung.kirsche++;
        }
    }
   
    public void transferPflaume(Obstsammlung sammlung) {
        if(pflaume > 0) {
            pflaume--;
            sammlung.pflaume++;
        }
    }
   
    @Override
    public String toString() {
        return "Äpfel: " + apfel + ", " +
                "Birnen: " + birne + ", " +
                "Kirschen: " + kirsche + ", " +
                "Pflaumen: " + pflaume;
    }
}