🚀 go-pugleaf

moi wrote:

> Dr.Ruud wrote:

>>>>     if (('F' == sex) || (('f' == sex) && (sex = 'F'))) do_female;
>>>>     else if (('M' == sex) || (sex = 'M')) do_male;
> 
> Mijn bezwaar hiertegen is, dat het veel-te-veel tokens zijn.
> Het *dwingt* de lezer om op de () te letten, zoals ook sommige
> construkties de lezer in de *illusie* brengen dat
> whitespace/indentering belangrijk is. Die belachelijke ('f' == gender)
> is *defensief*. 

Eens, een truuk om fouten op te vangen die je normaal met de compiler
hoort te zien. 

> Het voordeel van de switch is, dat ie makkelijk is, omdat iedereen
> hem onmiddelijk herkent. Bovendien *weet* iedereen dat er meer-dan 1 
> cases kunnen zijn (itt if/else, waarbij er altijd precies twee opties 
> zijn) Vandaar:
> switch(gender) {
> case 'm': gender = 'M';
> case 'M': do_gozert();
>    break;
> case 'f': gender = 'F';
> case 'F': do_wijffie();
>    break;
> default: do_teiltje("illegal gender: %x\n",  gender);
>      }
> 
> Iedere "getrainde" lezer zal hier een switch herkennen, zich
> realiseren dat fall-tru mogelijk is, en (wellicht bij tweede lezing)
> constateren dat dat ook daadwerkelijk (hopelijk) goed gebruikt wordt.
> IMHO kost de switch 2* zo weinig tijd *om te lezen* als de geneste 
> if-construktie. De rest van de tijd kan besteed worden aan begrijpen,
> wat 4* zo weinig tijd kost.

Bij langere zaken, in Perl, (maar ook in C hoor) gebruik ik een hash
table ipv. een lange switch. (Switch, Perl heeft geen switch, maar
gelukkig kan je op veel manieren er een maken, of je gebruikt de Switch
module :-) ). 

>> Je bent dan vast e1n van de velen die de "==" en "!=" niet echt als
>> symmetrische symbolen lezen. Ik zie veel liever een
>> if (NULL != lange-en-complexe-expressie)
>> dan andersom (en het liefst dus inclusief de onnodige "NULL !="),
>> want je weet meteen waarmee vergeleken wordt.
> 
> 
> Nee hoor. Ook hier kan je weer eea *weglaten*, waardoor de
> leesbaarheid verhoogd wordt: die "NULL !=" kan worden vervangen door
> "". Vergelijk:
> 
> while (NULL != (cp = fgets(fp, buff, sizeof buff))) {}
> 
> met:
> 
> while (cp = fgets(fp, buff, sizeof buff)) {}

Dit soort dingen kunnen fout gaan als cp zowel NULL als 0 kan worden,
vzviw. 

>>   unsigned int uiLen;
>>   int i;
> Variabelenamen om te HuiLen ....

Yup, Hungarian zou verboden moeten worden :-D.

> BTW leuk draadje, waar ging het over, eigenlijk ?

Iets met afvallen?

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