ペアプロってどうなの？

Pull Requestでしか開発したことがない人向けの、ペアプロの良し悪しが分かる資料。

Transcript

1. Copyright Drecom Co., Ltd. All Rights Reserved. !1
   ϖΞϓϩͬͯͲ͏ͳͷʁ
   吉岡ひろき(@irohiroki)
   2019-02-01
   

   View Slide

2. Copyright Drecom Co., Ltd. All Rights Reserved.
   Pull Requestの欠点
   コンテキスト伝達のコスト
   ཁ݅ɾ࢓༷
   ղܾํ๏
   ͳͥ͜͏ͳ͍ͬͯΔͷ͔
   レビューコメントを書くコスト
   ཧ༝
   ୅ସҊ
   ૬ख΁ͷ഑ྀ
   レビュー待ち
   ϚϧνλεΫʹΑΔޮ཰௿Լ
   ϦʔυλΠϜͷѱԽ
   !2
   എܠ
   

   View Slide

3. Copyright Drecom Co., Ltd. All Rights Reserved.
   ２人で
   １つの画面を見ながら
   １人がドライバー（コードを書く／コマンドを実行する）
   １人がナビゲーター（レビュー／作業の順番を考える）
   状況に応じて交代
   Pull Requestは簡略に
   !3
   ϖΞϓϩ
   

   View Slide

4. Copyright Drecom Co., Ltd. All Rights Reserved.
   即時コードレビュー／ダブルチェック
   ϛεͷൃݟ͕ૣ͍ˠख໭Γ͕গͳ͍
   トラックナンバー +1
   ยํ͕ٳΜͰ΋Өڹ͕খ͍͞
   フロー効率
   ݸʑͷ࢓ࣄ͕ૣ͘ऴΔ
   !4
   ͳͥ̎ਓͰʁ
   

   View Slide

5. Copyright Drecom Co., Ltd. All Rights Reserved.
   平行して２つの仕事
   ペアで１つずつ
   !5
   ϑϩʔޮ཰
   έϯτ
   ΢Υʔυ
   έϯτ
   ΢Υʔυ
   Ըܙ
   Ըܙ
   Ըܙ
   Ըܙ
   

   View Slide

6. Copyright Drecom Co., Ltd. All Rights Reserved.
   ミスを笑える
   ͪΐͬͱͨ͠ϛε͕͋ͬͨͱ͖স͑ΔɻετϨε͕ܰݮɻ
   心理的負担の軽減
   ੹೚΋̎ਓͰ෼୲
   決定の迅速化
   ઃܭ΍ωʔϛϯάɻ̎ਓ͕߹ҙͰ͖Ε͹ຆͲͷ৔߹ʹ໰୊ͳ͍
   知識とテクニックの伝搬
   ஌Βͳ͔ͬͨϝιουɺγϣʔτΧοτɺΦϓγϣϯʜ
   集中できる
   ૬खΛ଴ͨͤͳ͍Α͏ʹ
   !6
   ͦͷଞͷϝϦοτ
   

   View Slide

7. Copyright Drecom Co., Ltd. All Rights Reserved.
   ２人でやっても半分の時間にならないだろ？
   → なる。
   諸々の決断が速い
   手戻りが少ない
   Pull Requestを作るのが楽
   レビュアーが背景を理解する時間がいらない／

   開発の過程を再現する時間がいらない
   コメントを書く時間がいらない
   !7
   ϖΞϓϩ'"2
   

   View Slide

8. Copyright Drecom Co., Ltd. All Rights Reserved.
   Pull Requestは作るの？
   → 文化による。
   「GitHubで見たい」派
   レビュアーのための作文は不要
   コミットメッセージの方が重要
   !8
   ϖΞϓϩ'"2
   

   View Slide

9. Copyright Drecom Co., Ltd. All Rights Reserved.
   状況に応じて。
   実装のアイデアがある
   仕様に詳しい
   先行して着手してた
   など
   !9
   υϥΠόʔͷܾΊํ
   

   View Slide

10. Copyright Drecom Co., Ltd. All Rights Reserved.
    テストコード／プロダクトコード 1JOH1POH1BJSJOH
    
    ナビゲーターにアイデアが閃いたとき
    セッション（約１時間）毎
    など
    FAQ - マシン／キーボードは換える？
    Ping-Pong → 換えない
    セッション → 換える
    !10
    υϥΠόʔަ୅ͷλΠϛϯά
    

    View Slide

11. Copyright Drecom Co., Ltd. All Rights Reserved.
    1. 朝会でタスクを洗い出し・共有
    2. ペアでやるタスク／担当者／時間帯を打ち合わせ
    !11
    ϖΞͷεέδϡʔϦϯά
    

    View Slide

12. Copyright Drecom Co., Ltd. All Rights Reserved. !12
    εΩϧࠩ
    スキル差が大きい
    ॳֶऀ͸ଟ͘ͷ஌ࣝΛಘΒΕΔ
    ϕςϥϯ͸ࢦಋํ๏ΛֶͿ
    ͲͪΒ͕υϥΠόʔͰ΋Α͍
    スキル差が小さい
    ಘҙ෼໺ͷҧ͍͕ڞ૑ΛੜΉ
    ͲͪΒ͕υϥΠόʔͰ΋Α͍
    ͓͢͢Ίͷهࣄ ஌ࣝࠩ−εΩϧࠩΛຒΊΔͨΊͷϖΞϓϩʴαͷίπ
    

    View Slide

13. Copyright Drecom Co., Ltd. All Rights Reserved. !13
    ϖΞͷίπ
    敬意
    ૬ख͕୭Ͱ΋ɺઌഐͩͱࢥͬͯ
    話すタイミング
    υϥΠόʔ͕λΠϐϯάΛऴ͔͑ͯΒ
    調査は別々に
    ΤϥʔͷݪҼɺϥΠϒϥϦͷ࢖͍ํͳͲ

    ผʑʹࣗ෼ͷϚγϯͰௐ΂ͨํ͕ޮ཰త
    

    View Slide

14. Copyright Drecom Co., Ltd. All Rights Reserved. !14
    όϦΤʔγϣϯ
    ペアレビュー
    ιϩͰ࣮૷ˠϨϏϡΞʔ͕ՃΘͬͯϨϏϡʔ
    ର໘ͰίϛϡχέʔγϣϯίετΛ௿ݮ
    ࣮૷ऀˠϨϏϡΞʔɿίϯςΩετͷ఻ୡ
    ϨϏϡΞʔˠ࣮૷ऀɿϑΟʔυόοΫ
    ペアプロ → ソロ → ペアレビュー → ペアプロ
    ̍ਓ్͕தͰൈ͚ͨͱ͖
    

    View Slide

15. Copyright Drecom Co., Ltd. All Rights Reserved. !15
    ମݧऀͷ੠
    3,͞Μ
    ʮϖΞͰ͓ئ͍͍ͨ͠Ͱ͢ʯ

    ʢ؆୯ͳ࡞ۀ͔Βຊ֨తͳ࣮૷ʹҠΔͱ͖ʣ
    ),͞Μ
    ʮϖΞͩͱ҆৺Ͱ͢ʯ

    ʢࠓ·Ͱͱҧ͏छྨͷλεΫΛ୲౰ͨ͠ͱ͖ʣ
    

    View Slide

16. Copyright Drecom Co., Ltd. All Rights Reserved. !16
    σϝϦοτ
    時間と場所の拘束
    Ұॹʹ͍ͳ͍ͱͰ͖ͳ͍
    相手のマシンが使いにくい
    ΤσΟλʗγϣʔτΧοτͷҧ͍ͳͲ
    Α͘࢖͏γϣʔτΧοτ͸߹ΘͤΑ͏
    疲れる
    ໿̍࣌ؒͰٳܜΛɻͦ͏͠ͳ͍ͱ̍೔ଓ͔ͳ͍
    

    View Slide

17. Copyright Drecom Co., Ltd. All Rights Reserved. !17
    ϖΞ͕޲͔ͳ͍λεΫ
    ルーチンワーク
    レビューのいらない仕事
    概要設計など
    ௐࠪʗઃܭʗυΩϡϝϯςʔγϣϯ
    

    View Slide

18. Copyright Drecom Co., Ltd. All Rights Reserved. !18
    ·ͱΊ
    フロー効率
    精神的負担の軽減
    トラックナンバー増加 → 安定した業務の進行
    スキル差 → 双方の学び
    敬意
    時間と場所の拘束
    

    View Slide