Java return in Schleife

[Overskill]

Wieso wird hier angezeigt ich solle ein return hinzufügen? Wird nicht in jedem Fall etwas zurück gegeben?

	public int suchefor(int[] x, int y) {
		int z=0;
		
		for(int x1=0; x1<x.length; x1++) {
			if (y==x[x1]) {
				z=x1;
				return z;
			} else {
				return -1;
			}
		}

 

[Flown]

Nicht wenn das Array die Länge 0 hat.

Also nimm den else-Zweig weg und mach am Ende der Methode dein

 return -1;

 

[Gucky]

Für den Compiler ist es nicht klar, dass überhaupt in die Schleife gesprungen wird und deshalb möchte er, dass du noch ein return einbaust. Besser wäre es, wenn du den Rückgabewert in eine Variable schreibst und diesen am Schluss der Methode zurückgibst.

Das ganze Konstrukt macht für mich keinen Sinn. Wozu machst du eine Schleife, die nur maximal ein mal durchlaufen wird und die Inkrementierung der Laufvariable könntest du dir auch sparen.

 

[Sorainthy]

Das ganze Konstrukt macht für mich keinen Sinn. Wozu machst du eine Schleife, die nur maximal ein mal durchlaufen wird und die Inkrementierung der Laufvariable könntest du dir auch sparen.

DAS habe ich mich auch gefragt. Der gesamte Else Part macht in der For-Schleife keinen Sinn und führt zu diesem Fehlerverhalten.

 

[Thallius]

Ich denke das Ergebnis was der TO erwartet ist, dass er nur -1 zurück bekommt wenn y nicht im x Array entahlten ist. Das dem nicht so ist, wird er beim Ausprobieren vielleicht noch feststellen, wenn die App erstmal ohne Fehler compiliert

Gruß

Claus