自分がコードレビューで気をつけていること（2020.5 iOSアプリ開発版 ）

わたくし@yimajoが仕事のコードレビューで気をつけていることを書いておきます。特にiOSアプリ開発をしているので内容はiOSアプリ開発におけるコーディングやStoryboardについて書いていますが、その前提やメンタル面の話が多めです。 ちなみにiOSアプリのコードレビューで見ているポイント 2020年5月版にインスパイアされて書いてみました。 前提 レビュー時にリスクや課題、問題という言葉を意識して使い分ける 「このコードは問題です」という指摘をするとき、「問題」という言葉の意味をなるべく共通の理解を持つ 問題 現在起こっている正常でない状況 例: 仕様を表現できていない 例: 不具合がある 課題 問題を整理/分割したもの 例: deprecatedなAPIを使っている リスク 時間の経過もしくは何かの要因によって課題や問題になってしまう事柄 例: テストコードが書けていないので手を

[a1g]

元記事は良いけどこの方は保身が酷くて本当はどうでも良い指摘などなくコードが求められている品質に足らないから指摘していて、時間の都合上仕方なく目をつぶっているということを明示しているのを理解した方が良い

[rd050201]

レビュー観点とかまとめますかー