🚀 go-pugleaf

TRAX wrote:
> sorry, sorry ik weet het, ik post nogal veel met vraag om hulp maar ik zit
> er hier al twe uur op te zoeken, met breakpoints en alles en ik vind het
> niet :(
> de code:
> 
> for (int i=0; i<zijde; i++){
>   for (int j=0; i<zijde; j++){
>     getal=random(19);
>     if (controletabel[getal]>=2){
>       do{
>        getal=random(19);
>        }
>       while(controletabel[getal]>=2);
>       controletabel[getal]++;}
>       else{
>       controletabel[getal]++;
>     };
>     tabel[i][j]=getal;
>   };
> };

Naast de opmerkingen van KLinZ: je kan die code wat simpeler schrijven, 
waardoor ze overzichtelijker wordt (toch zeker in mijn ogen):

for (int i=0; i<zijde; i++) {
   for (int j=0; j<zijde; j++) {
     do {
       getal=random(19);
     } while (controletabel[getal] >= 2);
     controletabel[getal]++;
     tabel[i][j]=getal;
   }
}

De while van een do ... while wordt dikwijls op dezelfde regel gezet als 
de }, omdat do ... while veel minder voorkomt dan een gewone while; op 
die manier is in een oogopslag gemakkelijker te zien dat de while bij de 
lus er voor hoort.
De waarde in de controletabel ophogen deed je in beide takken van het 
if-statement; als het toch in alle gevallen gebeurt, kan je het beter 
buiten de if zetten.
En tenslotte, na een '}' die dient om een compound statement af te 
sluiten moet je geen ';' zetten.

-- 
"Codito ergo sum"
Roel Schroeven
http://roelschroeven.net