Generate and sort Javascript Random Numbers

Question

I started to print 1000 random numbers between 1 to 100 and then sort all 1000 numbers.

Here something wrong happen .

How use Math.random() for it ?

        <script type="text/javascript" language="Javascript">
function pick(n, min, max) {
    var values = [],
        i = max;
    while (i >= min)
        values.push(i--);
    var results = [];
    var maxIndex = max;
    for (i = 1; i <= n; i++) {
        maxIndex--;
        var index = Math.floor(maxIndex * Math.random());
        results.push(values[index]);
        values[index] = values[maxIndex];
    }
    return results;
}

function go() {
    var running = true;
    do {
        if (!confirm(pick(1000, 1, 100).sort(function(a, b) {
                return a - b;
            }))) {
            running = false;
        }
    } while (running)
}
</script>
</head>

<body>
         <h1>100  random number between 1 and 100</h1>
         <p><button onclick="go()">Click me</button> to start generating numbers.</p>
          <p>When the numbers appear, click OK to generate another set, or Cancel to stop.</p>
    </body>

Your second loop (the for loop) runs from 1 to the value of "n". Each time through the loop, you decrement "maxIndex". Well, that starts off as 100, but "n" is 1000. — Pointy
– Pointy, Commented Nov 16, 2014 at 20:39

Roland Mai · Accepted Answer · 2014-11-16 20:44:24Z

1

If you really just want to generate 1000 random numbers and then sort them the following should be enough:

function pick(n, min, max) {
    var results = [];
    for (i = 1; i <= n; i++) {
        var value = Math.floor(Math.random() * max + min);
        results.push(value);
    }
    return results;
}

function go() {
    var running = true;
    do {
        if (!confirm(pick(1000, 1, 100).sort(function (a, b) {
            return a - b;
        }))) {
            running = false;
        }
    } while (running)
}

answered Nov 16, 2014 at 20:44

Roland Mai

31.2k8 gold badges58 silver badges61 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Kamal Over a year ago

if want 1 million random numbers then ?

Roland Mai Over a year ago

Technically you can change the parameter to 1000000 but it's going to be a bit slow. Javascript is not good with this sort of thing.

Collectives™ on Stack Overflow

Generate and sort Javascript Random Numbers

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related