Rückgabewert von Methode nie erreicht aber gefordert?

[Verjigorm]

Hallo, kurze Frage:

Ich habe eine Methode, die einen String zurückliefert.
Es gibt 6 mögliche Rückgabewert die in einer verschachtelten if-else-anweisung stehen und hier durch ein return beendet wird

Jetzt kommt als Fehlermeldung, dass die Methode ein return am Ende erwartet.
Nun gut, hab ich ein "return null" ans Ende gemacht.
Aber es ist durch die voranstehende if-else-anweisung unmöglich, dass dieses return jemals erreicht wird.

Ist das ok so oder sollte man den if-else-block so umbauen, dass man am Ende nur ein return hat ?

ich finde es so besser, da man dann genau sieht, wo die Methode beendet wird.
Was meint ihr?

mfg Verjigorm

 

[Wildcard]

Wenn es unmöglich ist das das return erreicht wird hast du ein if zuviel.

 

[Leroy42]

Wenn es unmöglich ist das das return erreicht wird hast du ein if zuviel.

Nicht unbedingt. Auch bei

int compareTo42(int x) {
  if (x < 42) return -1;
  if (x==42) return 0;
  if (x > 42) return +1;
}

kann der Compiler nicht erkennen, das auf jeden Fall
eine if-Bedingung erfolgreich ist.

 

[Verjigorm]

mal nen einfaches beispiel zum vergleichen
edit: Leeroy war schneller und sein Beispiel gefällt mir besser

...
int zahl = 4711;

if(zahl >10)
     return tueEtwasmitZahl(zahl);
else
     return tueEtwasAnderesmitZahl(zahl);

return ???;

oder

...
int zahl = 4711;

if(zahl >10)
     zahl = tueEtwasmitZahl(zahl);
else
     zahl = tueEtwasAnderesmitZahl(zahl);

return zahl;

 

[Wildcard]

Nicht unbedingt. Auch bei

int compareTo42(int x) {
  if (x < 42) return -1;
  if (x==42) return 0;
  if (x > 42) return +1;
}

kann der Compiler nicht erkennen, das auf jeden Fall
eine if-Bedingung erfolgreich ist.

Eben. Ein if ist zuviel. Die letzte Bedingung musst du nicht mehr prüfen.

 

[Leroy42]

Eben. Ein if ist zuviel. Die letzte Bedingung musst du nicht mehr prüfen.

Ach wirklich? :shock: Wie konnte mir das bloß entgehen! ???:L

Ich meinte natürlich nur, daß man solchen Code schreiben kann und
der Compiler damit an seine (derzeitigen) Grenzen stößt.

 

[Verjigorm]

ok, ich lasse einfach das letzte return weg und gib genau diesen Wert am Ende zurück