Неудивительно, что обзоры кода больше всего интересуют получение знаний в области кодирования. Я вижу много разработчиков, страдающих от ревью кода, особенно новичков. Не уверен, что следует попросить переработать или просто оставить предложение. Итак, в этом посте я перечислил несколько основных элементов контрольного списка проверки кода, которые, как я полагаю, дадут вам некоторое представление о проверке кода и о том, на что следует обратить внимание в чужом коде, чтобы вы могли начать свое путешествие по проверке кода.
Работает ли код (насколько вы можете судить, прочитав его)? Правильно ли написана логика?
Я имел в виду, просто посмотрите на код, вам не нужно глубоко погружаться ни в какую бизнес-логику, если на первый взгляд вы почувствуете, что что-то не так и нелогично. ага, ты поймал. попытайтесь понять, что они пытаются сделать, конечно, это никогда не является ракетостроением, и я почти уверен, что вы можете понять, и если что-то подозрительно, поймайте это прямо здесь
Отсутствуют ли важные проверки (нулевая проверка и другие важные проверки)?
просто базовая проверка, что-то вроде использования данных API в html, поэтому условная проверка, доступны ли данные, затем визуализирует внутренние компоненты, если есть конкретное значение, прежде чем пытаться выполнить какую-либо операцию с ним и т. д.
Есть ли у циклов правильные условия выхода?
Я имею в виду, почему вы зацикливаете все данные, если ваше требование уже выполнено? используйте операторы break везде, где это необходимо. Это определенно оптимизирует производительность и время. И это подводит меня к тому, чтобы добавить еще один момент.
Использование подходящих методов
Я вижу, что большинство разработчиков, даже самых опытных, используют только методы foreach или конвейера для достижения чего-то, что легко сделать с помощью правильных методов. например, вместо того, чтобы передавать фильтр и карту, используйте методы уменьшения массива. Просто используя правильные методы, вы можете значительно улучшить производительность вашего приложения.
Изменения CSS необходимо тестировать во всех адаптивных форм-факторах
не имеет значения, что у вас есть такое же веб-приложение в мобильном приложении. важна отзывчивость. буквально. Сколько из вас открыли мобильное приложение в браузере? По крайней мере, на популярных устройствах нужно сделать отзывчивость, и это никогда не является дополнительной работой веб-разработчика, а является ее частью. Начните с небольших устройств и перейдите на большие экраны. поэтому всегда просите отзывчивые экраны в обзоре.
Правильный ли отступ в коде?
Иногда это звучит очень просто и неважно, но это один из самых важных моментов, которые необходимо проверить. Как насчет того, чтобы работать над неуклюжим кодом с правильным отступом? следовательно, это важно как часть контрольного списка.
Не накапливайте технические долги
Он начинается со слов «будет обработан в следующем pr». Действительно? вы когда-нибудь видели, как исправляли старый код? когда-то технический долг, он всегда накапливается и никогда не уменьшается, поэтому никогда не поощряйте его.
больше пунктов контрольного списка будет в предстоящем блоге.
счастливого кодирования ❤
