Повторяемость генерируемых чисел

[nikanor]

Добрый день.

Моя цель сгенерировать ряд случайных чисел.

//генерация случайных чисел от min до max
function generate_number(min, max){

   var temp_number = min + Math.floor((max-min)*Math.random()); //временный массив

   //проверка повторений
   for(i=0; i<mas_gen_number.length; i++)		
	  if(temp_number == mas_gen_number[i])
		 generate_number(min_number, max_number); //повторная генерация числа

	mas_gen_number [mas_gen_number.length] = temp_number; //сохранить число в массив

	return temp_number;
}

Несмотря на проверку, периодически встречаются повторяющие числа. В чем моя ошибка?

Edited March 8, 2010 by nikanor

[s0rr0w]

mas_gen_number [mas_gen_number.length] = temp_number; //сохранить число в массив

Не смотря на результат проверки, вы все равно сохраняете число в массив.

[nikanor]

Я же, в случае выявления дубля, повторно вызываю функцию раньше сохранения результата генерации в массив.

[s0rr0w]

Я же, в случае выявления дубля, повторно вызываю функцию раньше сохранения результата генерации в массив.

И все равно записываете предыдущий сгенерированный результат в массив.

[nikanor]

И все равно записываете предыдущий сгенерированный результат в массив.

Не понятно, почему

Ошибку исправил, может кому пригодиться:

//генерация случайных чисел от min до max
function generate_number(min, max){

   do {

	  var double_number = false; //повторения отсутствуют

	  var result = Math.floor(Math.random()*(max+1-min)+min); //генерация случайного числа

	  //проверка повторений
	  for(i=0; i<mas_gen_number.length; i++)
		 if(result == mas_gen_number[i])
			double_number = true; //повторения присутствуют
   }
   while(double_number);

   mas_gen_number.push(result); //сохранить число в массив

   return result;
}

[s0rr0w]

Не понятно, почему

Читаем написанный код

if(temp_number == mas_gen_number)

//Если случайное число равно хоть одному значению элемента массива

generate_number(min_number, max_number); //повторная генерация числа

//Однако, неважно, найдено число или нет, сгенерированно оно заново или нет, мы в любом случае добавляем число в массив.

mas_gen_number [mas_gen_number.length] = temp_number; //сохранить число в массив