🚀 go-pugleaf

In article <vNwQa.10678$Yf3.823463036@hestia.telenet-ops.be>, 
slavearc@pandora.be says...

<knip>

> 
> void CMouseDlg::OnKeyDown(UINT nChar, UINT nRepCnt, UINT nFlags)
> {
>      char lsChar;  //ingedrukt key
>      HCURSOR lhCursor; //handle naar af te beelden cursor
> 
>      lsChar = char(nChar); //cast key in char
> 
>      if(lsChar == 'A')
>      {
>           lhCursor = AfxGetApp()->LoadStandardCursor(IDC_ARROW);//get cursor
> handle
>           SetCursor(lhCursor); //zet cursor
>      }
>     if(lsChar == 'B')
>      {
>           lhCursor = AfxGetApp()->LoadStandardCursor(IDC_IBEAM);
>           SetCursor(lhCursor);
>      }
>      if(lsChar == 'C')
>      {
>           lhCursor = AfxGetApp()->LoadStandardCursor(IDC_WAIT);
>           SetCursor(lhCursor);
>      }
>      if(lsChar = 'X')
>      {
>           lhCursor = AfxGetApp()->LoadStandardCursor(IDC_ARROW);
>           SetCursor(lhCursor);
>           OnOK();
>      }
>      else
>      {
>           SetCursor(lhCursor);
>      }
> 
> 
>      CDialog::OnKeyDown(nChar, nRepCnt, nFlags);
> }

Update: Ik heb ook eventjes naar de rest van de code gekeken en er zijn 
nog een paar kleine opmerkingen.

- Gebruik 1 groot "if-else if" blok ipv afzonderlijke if's. Hierdoor 
wordt de code leesbaarder en vermijd je onnodige vergelijkingen
- Vergelijk nChar met de echte Virtual Key Codes (VKEY_A, VKEY_B, ...) 
ipv van de cast naar char. Nu komt dit toevallig uit maar dit is niet 
zeker voor andere keys.
- Verplaats de SetCursor na het "if-else if" blok, omdat dit in elke if 
voorkomt en dus niets met de if te maken heeft. 
- Als je intensief van die paar Cursors gebruik gaat maken is het 
optimaler van ze in static const variabelen te steken en die dan te 
gebruiken. Zo vermijd je dat ze steeds geladen worden bij elke onKey
- In het else-blok doe je een SetCursor van een mogelijk ongedefinieerde 
HCURSOR. Dit zou ook wel eens een crash kunnen veroorzaken.

Groeten,

Haplo