Deadcode Warnung?

[acevik94]

Liebe Community,

Ich habe in meinem Programm eine Arrayliste und möchte mit einer String-Methode
die Elemente zurückgeben. Dazu habe ich folgenden Quellcode:

ArrayList<String> ownship1 = new ArrayList<String>();

String getOwnship1(){
    for(int i = 0; i < ownship1.size(); i++){
        return ownship1.get(i);
    }
return "";
}

Bei " i++ " bekomme ich deadcode Warnung. Wie kann ich diese Warnung fixen ?

PS :
Wenn ich

return "";

nicht schreibe bekomme ich diesen Fehler

This method must return a result of type String.

Danke im Voraus

acevik94

 

[MWin123]

return ownship1.get(i); gibt genau einen Wert zurück und bricht die Schleife ab.

 

[acevik94]

return ownship1.get(i); gibt genau einen Wert zurück und bricht die Schleife ab.

Eigentlich nicht . Wenn ich per Buttonclick Elemente in die Liste einfüge und die Liste ausgeben lasse, gibt die Methode alle Elemente aus.

Edit:
Also am Anfang ist die Liste leer. Die Elemente füge ich erst spaeter ein.

 

[Joose]

Eigentlich schon. Nach einem "return" wird die Methode verlassen und zum Aufrufenden Code "zurückgesprungen". (Da ein return den entsprechenden Wert an den aurufenden Code zurückliefert)
Sprich dein aktueller Code so ist sinnlos.

Du deklarierst i als Zählervariable für eine Schleife. Prüfst ob i < deiner Listengröße ist, wenn nicht wird die Schleife nicht ausgeführt und du brauchst natürlich einen return Wert (daher auch der Fehler wenn du "return "";" weglässt).
Sollte deine Liste Elemente beinhalten so wird einfach das erste Element (Index 0) zurückgegeben und die Methode ist beendet. Das "i++" kann so nie ausgeführt werden und ist somit "toter Code"

 

[acevik94]

Eigentlich schon. Nach einem "return" wird die Methode verlassen und zum Aufrufenden Code "zurückgesprungen". (Da ein return den entsprechenden Wert an den aurufenden Code zurückliefert)
Sprich dein aktueller Code so ist sinnlos.

Du deklarierst i als Zählervariable für eine Schleife. Prüfst ob i < deiner Listengröße ist, wenn nicht wird die Schleife nicht ausgeführt und du brauchst natürlich einen return Wert (daher auch der Fehler wenn du "return "";" weglässt).
Sollte deine Liste Elemente beinhalten so wird einfach das erste Element (Index 0) zurückgegeben und die Methode ist beendet. Das "i++" kann so nie ausgeführt werden und ist somit "toter Code"

Super . Danke dir. Jetzt klappts ^^