C#: dummy tips: IF og lignende uten { og }

[HDSoftware]

ARGH!! Ble skikkelig lurt her av koden min. Vil bare nevne det for dere slik at dere ikke går i samme fella. Utrolig vanskelig å skjønne at det er et problem før man virkelig går over koden step'by'step

 

i sin absolutt enkleste form:

if (verdi1 > verdi2)
 if (verdi1 < verdi3)
    throw new exception("Noe gikk galt");
 return false;
return true;

 

Denne koden vil aldri returnere TRUE!!!

 

Mange vil si "Så klart ikke", inklusivt meg, men jeg gikk rett i den fella i dag og det er utrolig fort gjort å gjøre denne blemma....

 

Såh - bruk alltid krøllparanteser! UANSETT!!!

if (verdi1 > verdi2)
{
 if (verdi1 < verdi3)
 {
   throw new exception("Noe gikk galt");
 }
 return false;
}
return true;

 

Altså - bare et tips til deg som lurer på hvorfor rare ting skjer i IF testene....

[Gjest Slettet+56132]

Først og fremst; C# bruker krøllparanteser/braces til stort sett all form for markering av funksjoner og alt mulig, så å glemme dem i et språk som er "bygd rundt dem" er bare smått teit. Sorry altså, men jeg synes det blir for dumt å komme med et dummytips som sier at du alltid må huske på {}, det er i grunn helt innlysende fra alle kodeeksempler og tutorials og bøker.

[Degeim]

Det han mener er at det er lett å gå i en felle hvis du sløyfer dem på de områdene der du faktisk kan sløyfe dem (feks der du bare har én linje med kode inne i en if-blokk eller for-loop).

 

if(done)
    return true;

//do something
return false;

 

Jeg for min del elsker å sløyfe dem alle steder jeg kan; auto-indenteringen til VS hjelper meg som regel å holde styr på hva som går innunder hvilken if/for hvis jeg ikke selv klarer det.

Endret 25. mars 2010 av Degeim

[GeirGrusom]

if a
 if b
   return true;
 else if c
   break;
 else
   if d
     return false;
   else
     if e
       return false;
     else if d
       return true;
else 
 return true;

Her er det en felle på siste linje.

[HDSoftware]

Degeim: Helt korrekt anntatt ;-)

[Gjest Slettet+56132]

Ah, daså. Misforstå alt altså. Hehe, beklager