Почему алгоритм работает некорректно?

Question

[Ivan S. Glushko]

Здравия.
Ума не приложу, почему не работает корректно алгоритм.

var arr, max_number, i, unic;
arr = [];
max_number = 42;
while (arr.length < 42) {
  do {
    unic = true;
    a = Math.floor(Math.random() * 42);
    for (i = 0; i < arr.length; i++) {
      if (a == arr[i]) {
        unic = false;
        break;
      }
    }
  } while (!unic) { 
     a = Math.floor(Math.random() * 42);
  }
  arr.push(a);
}

Задача заполнить массив случайными уникальными числами. В итоге массив "arr" заполняется местами одинаковыми числами. Заранее благодарю за попытку предложить варианты перемешивания массива, но увы: вопрос обставлен иначе.

Answer 1

[Евгений Петров]

Про неверный синтаксис do while уже написали.
Я бы делал примерно так

var x = set(42);

console.log(x);

function set (max_number) {
	var array = [];

	set.unic = [];

	while (array.length < max_number) {
		array.push(get());
	}

	delete set.unic;
	return array;

	function get (a) {
		do {
			a = Math.floor(Math.random() * max_number);
		} while (isFinite(set.unic[a]));

		return a;
	}
}

Answer 2

[index0h]

Ужос. Где var перед переменной a? Зачем вам явный тройной цикл, есть же indexOf

var data = [],
    count = 42,
    tmpVar;
while (data.length < count) {
    tmpVar = Math.floor(Math.random() * count);
    if (data.indexOf(tmpVar) === -1) {
        data.push(tmpVar);
    }
}

Answer 3

[Сергей]

var arr, max_number, i, unic;
arr = [];
max_number = 42;
while (arr.length < 42) {
  do {
    unic = true;
    a = Math.floor(Math.random() * 42);
    for (i = 0; i < arr.length; i++) {
      if (a == arr[i]) {
        unic = false;
        break;
      }
    }
  } while (!unic);
  arr.push(a);
}

Вы сначала находили в цикле do уникальное значение, а потом присвоили переменной a новое значение после цикла.

Answer 4

[Kano]

Не могу понять от куда у вас могла взяться приписка после второго while

{ 
     a = Math.floor(Math.random() * 42);
  }

Уберите этот код и объявите переменную в начале кода через var a; а то судя по коду вы используете глобально определенную переменную "а" или же объявляете её глобально, а это не очень правильный подход