Iterator + Datamatiker + Lærer

public boolean hasNext() {
if(i < 0 || i >= list.size())
return false
else
return true
}

if(true) { return true; } else { return false; }

Windcape (1) skrev:

Implementering af en iterator i java. Af vores lærer på 3 semester af datamatiker i Århus.

Kig kode, og grin med.

public boolean hasNext() {
if(i < 0 || i >= list.size())
return false
else
return true
}

Først og fremmest kan det interne index 'i' aldrig blive 0.

Sludder... første element (såfremt i er en cursor) vil være 0.

Derudover bliver der lavet noget kode der kan forsimples til:

if(true) { return true; } else { return false; }

Ja... vis det! Forresten kan din ovenstående kode forsimples til

return true;

!

Det er til grin at en lærer ikke ved bedre.

NioBe (2) skrev:

Jeg havde en SQL lære der ikke kunne en skid SQL... Det endte med at vi alle fik en karakter på 7, fordi han viste ikke noget om SQL undtagen at finde de små kommandoer i en bog, for at vide hvad de betød/gjorte, uden selv at kunne forstå det...

if(i < 0) {
throw new IllegalStateException("Object can not be iterated in this state");
}

if(...) {
return true;
} else {
return false;
}

return ...;

arne_v (5) skrev:

Der er set langt langt værre kode ude i virkeligheden !

return (i > list.size())

if(true) {
return true;
}

Windcape (7) skrev:

Jeg synes at lave statements i stil med dette her viser at man ikke har særlig god styr på hvad der rent faktisk er logiske statements i ens kode.

if(true) {
return true;
}

Det bør virkelig ikke forekomme på en uddanelse.

Pally (8) skrev:

at underviseren blev sur for at høre om alternativer

Windcape (9) skrev:

Vores underviserer er generelt meget negative omkring alternativer til deres måder at gøre tingene på.

Hvilket de få af os som har erfaring før vi startede på uddanelsen, har det med at benytte os af.

Der har mange gange været direkte dum kode præsenteret af vores lærere, både uoverskuelig og dårlig kodet.

Jeg synes ikke at bruge 4 linjer til at skrive en linje på 10 tegn med er smart overhovedet. Folk skal lære at forstå hvordan en if sætning rent faktisk fungerere.

Pally (10) skrev:

...Men det konkrete eksempel mener jeg stadig er åben for diskussion. Jeg vil til enhver tid foretrække at review'e kode a la:

if (somethingSimple1)
return false;

if (somethingSimple2)
return false;
...
return true;


fremfor:
return (somethingVeryLongAndComplexWithLotsOfAndsAndOrs)

...
return cursorWithinBounds() && somethingElse());
...

private boolean cursorWithinBounds() {
....
}


private boolean somethingElse() {
....
}

Pally (10) skrev:

return (somethingVeryLongAndComplexWithLotsOfAndsAndOrs)

Corholio (11) skrev:

Jeg mener desuden også det er MEGET dårlig stil at have return statements i enkelt-line if-statements - det bliver næsten til spaghetti-kode.