Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 18.01.2018, 07:00
Новичок на форуме
Отправить личное сообщение для KonstBessonov Посмотреть профиль Найти все сообщения от KonstBessonov
 
Регистрация: 18.01.2018
Сообщений: 7

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

Может кто-нибудь посмотреть, рассказать про грубые ошибки?
https://github.com/KonstBessonov/Uchi.ru-test
Ответить с цитированием
  #2 (permalink)  
Старый 29.01.2018, 08:06
Новичок на форуме
Отправить личное сообщение для FAULT Посмотреть профиль Найти все сообщения от FAULT
 
Регистрация: 29.01.2018
Сообщений: 2

Видимо потому что работал не через JQuery
Ответить с цитированием
  #3 (permalink)  
Старый 29.01.2018, 19:13
Профессор
Отправить личное сообщение для Царь Леонид Посмотреть профиль Найти все сообщения от Царь Леонид
 
Регистрация: 22.08.2013
Сообщений: 217

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

Последний раз редактировалось Царь Леонид, 29.01.2018 в 19:15.
Ответить с цитированием
  #4 (permalink)  
Старый 30.01.2018, 14:34
Новичок на форуме
Отправить личное сообщение для KonstBessonov Посмотреть профиль Найти все сообщения от KonstBessonov
 
Регистрация: 18.01.2018
Сообщений: 7

Понял, спасибо. будем тренироваться.
Ответить с цитированием
  #5 (permalink)  
Старый 30.01.2018, 14:51
Профессор
Отправить личное сообщение для Nexus Посмотреть профиль Найти все сообщения от Nexus
 
Регистрация: 04.12.2012
Сообщений: 3,791

Строка 3.

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

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

Последний раз редактировалось Nexus, 30.01.2018 в 14:56.
Ответить с цитированием
  #6 (permalink)  
Старый 04.02.2018, 14:07
Новичок на форуме
Отправить личное сообщение для KonstBessonov Посмотреть профиль Найти все сообщения от KonstBessonov
 
Регистрация: 18.01.2018
Сообщений: 7

Сообщение от Nexus Посмотреть сообщение
Функций "stage[\d]" можно было избежать, пропала бы и необходимость в приватных функциях "on[EventName]" контроллера.
С "stage\n" не понял немного. Там в 1 и 2, конечно, логика похожа, но в 3 и 4 совсем другая. С oninput/onkeypress в приницпе да, может и не стоило их использовать - всё-таки не рекомендуется.

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



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Проверка кода на ошибки. pokk Общие вопросы Javascript 1 04.03.2016 11:07
Ошибки верстки, едет верстка в IE9 Sanu0074 (X)HTML/CSS 3 01.04.2015 08:33
генерация и логирование ошибки Hapson Общие вопросы Javascript 31 03.04.2014 17:20
Ошибки при асинхронных AJAX запросах San4ezy Общие вопросы Javascript 4 27.03.2014 21:48
Как узнать какие ошибки произошли при парсинге xml файла faunder Events/DOM/Window 0 12.09.2008 14:17