Upgrade to Pro — share decks privately, control downloads, hide ads and more …

コードレビューをレビューできてますか?(code review)

86514eb926fa17a85692b0f609d89b59?s=47 great tower
December 16, 2021

コードレビューをレビューできてますか?(code review)

86514eb926fa17a85692b0f609d89b59?s=128

great tower

December 16, 2021
Tweet

Transcript

  1. Copyright© Speee, Inc. All rights reserved. Confidential 株式会社Speee
 2021/12/01
 コードレビューをレビューできてますか?


  2. Copyright© Speee, Inc. All rights reserved. 2 八木 塔偉(やぎ とうい) 

    @great_tower511
 株式会社Speee エンジニア 
 2021年8月から
 新規事業開発(toC&toB) 
 
 趣味
 漫画(少年漫画)
 ゲーム(FPS)
 
 
 
 プロフィール

  3. Copyright© Speee, Inc. All rights reserved. 3 伝えたいこと
 ・なんとなくフレームワーク、日々の業務に則って開発、レビューしている方 


    
 ・とりあえず動けばOKな環境でエンジニアリングしている方 
 
 
 ・自分のレビューに疑いを持っている方 
 
 
 ・レビュー力は経験で片付けてしまっている方 
 
 
 が自分からもっと良いレビューできるようにしようと動き出すきっかけになっていた だけたら嬉しい!! 

  4. Copyright© Speee, Inc. All rights reserved. 4 背景
 1~2年目の子達
 僕


    3ヶ月前まで
 10年選手のエンジニア 
 僕
 現在

  5. Copyright© Speee, Inc. All rights reserved. 5 
 ちゃんとコードレビューできていなかったなぁ
 


    
 環境が変わって気がついた。

  6. Copyright© Speee, Inc. All rights reserved. 6 
 経験・・・?
 


    
 どうすればレビューが自信を持ってできるようになるのか

  7. Copyright© Speee, Inc. All rights reserved. 7 
 愚直に過去のPRについているレビューコメントを集めてみる
 


    
 
 やってみたこと

  8. Copyright© Speee, Inc. All rights reserved. 8 やってみたこと(具体例)
 
 


    当たり前だよねっていうところも含めて見たものは全部集めてみた
 
 
 

  9. Copyright© Speee, Inc. All rights reserved. 9 集めてどうしたか・・
 
 ・このコメントはできないな


      ・そもそもその機能やライブラリを知らない?
   ・実装方法が思いつかない?
 
 
 ・このコメントは僕もできるな
   ・本当にレビュー依頼がきたときできてた?
 
 
 

  10. Copyright© Speee, Inc. All rights reserved. 10 やってみたことのまとめ
 ・僕に一番足りないなとわかったこと 


    ・設計の原理原則が身に染み付いているからできている 
 
 ・一旦置いておく
 ・もっといい命名できている
 
 ・言語やフレームワーク、ライブラリの機能でもっといいものを知っている 
 
 

  11. Copyright© Speee, Inc. All rights reserved. 11 まとめ
 自身のレビューを振り返ったことで課題が明確になりまし た!


    
 レビューはコメントは色んな人の知識と経験の集合体
 
 だからこそ、ちゃんと振り返れば自分の足りない部分がどこ かにはある