Как создать массив со случайными числами без повторений?

Question

[tolik12345]

Добрый вечер.

Стоит задача создать массив с 5-ю случайными, неповторяющимеся числами от 1 до 10. Элементы в массиве должны быть в рандомном порядке.

Мое решение https://codepen.io/tolik12345/pen/pooboaE?editors=0012

Вроде как работает но меня смущает while... подскажите насколько такой подход правильный и может есть более разумный способ реализовать эту задачу?

Comments

[dollar]

В каком смысле правильный? Если у вас 5 чисел, то любой подход правильный.
А если нужен массив из 5 миллионов чисел, то уже можно подумать.
Оптимизация зависит от того, где слабое место, а на малых числах пофиг на оптимизацию.

[xmoonlight]

dollar, о чём думать?) (хоть 5, хоть 10 млн - числа же все разные и брать нужно неповторяющиеся!)

[dollar]

xmoonlight, о том, как создать массив со случайными числами без повторений

[xmoonlight]

dollar, ну как? splice() :)

Answer 1

[Сергей Соколов]

создать массив допустимых значений и «вынимать» из него случайные.

const values = [...Array(10)].map((_, i) => i);
const result = [...Array(5)].map(() => values.splice(Math.floor(Math.random() * values.length), 1)[0]);
result // [ 2, 5, 3, 4, 1 ]

Comments

[tolik12345]

создать массив допустимых значений

Создал массив из допустимых значений https://codepen.io/tolik12345/pen/XWWKWoV?editors=0011
теперь нужно дергать из него элементы по индексу в новый массив пока длина исходного массива не будет равна 0 ?

[tolik12345]

все вижу ответ

Answer 2

[Developer]

while(true){
	if (arr.length == 5) break;

Засунь условие из if в while. Естественно, с модификацией

Comments

[tolik12345]

Простите вот этого не понял

Естественно, с модификацией

?

[Developer]

tolik12345, Нууу....
While - это тот же if, но встроенный в цикл. Разве не так?

[tolik12345]

Developer, все, понял, спасибо.

Answer 3

[Алексей]

const array = Array.from({ length: 20 }, (v, i) =>
  Math.floor(Math.random() * (10 - 1) + 1 ));
const set = new Set(array);
const result = [...set].slice(0, 5);
console.log(result);

или так

const array = Array.from({ length: 10 }, (v, i) => i)
  .sort((a, b) => 0.5 - Math.random())
  .slice(0, 5);
console.log(array);

Answer 4

[Олег]

https://stackoverflow.com/questions/2450954/how-to...

возьмите массив [1,2,3, .. ,10]
перемешайте . возьмите из него первые 5

Answer 5

[coderisimo]

уот так уот! ))