コードレビューは否定の道

バグになりそうな点、バグっているところ

仕様にそっていないところ

保守性がよくなさそうなところ

今後問題になりそうなところ

よく分からかったところ

もっと良い書き方があるところ

褒めや良い点の指摘はあまりない

人は、「ここが何となく嫌だ」「ここに違和感がある」の方が意見しやすい

なので、コードレビューを受ける側はそういう心構えが必要

しかし、結局のところは、本人に任せて、外野は「明らかにバグになる箇所」や「明らかにパフォーマンス劣化を起こす箇所」などの指摘と理解を深めるための質問に徹するべきであり、「アドバイス」はしないほうがいい