Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Ошибки в тестовом задании (https://javascript.ru/forum/project/72249-oshibki-v-testovom-zadanii.html)

KonstBessonov 18.01.2018 07:00

Ошибки в тестовом задании
 
Решил я тут сменить специализацию и уйти в веб-программисты. Подучил JS, походил по сайтам, обратился в одну из организаций, дали тестовое задание на джуниора - сделал. Обратно пришёл отказ без объяснения причин.

Может кто-нибудь посмотреть, рассказать про грубые ошибки?
https://github.com/KonstBessonov/Uchi.ru-test

FAULT 29.01.2018 08:06

Видимо потому что работал не через JQuery :dance:

Царь Леонид 29.01.2018 19:13

Я не претендую истину в высшей инстанции, я простой верстальщик) В целом норм, но
1. Это одна большая простынюка. Я бы разбил по файлам и всю логику спрятал, наружу бы вынес метод который на входе принимал бы 2 числа
2. Мало комментариев, много магии. Читать сложно
Отказывают без мотивации очень нехорошие люди, которые не ценят чужой труд и время, так что это к лучшему, что не взяли)

KonstBessonov 30.01.2018 14:34

Понял, спасибо. будем тренироваться.

Nexus 30.01.2018 14:51

Строка 3.

Функций "stage[\d]" можно было избежать, пропала бы и необходимость в приватных функциях "on[EventName]" контроллера.

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

KonstBessonov 04.02.2018 14:07

Цитата:

Сообщение от Nexus (Сообщение 476640)
Функций "stage[\d]" можно было избежать, пропала бы и необходимость в приватных функциях "on[EventName]" контроллера.

С "stage\n" не понял немного. Там в 1 и 2, конечно, логика похожа, но в 3 и 4 совсем другая. С oninput/onkeypress в приницпе да, может и не стоило их использовать - всё-таки не рекомендуется.

Цитата:

Сообщение от Nexus (Сообщение 476640)
Методы для всех функций не добавляются в прототип, т.е. для каждого отдельного экземпляра класса в памяти будет храниться своя реализация каждого метода (если не ошибаюсь. Не ошибаюсь).

Ну, на файлы я код разбил, комментарии прописал. Следующим шагом перепишем на прототипный стиль.


Часовой пояс GMT +3, время: 16:09.