Dead Code

[Voreck]

while (gefunden = false && index == matrikel.size() - 1) {
            if (matrikel.get(index) == matrkel) {
                gefunden = true;
            } else {
                index++;
            }
        }

warum wird

index == matrikel.size() - 1

als dead code angezeigt, obwohl ich index mit

index++;

erhöhe?

 

[stg]

Du hast hier eine Zuweisung:
gefunden = false

 

[Voreck]

oh danke
ich übersehe immer die leichtesten sachen

 

[CSHW89]

Deshalb bei Boolean am besten immer

gefunden
!gefunden

...statt...

gefunden == true
gefunden == false

...schreiben. Der Tippfehler '=' ist oftmals für den Compiler kein Problem, führt aber zur Laufzeit zu schwierig zu findenen Fehlern. Bei allen anderen Typen fällt das einfache '=' bei Bedingungen immer auf, da die Zuweisung dann nicht vom Typ Boolean ist.

lg Kevin

 

[Voreck]

Du hast hier eine Zuweisung:
gefunden = false

jetzt gehts, aber ich verstehe nicht, was das mit dead code zu tun hat

 

[Joose]

Du weißt der Variable "gefunden" den Wert "false" zu.
Wenn die Variable diesen Wert hat muss der hintere Teil der Bedingung (alles nach dem "&&") nicht mehr überprüft werden -> Toter Code da er nie überprüft/ausgeführt wird.

 

[Tobse]

BTW: Du prüfst, ob ein bestimmter Matrikel in deiner Collection drin ist. Mach das doch einfach mit Collection#contains:

boolean gefunden = matrikel.contains(matrkel);

UND:
Deine Variablen-Namen sind absoluter Käse: matrikel und matrkel. Das ist doch das selbe? Ah ne, beim zeweiten fehlt ein i. Nimm aussagekräftigere Namen und vermeide solche komischen Verkürzungen. Z.B.: matrikel und gesuchterMatrikel.