Ошибки начинающих разработчиков (как проверить код перед ревью) | Фронтенд-разработка HD
Ошибки в коде начинающих разработчиков и на что нужно обратить внимание перед тем, как отправить задачу в ревью. Видео рассчитано на начинающих разработчиков и, т. к. я фронтенд-разработчик, ориентируюсь на фронтенд. В видео 16 советов, как проверить свой код, чтобы вашу задачу реже возвращали с комментариями "не работает", "вёрстка уехала" и правками по стилю кода. Делать ошибки и не уследить за 100% — это нормально. Мы не машины и можем ошибаться. Поэтому ревью и обязательно) Но если можем дополнительно проверить и сократить время на ревью — думаю, стоит это делать) По всем этим граблям проходилась (и реже, но на некоторые ещё иногда наступаю) в обеих ролях ССЫЛКИ ИЗ ВИДЕО: Советы по стилю кода: https://learn.javascript.ru/coding-style Правила форматирования CSS: https://learn.javascript.ru/css-format Плагин для наложения макета на веб-странице: https://chrome.google.com/webstore/detail/perfectpixel-by-welldonec/dkaagdgjmgdmbnecmcefdhjekcoceebi Мой INSTAGRAM: https://www.instagram.com/listopadova.antonina.web Мой канал в Telegram: https://tmgo.me/alistopadova ТАЙМ-КОДЫ: 00:00 О чём видео, для кого и зачем) 00:20 Статусы работы над задачами и о каком этапе речь в видео: о переходе из "в разработке" в состояние "ревью". 01:00 Проблема, которую помогают частично/полностью решить 16 способов проверки своего кода из видео. 01:45 №1 Стиль кода. 02:43 №2 Проверка условий на избыточность. 03:08 №3 Понятные имена переменных/функций/методов/т. д. 03:50 №4 Проверка логики работы добавленной функциональности. 04:58 №5 Почистить от "экспериментов" и логирования. 05:09 №6 Проверка ошибок в консоли в панели разработчика. 05:14 №7 Перечитать задачу. 05:25 №8 Результат вёрстки должен выглядеть как на макете. 06:25 №9 Проверка вёрстки на разных разрешениях экрана. 07:17 №10 Проверка использования переменных в стилях. 07:38 №11 При создании переменной в стилях убедиться, что переменной с таким именем/значением ещё нет. 07:48 №12 Проверка вёрстки и логики в разных браузерах. 08:19 №13 Запуск локальных тестов. 08:25 №14 О прохождении тестов в CI после push. 08:48 №15 Отреагировать на все комментарии при повторном возврате задачи на ревью. 09:30 №16 Точно запушить изменения :) "Как оно бывает" — эмоции со стороны ревьюера и со стороны того, кого ревьювят. 10:41 Окончание и "неудавшиеся дубли" Подробности в видео, а тут коротко 16 пунктов: 1. Стиль кода. В каждом проекте стиль может немного отличаться, но основные моменты и принципы одинаковые. Пара ссылок, где можно узнать основные общепринятые нормы, чуть выше в описании. 2. Проверьте условия на избыточность. 3. Имена переменных должны максимально понятно передавать их смысл. Ещё обязательно проверьте, что не написали имя новой переменной с ошибкой. 4. Если в задаче делалась какая-то логика, а не просто вёрстка, то нужно проверить логику работы и все варианты пользовательских сценариев взаимодействия с новой функциональностью. 5. Если добавляли какое-то логирование или что-то ещё для локальной от