In Safari and I think also IE7 and 8 Math.random() is NOT random?

Question

http://koreanwordgame.com/

This page first loads 4 words into the options DIVs via Ajax and then randomizes the correct answer with the following function, passing the D

Answer 1

Part of the problem you're having is how you're using the random function. The sort function takes another function that is used to compare any two elements in the list. One of the assumptions made by the sort function is that the results of the comparison between any two elements is both stable and consistent (i.e., if a < b and b < c, then a < c), and you're function violates that assumption. So, there's really no guarantees as to what the sort function will do in this case. Depending upon how the sort function is implemented (I don't know which algorithm is used in javascript off hand), you will most likely not get a truely random order where each item is equally likely to appear in any given position.

The simplest solution I can think of is to generate a random number for each item you want to sort, and then sort the items by that randomly generated number. This is different from your approach in that comparing any two elements in your list will always produce the same result while the items are being sorted.

Answer 2

The problem isn't Math.random(). It's your "randomizing" sort function, which only ever returns 0 or 1, and never -1. Here's how to properly generate a random int in the interval [-1. 1], based on MDC's getRandomInt function:

Math.floor(Math.random() * 3) - 1;

(simplified from getRandomInt(-1, 1)).

---

That said, as @32bitkid commented, the right way is to use a Fischer-Yates shuffle.

Answer 3

return( temp%2 ); Leaves you with only 0 or 1

Try using return( temp%4 ); Which would leave you with 0,1,2,3

Although I am not sure what the modulus is for.
This will get you a random number between 0 and 3:

Math.floor(Math.random()*4)

So all you need to do is:

var random = function(r){
    r.children().sort(function(a,b){
        return Math.floor(Math.random()*4);
    }).appendTo(r);            
};