【iOS】コードレビューをする上で気をつけること

最初にファイルのimportをみる

• 1つのファイルにimportが4~5つもあったらファイル内で色々やらせすぎなので、分ける必要がある。

"!"マークで検索をかける。

• "!"マークは強制アンラップ、なければ無いほうがいい。強制アンラップは避ける。

ロジックを見て冗長であり、まとめられそうかみる、

• 冗長の場合、冗長でない書き方をレクチャーする。
• for文 で回しているのをmap使った方がいいとか。
• 仕様がわからない場合、ロジックはレビュアーにお任せする。あくまでコード面で冗長性がないかを確認。

リードエンジニアのレビューを踏襲して真似してみる。

• 真似してみてリードエンジニアの考え方を学ぶ

Swiftらしくかけているかを意識する。

• Objective-CからSwift移行の場合など。
• Swiftyに。

また思いついたら追記します。