2021-10-17 13:01:12
Как критиковать код
Критика кода (и программных решений в целом) - важный навык. Он помогает повышать уровень знаний команды и улучшать решения. Но это не то, что приходит само по себе. Подобно тому, как вы учитесь редактировать чужие эссе, вы должны научиться критиковать код членов команды.
Во-первых, разберитесь в контексте. Есть два типа, которые имеют значение. Первый - это бизнес-контекст. Какую проблему пытается решить эта система? Какие ограничения есть у бизнеса (время, деньги, труд, понимание)?
В стартапе часть кода будет написана быстро и грязно, потому что стартап пытается найти то, за что люди будут платить деньги. Быстрое исследование более ценно, чем безупречная архитектура и ремонтопригодность.
Вам также необходимо понимать технический контекст. Каков масштаб / сроки / уровень знаний команды? Все это должно послужить основой для вашей критики.
Если вы являетесь частью команды, вы можете знать эти контексты, но со временем они меняются. Поэтому всегда стоит дважды проверять, прежде чем начинать обзор, чтобы убедиться, что вы не придерживаетесь устаревших предположений.
Затем определите, «на каком холме вы хотите умереть»; то есть то, что достаточно важно, чтобы вы могли прокомментировать. Какие аспекты кодекса важны для вас с учетом личного опыта, целей команды или этических соображений? Если вы не знаете, что ценит команда, спросите! А затем запишите это, чтобы другим было полезно.
Наконец, примечание о тоне. Вы можете быть резкими с отзывами, но не путайте чью-то реализацию с тем, кто они есть. Скорее всего, они сами достаточно этого делают; Я знаю, что иногда зацикливаюсь на своем коде и принимаю его на свой счет, когда обнаруживаю, что этого не хватает. Я считаю, что лучше всего подходит тон, предполагающий позитивные намерения, но при этом задавая сложные вопросы. Поверьте, что член вашей команды сделал все, что мог, и постарайтесь сосредоточиться на его улучшении и обучении вас обоих.
8.3K views10:01