Slide 1

Slide 1 text

【開発工数1/4に削減!】 コメントファーストの開発方法 @masafumi330 1/11

Slide 2

Slide 2 text

自己紹介 - masafumi330 - TuneCore Japan (2022年~現在) - バックエンドエンジニア (Go) - 熊本出身 - 妻と二人暮らし - 趣味:筋トレ、Netflix - 最近よく聴くアーティスト - Dua Lipa - Lauv - X(Twitter): @masafumi330 この写真は妻が撮ってくれました! 2/11

Slide 3

Slide 3 text

TuneCore Japan 3/11

Slide 4

Slide 4 text

こんなシーンないですか? - Pull Request レビューでレビュー修正のコメントが積み重なる - 実装途中で方針が何度も変わってしまう - 実装漏れ、仕様漏れが起きてしまう 4/11

Slide 5

Slide 5 text

どうすればいい? 5/11

Slide 6

Slide 6 text

真っ先にコメントを書いたPull Requestを出すのです! 6/11

Slide 7

Slide 7 text

コメントを書いていく - 既存コードリーディングしながら、実装していく箇所にコメントを書いていく - 何を取得するのか?条件分岐は?処理の流れは? - 新規ファイル作成、I/F、関数やメソッドのシグネチャ ex.) // TODO: // - ここで XXX を取得 // - もし YYY であれば YYY // - それ以外は ZZZ ex.) // TODO: // - valid // - ドメインオブジェクトを生成 // - 永続化 // - response 7/11

Slide 8

Slide 8 text

Pull Requestの作成 - Descriptionにレビュー観点として「方針の確認」を記載 - ex.) ざっくり実装の概要を TODOで書いていますが、方針問題ないか確認お願いします! - レビュワーをアサインしてレビュー依頼 8/11

Slide 9

Slide 9 text

効果 - レビュワーとの認識の齟齬が無くなる - コメントを書いていく中で考慮漏れに気づく - 詳細実装に入った際、迷いなく開発が進むので、爆速で実装が終わる 実際この方針に変えたところ。。。 開発工数が 1/4 ほどまで削減できました! 9/11

Slide 10

Slide 10 text

まとめ - 実装量が多くなりそう、複雑度が高そうな開発だと感じたら、 まずはコメントで実装方針を書いてみる - コメントで先に方針を固めておくことで、開発工数の削減、 考慮漏れのミス、仕様の確認をすることができる - 迷いなくコーディングを進められることが気持ちいい!!! 10/11

Slide 11

Slide 11 text

ご清聴ありがとうございました! https://www.tunecore.co.jp/jobs チューンコアジャパンではエンジニアを募集しています。 11/11