[JS] geen dubbele waarden

[Verwijderd]

ik wil 5 verschillende waarden genereren, hiervoor heb ik een random script en een for loop:

JavaScript:

function rand(nr) {
    return (Math.floor(Math.random() * nr + 1));
}

for (i=0;i<5;i++) {
    thenewx = rand(500);
    document.write(thenewx+'<br>');
}

Dat gaat goed. Alleen nu wil ik dat de waarden allemaal 50 verschillen van elkaar. Dus kwam ik op het idee om een array te maken, waarin de waarden worden opgeslagen. Elke keer voor hij een waarde opslaat, check hij of de waarde inderdaad 50 verschilt van alle andere opgeslagen waarden. Zo nee, dan moet hij een nieuw getal genereren, zo ja, dan gaat hij verder:

JavaScript:

function rand(nr) {
    return (Math.floor(Math.random() * nr + 1));
}

arrayx = new Array();

for (i=0;i<5;i++) {
    ok = 'no';
    while (ok == 'no') {
        thenewx = rand(500);        
        for (j=0;j<=arrayx.length; j++) {       
            thisx = arrayx[j];          
            if (thenewx > (thisx + 50) || thenewx < (thisx - 50)) {
                ok = 'yes';
                j = arrayx.length;
                arrayx[i] = thenewx;
                document.write(thenewx+'<br>');
            }
        }       
    }   
}

Alleen als ik deze code uit wil voeren, loopt mijn browser vast. Hij blijft dus ergens in een loop hangen denk ik, maar waar? Ik vermoed bij de laatste if statement. Of wat doe ik fout?

[RobIII]

Heb je dit al eens gelezen? Debuggen: Hoe doe ik dat?
Met een paar alertjes op strategische plekken moet dit prima te debuggen zijn

Waarom gebruik je eigenlijk 'no' en 'yes' en niet gewoon true en false

[ Voor 15% gewijzigd door RobIII op 24-04-2007 13:14 ]

[CodeCaster]

Waarom vul je niet gewoon een array met het aantal random waardes wat je wil hebben, check je de array op dubbelen, en vermenigvuldig je uiteindelijk alle getallen in de array met 50?

[Icelus]

RobIII schreef op dinsdag 24 april 2007 @ 13:14:
Heb je dit al eens gelezen? Debuggen: Hoe doe ik dat?
Met een paar alertjes op strategische plekken moet dit prima te debuggen zijn

Of installeer FireBug waarmee je stap-voor-stap door de code kan lopen.

[André]

Je moet eerst de array doorlopen om te matchen met alle bestaande waardes. Als dat gebeurd is kun je pas het nieuwe getal toevoegen

JavaScript:

function rand(nr)
{
  return (Math.floor(Math.random() * nr + 1));
}

var arrayx = new Array();

for (i=0;i<5;i++)
{
  var ok = false;
  while (ok == false)
  {
    thenewx = rand(500);        
    
    ok = true;
    for (j = 0; j <= arrayx.length; j++)
    {        
      thisx = arrayx[j];            
      if (thenewx < (thisx + 50) && thenewx > (thisx - 50))
      {
        ok = false;
      }
    }
    
    if (ok == true)
    {
      arrayx[i] = thenewx;
      document.write(thenewx + '<br>');
    }
  }    
}