Почему в примере не правильно работает?

Question

[fanhypermax]

Вот в данном примере, не всегда цифры правильно выводятся, если max число десять бывает выводит 9 случайных цифр а не 10

Comments

[Negwereth]

Ваш код читать невозможно, он отформатирован отвратительно.

[fanhypermax]

Negwereth: Ч там читать отвратительно, кода та малый кусочик! Специально так сделал что бы уместился!

[Negwereth]

fanhypermax: куда уместился? Вы думаете, в браузеры прокрутку за отдельную доплату ставят?

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

[alex]

fanhypermax: ну сжали бы тогда его уж uglify'ем

[Negwereth]

fanhypermax: и, кстати, если интервал пользуете, то надо его очищать, а не в середине функции выполнение отменять.

[fanhypermax]

Ок я отступил

[fanhypermax]

Negwereth: Да это просто для демонстрации

[Negwereth]

fanhypermax: я не про пустые строки, а про форматирование вообще: как скобки расставлены, где блоки начинаются\заканчиваются

[Negwereth]

fanhypermax: https://jsfiddle.net/pum5he9o/9/
сравните по читабельности со своей версией

[fanhypermax]

Negwereth: Ок понял, спасибо конечно но почему - то неправильно выводит

[Negwereth]

fanhypermax: я в ответе отдельно указал на возможную причину.

Можете задачу озвучить?

[fanhypermax]

Negwereth: Что бы данные выводились, из массива и не повторялись

var fruits = ["Яблоко", "Апельсин", "Слива"];

fruits[RandomInt(0,fruits.length)];

[Negwereth]

fanhypermax: ок. А куки там зачем?
Решение такой задачи намного проще и банальней.

Вот пример:

"use strict";
const arr = ["A", "B", "C", "D", "E", "F", "G", "H", "I", "J"];

// Oneliner to obtain random value from range. This is an arrow function
const getIndex = (min, max) => Math.floor(Math.random() * (max - min + 1)) + min;

// Rolling random values till we get one not used before
const generateUniqueIndex = (storage, min, max) => {
	var index;
	while (index === undefined || storage.includes(index)) {
		index = getIndex(min, max);
	}
	return index;
};

function printRandom (arr) {
	const indexes = []; //Array to store used indexes
	const [min, max] = [0, arr.length - 1];

	// Iterating with random indexes but from 0 to array length
	for (let i = 0; i <= max; i++) {
		let index = generateUniqueIndex(indexes, min, max);
		console.log(arr[index]); // Do some action with active index
		indexes.push(index); // Save used index
	}
}

printRandom(arr);

[alex]

fanhypermax:

Что бы данные выводились, из массива и не повторялись

spoiler

function randomer(arr) {
	let array = arr.slice();
	let rnd = (min, max) => Math.floor(Math.random() * (max - min + 1)) + min;
	
	return () => array.length !== 0 ? array.splice(rnd(0, array.length - 1), 1)[0] : false;
}

const getRndItem = randomer(['Яблоко', 'Апельсин', 'Слива', 'Арбуз', 'Клубника']);

console.log(getRndItem()); // 'Слива'
console.log(getRndItem()); // 'Клубника'
console.log(getRndItem()); // 'Арбуз'
console.log(getRndItem()); // 'Апельсин'
console.log(getRndItem()); // 'Яблоко'
console.log(getRndItem()); // false
console.log(getRndItem()); // false

[kpa6uu]

Alex: красиво, чувак. Красиво!

Answer 1

[Negwereth]

Для начала - определитесь, где вызывать следующую итерацию, а то у вас и интервал, и рекурсия. В таком случае у вас пропуски итераций выходят.

Вот вам пример решения подобной задачи:

https://jsfiddle.net/babich_ss/rq83y3bx/

То же, но с интервальным исполнением:

https://jsfiddle.net/babich_ss/rq83y3bx/2/