🚀 go-pugleaf

Je kan ook het warning level verhogen naar 4
(project/settings/c-c++)

Dit soort fouten geeft dan netjes de melding:

warning C4706: assignment within conditional expression

Ter info, wij compileren altijd op het hoogste level. Hoogst zelden dat er
eentje over blijft ... na voldoende correct typewerk.

bvb: while(1) ... geeft C4706 op level 4

        for(;;) ... doet net hetzelfde zonder meldingen



Patrick Smout

"Haplo" <Wim.Valgaeren@p.b.invalid> schreef in bericht
news:MPG.197d126aa6eb9dc998968c@news.telenet.be...
> In article <vNwQa.10678$Yf3.823463036@hestia.telenet-ops.be>,
> slavearc@pandora.be says...
>
> <knip>
>
> >      if(lsChar = 'X') <- Hier is het probleem
> >      {
> >           lhCursor = AfxGetApp()->LoadStandardCursor(IDC_ARROW);
> >           SetCursor(lhCursor);
> >           OnOK();
> >      }
>
> Je doet hier een assignatie ipv een vergelijking. Vermits 'X'
> verschillend is van 0 geeft dit true terug en doe je de OnOK() waardoor
> de dialog afsluit.
>
> Om zulke fouten te vermijden draai je best zulke vergelijkingen om :
> Voorbeeld : if ('X' == lsChar)
>
> Als je nu 1 = teken zet ipv 2 krijg je een compileerfout, wat
> gemakkelijker te verbeteren is als een logische fout.
>
> Groeten,
>
> Haplo