Форма логина⁠⁠

Не, я не говорю, что чел вообще кодить не может. Но использовать eval в данном случае, это как перевозить бензин в ведрах, вроде и проще его туда заливать, но ведь опасно и неудобно, а решение с использованием канистры с лейкой не такое уж и сложное, и вроде даже очевидное. Куча if-else пофиг, специалисты по машинному обучению часто так делают, те кто пишет одноразовые скраперы, т.к. важен не код, а то, что он делает, если дальше его сопровождать не нужно.

В чем и суть, этот код не простой, он не прямолинейный, он странно замудренный, профи бы (да даже средний) бы в лоб решил бы его короче, а eval не использовал бы даже под веществами. Я не хочу обидеть автора данного кода, просто хочу, чтобы он любил свой код. Не надо вылизывать каждую строчку, но и сначала подумать алгоритм, а потом писать его. Всегда есть соблазн написать абы как, а потом в лучшем случае, на пикабу будут обсуждать твой код.

Меня возмутило то, что заминусовали того чела, хотя он хоть и в грубой форме, но написал то, что есть. Это критика, стоило в нее добавить конкретики, что я и сделал, и очень надеюсь, что автор ее принял без обид. Миру нужны хорошие программисты, тех, кто делает абы как, до ужасно много, что шутки про большое количество юристов переходит на айтишников. В работе ему часто придется делать аналогичные "демонстративные штуки", и он также будет иметь соблазн написать абы как - это не профессионально. Больше всего меня возмутило даже не это, а то, что люди начали защищать, и говорить, что это нормальный код, начали писать про то, что можно было конечно в одну строчку сделать (что еще хуже), вместо чем нормально написать в чем проблема. Потом другие новички это прочитают, и намотают на ус.

Не, ну остальное терпимо для демонстрационного проекта, но eval то зачем? Выкинь и забудь. В крайнем случае new Function, которая возвращает нужный элемент.

минифицирован

Это нет, не важно, это демонстрация.

Ну там можно было что то типа:

$(el).attr("class","eye-" + len);

или

el.style.left = (25 * len) + "px";

Но как уже писали, это все было for fun и не характеризует программиста.

Быдлокод это когда запрос к БД в цикле, каким бы коротким и красивым этот код не был.