7 вещей, на которые я обращаю внимание, прежде чем утвердить запрос на включение

7 вещей, на которые я обращаю внимание, прежде чем утвердить запрос на включение Проверка кода — это один из тех навыков, которым вас никто не учит. Вас бросают в это, вы начинаете с придирок к именам переменных и в конечном итоге — обычно после появления ошибки, которую можно было бы обнаружить за пять секунд — вы разрабатываете настоящую систему. Это система, которую я использую. Это не является исчерпывающим и не является чем-то необычным. Это всего лишь семь вопросов, которые я задаю себе в каждом пиаре, примерно в таком порядке, прежде чем нажать «Одобрить». Я включил реальные примеры кода для каждого из них — плохую версию, лучшую версию и то, что я на самом деле ищу как рецензент. Давайте займемся этим. 1. Действительно ли код решает проблему? Это звучит очевидно, но именно эту проверку люди пропускают чаще всего, потому что возникает соблазн сразу перейти к чтению синтаксиса вместо того, чтобы сначала спрашивать: «Соответствует ли это билету?» Я прочитал описание PR и связанную с ним проблему, прежде чем прочитал хотя бы одну строку кода. Затем я спрашиваю: если бы я реализовал именно этот сценарий, дал бы он правильный результат? Не «выглядит ли это как разумный код» — решает ли это реальную проблему. Плохая версия — закрывает тикет «фильтровать неактивных пользователей», но незаметно меняет поведение в других местах: функция getUser