Size: a a a

2021 July 07

В

Виталий in JS Liberty
@victoria_uranium  Вик, чекни плз, по возможности, на сколько сильно  хреново я реализовал ту задачку с подменом дома
https://codesandbox.io/s/2xoj0
источник

Е

Евгений in JS Liberty
help
источник

АК

Алексей Колмогоров... in JS Liberty
А сколько элементов в массиве?
источник

Е

Евгений in JS Liberty
all?
источник

АК

Алексей Колмогоров... in JS Liberty
И у тебя скорей всего ошибка на строке 51. Ты передаёшь arrayResult из глобальной области а не внутренний result.
источник

Е

Евгений in JS Liberty
не, такая же ошибка
источник

В

Виталий in JS Liberty
подебаж что тебе твой рандом возвращает
источник

АК

Алексей Колмогоров... in JS Liberty
Ну так пока основную проблему не решишь ошибка и будет, стэк переполняется из-за рекурсии. У тебя проблема с тем что ты используешь глобальные объекты, а не переданные. На строке 47 то же самое.
источник

Е

Евгений in JS Liberty
да, я убрал 51 строку и нет ошибки
источник

Е

Евгений in JS Liberty
спасибо, буду думать как по-другому делать
источник

АК

Алексей Колмогоров... in JS Liberty
Так ты изначальную задачу опиши.
источник

АК

Алексей Колмогоров... in JS Liberty
Судя по названию функции, тебе просто нужна случайная строка из массива. Но тогда тебе бы хватило только 42-ой строки.
источник

Е

Евгений in JS Liberty
Немного не правильно название. функция должна рандомно перезаписывать данные в другой массив
источник

Е

Евгений in JS Liberty
из all в result
источник

Е

Евгений in JS Liberty
а через random я сравниваю, если тру то там стираем туда записываем, если нет то вызываем опять
источник

Е

Евгений in JS Liberty
только я не знаю что делать если закончатся элементы
источник

АК

Алексей Колмогоров... in JS Liberty
А зачем рекурсия тогда?
источник

Е

Евгений in JS Liberty
я сам только что понял, что она бесполезна
источник

АК

Алексей Колмогоров... in JS Liberty
Покажи лучше пример исходного массива и полученного.
источник

Е

Евгений in JS Liberty
источник