Будет ли eval() корректно работать?

Question

[TosterUserName]

Написал функцию myFunc, где в зависимости от значений инпутов формируется условие, записывается в строку и подставляется в if через eval() чтобы показать подходящие элементы массива (уверен, это более хитро делается, но могу только так). Прочитал, что этот eval() плохая практика и его лучше избегать, но ничего не понял. Подскажите пожалуйста, будет ли он в моем случае корректно работать?
https://codepen.io/mycodepenio/pen/xWrVVe

Answer 1

[Андрей Александров]

Работать будет. ;-) сейчас же работает.
Но можно не уследить в процессе правок кода и злоумышленник сможет выполнять у вас на сайте произвольный код. Потому и считается использование eval() плохим способом.
Да и логика теряется, сложно разбираться в том, что код делает.
И если сейчас вы помните, то например лет через 5 уже будет проблемно понят, что и зачем.

Кто мешает сохранить результаты в ещё один массив, вместо формирования "хитрой" строки,
а затем уже сравнивать его внутри цикла исходного массива. B-)

Comments

[TosterUserName]

Спасибо, что откликнулись. Добавляю значения в массив, чтобы потом можно было сравнивать, но как цикл в цикле сделать, чтобы сравнивал в нужном порядке не пойму. Может там еще третий нужен? Подскажите как сравнивать его внутри цикла исходного массива?
https://codepen.io/mycodepenio/pen/yKoegO

[Андрей Александров]

Зачем push то использовать?! С ним у вас никакого контроля куда записать.
Можно же просто написать:
conditionFull[inputName] = inputValue;
inputName лучше оставить как было, т.к. input'ы могут и перепутаться местами в HTML в процессе, и всё равно они у вас цифровые.

А циклы, так всё же уже было, просто в имеющийся добавить цикл по полученному массиву, как-то так:

for (var key in mass) {
	var r = (mass[key][0] >= conditionFull[0]);//compare result
	for (var c=1,l=conditionFull.length;c<l;c++){
		r = r && (mass[key][c] >= conditionFull[c]);
	}
	if (r){
		//addKey.className = "podoshlo";
	}
}

[Андрей Александров]

И да conditionFull надо хоть нулями инициализировать, или какие там условия сравнения.
И условия сравнивать не через for, т.к. некоторые слайдеры могут быть выключены, а через forEach, наверное.

[TosterUserName]

Андрей Александров, Спасибо, буду соображать)

[TosterUserName]

Короче, как-то так вроде работает, пусть будет, кода больше, но я хоть могу его понять
https://codepen.io/mycodepenio/pen/JLywoN