Поэтому откровенные ошибки надо подчеркнуть красным и внятно, но без рассюсюкивания и реверансов, объяснить, почему так делать нельзя. Места, которые могут быть переписаны производительнее, изящнее, короче — имеет смысл обозначить и снабдить комментарием «а вот еще как можно: что думаешь?», но не настаивать на изменениях.
Претензии к коду наподобие «некрасивый», «невнятный», «неаккуратный», «как говно» — нужно вантузом затолкать себе обратно в горло и никогда больше оттуда не доставать.
- Свой код я всегда отдаю на CR двоим: джуну и синьёру. Первому это и приятно и полезно, а второй — может и ошибку найти (первый тоже может, на самом деле, причем чуть ли не чаще, из-за незамыленности взгляда).
- Никогда, ни при каких обстоятельствах, я не вношу изменения в чужой код сам, вместо того, чтобы объяснить, как надо, и предоставить возможность внесения изменений автору.
- После того, как CR готов к выпуску в свет, я всегда оставляю последний комментарий с набором смайликов, означающих «урра, зарработало!». Я научился этому у Жозе Валима, но смайлики выбрал другие.