🚀 go-pugleaf

Edwin Martin wrote:

> Dr.Ruud wrote:
> 
>> Zo zijn er ook coders die dit soort 'academische truken'
>> niet graag zien:
>> 
>>   switch(sex) {
>>       case 'm':
>>       default:
>>           sex = 'M';
>>       case 'M':
>>           do_male();
>>           break;
>>       case 'f':
>>         sex = 'F';
>>       case 'F':
>>         do_female();
>>   }
> 
> En terecht. Die is zeer foutgevoelige code. Iemand die een 
gebruikelijke 
> switch-case verwacht kan iets veranderen waardoor het fout gaat.

Die snapt dan fall through niet. Ik gebruik voor de duidelijkheid iets 
als:

   switch(sex) {
       case 'm':
       default:
           sex = 'M';

    	     /* fall through */
       
       case 'M':
           do_male();
           break;
       case 'f':
         sex = 'F';

    	    /* fall through * /

       case 'F':
         do_female();
   }

Uiteraard zou ik bovenstaande zo niet programmeren (maar iets als):

if ( sex == 'F' || sex == 'f' ) {

    	sex = 'F';
    	do_female();

} else {

    	sex = 'M';
    	do_male();
}


> Nu kun je wel zeggen dat die persoon beter moet uitkijken,

Zeker, want die zal vlug fouten maken. Uiteraard is zo'n blokje code 
gedocumenteerd.

> maar goede 
> code schrijven is makkelijker dan bij elke wijziging eerst de hele 
code 
> doornemen.

Goede keuze van namen e.d. helpt. Af en toe een hint als documentatie 
wat meer.

> Kwaliteitscode is code die gemakkelijk onderhouden en uitgebreid kan 
> worden. Alleen als de code heel snel of heel klein moet zijn, zou je 
> daarvan kunnen afwijken.

Hangt helemaal van het probleem af wat opgelost moet worden. Soms moet 
je lelijk programmeren om code sneller te krijgen.

> Betere code is:
> 
> // Maak sex-variabele uppercase
> // nog makkelijker is toupper() gebruiken
> switch (sex) {
>      case 'f':
>           sex = 'F';
>           break;
>      case 'm': // Hoeft eigenlijk niet
>      default:
>           sex = 'M';
>           break; // Hoeft eigenlijk niet
> }
> 
> // Voer sexe-afhankelijke code uit
> // Kan ook met een if()-statement
> switch (sex) {
>      case 'F':
>           do_female();
>           break;
>      case 'M':
>      default:
>           do_male();
>           break;
> }

Dit verdient zeker geen prijs, zie boven :-D.

-- 
John Bokma
                 Voorbeeldscripts in Perl: http://johnbokma.com/perl/
Ervaren Perl / Java programmeur beschikbaar:  http://castleamber.com/
Tevreden opdrachtgevers:     http://castleamber.com/testimonials.html