Return in If

[Rantanplan69]

Hallo,
ich habe ein Problem, ich will ein einer boolean methode in verschatelten ifs true zurückgeben.
Jedoch sagt mir der Compiler, dass er ein return Wert haben will.
Also habe ich außerhalb der Klammer ein return false gesetzt.
Ich weiß aber nicht ob das dann erreicht wird, was ich haben will.
Denn wenn die if Bedingungen erfüllt werden, sendet er true zurück und dann am Ende der methode wieder false

    boolean win1(){
    
    if (f.a[0]==1){
    	if(f.a[1]==1){
    		if(f.a[2]==1){
    			System.out.println("Player 1 wins");
    			return true;
    		}
    	}
        if (f.a[0]==2){
        	if(f.a[1]==2){
        		if(f.a[2]==2){
        			System.out.println("PC wins");
        			return true;
        		}
        	}
        }
    }
    return false;
    }

LG
Leider bin ich noch nicht so weit zu testen, ob das funktionieren würde, daher die Frage.
Ich bezweifle es eher auch, also wie kann ich das Problem lösen

 

[bone2]

Es ist unlogisch/falsch geschachtelt, abgesehen advon bekomme ich keinen return hinweis mit dem code.

 

[Kevin94]

Mal abgesehen davon, dass du einen Fehler in deiner Klammernsetztung hast, würde der Code genau das tun was er soll. Eine Methode kann nicht "wieder" etwas anderes zurückgeben. Wenn die Bedingungen in den if's erfüllt sind, wird true zurückgegeben und die Methode ist beendet, wenn keine der Bedingungen zu trifft, wird false zurück gegeben. das ist auch der Grund warum am Ende ein return stehen muss.

Du kannst deinen Code übrigens auch vereinfachen:

boolean win1(){
    if (f.a[0]==1 && f.a[1]==1 && f.a[2]==1){
        System.out.println("Player 1 wins");
        return true;
    }
    else if (f.a[0]==2 && f.a[1]==2 && f.a[2]==2){
        System.out.println("PC wins");
        return true;
    }
    else {
        return false;
    }
}

 

[bone2]

das else{} für return false ist natürlich überflüssig.

        if (f.a[0] == 1 && f.a[1] == 1 && f.a[2] == 1) {
            System.out.println("Player 1 wins");
            return true;
        }
        else if (f.a[0] == 2 && f.a[1] == 2 && f.a[2] == 2) {
            System.out.println("PC wins");
            return true;
        }
        
        return false;

@TE, bei einem return verlässt er die methode. keine einzige zeile danach wird noch verarbeitet. (abgesehen von try-finally)

probier mal nach return false noch code zu schreiben, dann bekommst du den hinweis auf unreachable code.

 

[Kevin94]

Das else ist nicht notwendig, aber erlaubt. Und imho erleichtert es das Verständis für die Funktionsweise, was in diesem Fall ja das Problem ist.

 

[SlaterB]

das else{} für return false ist natürlich überflüssig.

schreibt wer der das dann korrigiert, aber das genauso überflüssige 'else if' beläßt

für die Übersicht sollte es auch nicht so viel helfen wenn man die ifs mit return-Abbruch erfassen kann,
aber geht natürlich

 

[bone2]

schreibt wer der das dann korrigiert, aber das genauso überflüssige 'else if' beläßt

Jetzt wo ich so drüber nachdenke

 

[hüteüberhüte]

Entweder:

        if (f.a[0] == 1 && f.a[1] == 1 && f.a[2] == 1) {
            System.out.println("Player 1 wins");
            return true;
        }
        if (f.a[0] == 2 && f.a[1] == 2 && f.a[2] == 2) {
            System.out.println("PC wins");
            return true;
        } 
        return false;

oder (wenn in f.a nur 1 oder 2 steht):

Arrays.sort(f.a);
return Arrays.binarySearch(f.a, 1) < 0 || Arrays.binarySearch(f.a, 2) < 0;

 

[Landei]

Eine Methode kann nicht "wieder" etwas anderes zurückgeben.

Doch:

    public static boolean changedMyMind() {
        try {
            return true;
        } finally {
            return false;
        }
    }

 

[SlaterB]

> oder (wenn in f.a nur 1 oder 2 steht):

eine wohl zu kühne Annahme, aber zum Spass dann noch als Vorschlag:

return f.a[0] == f.a[1] && f.a[1] == f.a[2];

mit Sortierung Umstellung der Felder ist vielleicht auch nicht so angebracht

 

[piu58]

Das Problem, was der Compiler bemerkt ist, dass nicht bei allen Datenkonstellationen ein Wert zurückgegeben wird. Das muss die Methode aber so haben. Für dich ist es logisch, dass einer der beiden Spieger gewinnen muss, also einer der beiden Zweige betreten werden muss. Wenn du das mit else formulierst (falls möglich), dann sieht das der Compiler auch. Es sind aber rein mathematisch Datenkonstellationen denkbar, wo das Programm einfach bis zum Ende durchläuft, z.B. bei Spieler-N° 3.