2016-03-09 2 views
0

Я пытаюсь создать не повторяющиеся случайные числа между 1-5, 1-10, 1-20 и т. Д. Я нахожусь в Fisher-Yates Shuffle, но я не уверен, что я реализовал его наилучшим образом. Мой план состоит в том, чтобы связать каждое случайное число с заданным именем переменной. Я хочу убедиться, что синтаксис верен для назначения случайных значений предопределенным именам переменных. Я новичок в JavaScript и буду благодарен за понимание. Вот моя первая цветопередача:Случайное назначение повторного присвоения номера переменным

function shuffle(array) { 
 
    var i = array.length, 
 
     j = 0, 
 
     temp; 
 

 
    while (i--) { 
 

 
     j = Math.floor(Math.random() * (i+1)); 
 

 
     
 
     temp = array[i]; 
 
     array[i] = array[j]; 
 
     array[j] = temp; 
 

 
    } 
 

 
    return array; 
 
} 
 

 
var ranNums = shuffle([1,2,3,4,5]); 
 

 
var ranNum1 = ranNums.value;  
 
var ranNum2 = ranNums.value;  
 
var ranNum3 = ranNums.value;  
 
var ranNum4 = ranNums.value; 
 
var ranNum5 = ranNums.value;

+1

Если ваш код работает, и вы хотите, общее мнение о нем (а "коде обзор") , пожалуйста, разместите свой вопрос на http://codereview.stackexchange.com. Если ваш код не работает, отредактируйте свой вопрос, чтобы четко указать, что не работает, и любые сообщения об ошибках, которые вы получаете. – Matt

+3

Часть вашего вопроса, в котором говорится: «Я нахожусь в Fisher-Yates Shuffle, но я не уверен, что я реализовал его наилучшим образом», что делает его похожим на вопрос Code Review, но тот факт, что вы надеваете Не понимаю, как назначить отдельные значения массива переменным, поэтому сомнительно, что вы сами написали этот код, что является требованием для Code Review. – janos

+0

Вы правы, я не пишу эту версию рыбалки-ятсов, но я собираюсь ее использовать. Мои знания о JS отсутствуют, и я хотел удостовериться, что я правильно назначил переменные, я не был таким. Ответ предоставил работы, спасибо. – Bonedaddy

ответ

0

попробуйте использовать следующий код FIDDLE:

var ranNum1 = ranNums[0];  
var ranNum2 = ranNums[1];  
var ranNum3 = ranNums[2];  
var ranNum4 = ranNums[3]; 
var ranNum5 = ranNums[4];