$30 off During Our Annual Pro Sale. View Details »

Let's learn code review

Let's learn code review

春和の候!若手エンジニアふんわりLT Night!_vol4で発表した資料

RioFujimon

April 25, 2024
Tweet

More Decks by RioFujimon

Other Decks in Programming

Transcript

  1. 自己紹介 2 - 名前:藤門 莉生(Rio Fujimon) - 所属:Sansan株式会社 - 📱

    23卒 iOSエンジニア - 🍎 Eight(iOS版)を開発 - SNS:@RioFujimon
  2. コードレビュー観点 ① 18 1. コードが適切に設計されているか? 2. 機能がコードのユーザーにとって良いものであるか? 3. UIの変更が適切で見栄えが良いか? 4.

    並行プログラミングが安全に行われているか? 5. コードが必要以上に複雑でないか? 6. 開発者が将来必要になるかもしれないが、 現時点では必要性が分からないものを実装していないか?
  3. コードレビュー観点 ② 19 7. コードに適切な単体テストが含まれているか? 8. テストが適切に設計されているか? 9. 開発者がすべてのものに明確な名前を付けているか? 10.

    コメントが明確で有用であり、 主になぜそのようになっているかを説明しているか? 11. コードが適切に文書化されているか? 12. コードがスタイルガイドに準拠しているか?