Zufallszahlen

[El Hadji]

Servus Community,
Ich habe ein kleines Problem beim schreiben eines einfachen Programms. Und zwar möchte ich eine Urne erstellen in der Bälle mit 2 verschiedenen Farben (blau, rot) drinnen sind und dann ziehe ich Bälle, wenn ich einen Blauen gezogen habe soll der Ball nicht wieder zurückgelegt werden. Falls keine Bälle mehr in der Urne vorhanden sind soll null zurückgegeben werden. Hier mein Programm

import java.util.Random;

public class Urne1
{
    
    private int [] baelle;

    
    public Urne1(int anzahlRot, int anzahlBlau)
    {
        baelle = new int [2];
        baelle [0] = anzahlRot;
        baelle [1] = anzahlBlau;
        
    }

    
    public String nextBaell()
    {
        if(baelle [0] + baelle[1] == 0)
        return null;
        
        else if(baelle [0] == 0)
        {
            return "blau";
            baelle[1] = baelle [1]-1; // unreachable statement
        }
    
        
        else if(baelle [1] == 0)
        {
            return "blau";
            baelle[1] -= 1;
        }
        
        else
        { 
            Random r = new Random();
            if(r.nextInt(2) == 0)
            {
                return "rot";
                baelle[0] -= 1;
            }
            
            else return "blau";
            baelle[0] -= 1;
        }
    }
}

Un ich verstehe nicht warum ich als Fehlermeldung "unreachable statement" bekommen, wenn meine baelle[0] = 15 rote Bälle sind und ich nehme einen weg habe ich doch noch 14 ?!

 

[Gossi]

            else return "blau";
            baelle[0] -= 1;

!

baelle[0] -= 1;

Wird nicht erreicht, da du vorher definitiv ein return hast und damit die Methode beendet wird...

 

[lipox]

baelle[0] -= 1;

ist ein "unreachable statement".
Du hast vorher sowohl im if-zweig, als auch im else-zweig ein return. D.h. diese Zeile wird niemals erreicht.

 

[Landei]

Egal wie die Bälle verteilt sind, die Zeile [c]baelle[1] = baelle [1]-1;[/c] kann niemals ausgeführt werden: Selbst wenn die Bedingung [c]baelle [0] == 0[/c] erfüllt ist, wird der Block vorher über [c]return[/c] verlassen. Da das gewöhnlich nicht im Sinne des Erfinders ist, warnt dich der Compiler.

 

[Gossi]

Egal wie die Bälle verteilt sind, die Zeile [c]baelle[1] = baelle [1]-1;[/c] kann niemals ausgeführt werden: Selbst wenn die Bedingung [c]baelle [0] == 0[/c] erfüllt ist, wird der Block vorher über [c]return[/c] verlassen. Da das gewöhnlich nicht im Sinne des Erfinders ist, warnt dich der Compiler.

Nicht nur da, ich sehe grade, dass die Returns überall vor dem auszuführendem Code sind...

 

[El Hadji]

danke, stimmt danke für die aufklärung.
Also wenn ich das return nach hintenschiebe müsste es funktionieren?

 if(baelle [0] == 0)
        {
            baelle[1] = baelle [1]-1;
            return "blau";     
        }

 

[Gossi]

Normalerweise schon, ja ^^

 

[El Hadji]

Recht herzlichen Dank