リファクタリングの実情

20

Click here to load reader

Transcript of リファクタリングの実情

Page 1: リファクタリングの実情

リファクタリングの実情

石橋隆平(UZABASE SPEEDA)

Page 2: リファクタリングの実情

自己紹介 - 石橋隆平

● 好きな言語はScala、仕事はJava。● リファクタリング好き。

● プライベートはほぼすべて3歳の娘の育児。

● Scala Matsuri 2014でScala設計者Odersky先生にコードレビューしてもらい、

サイン入りコップ本を貰った。

● twitter:@cactaceae

直筆サインとメッセージ

Page 3: リファクタリングの実情

会社紹介

Page 4: リファクタリングの実情

会社紹介

先月上場しました

Page 5: リファクタリングの実情

サービス

SPEEDA 企業・業界分析の情報プラットフォーム

 金融機関・コンサルティングファームに多数導入

 絶賛海外進出中

NewsPicks 経済情報に特化したニュースアプリ

 記事とコメントを合わせて読むことでニュースを多角的に理解できる。

 編集部のオリジナル記事がある。

 

Page 6: リファクタリングの実情

石橋隆平(UZABASE SPEEDA)

リファクタリングの実情

Page 7: リファクタリングの実情

リファクタリングしてますか?

Page 8: リファクタリングの実情

見てみましょう

Page 9: リファクタリングの実情

git

Page 10: リファクタリングの実情

hg

Page 11: リファクタリングの実情

SPEEDAの場合

Page 12: リファクタリングの実情

SPEEDAの場合

4年前

リファクタリングコミット数 / コミット総数

= 74 / 7152

= 1.03%

 

Page 13: リファクタリングの実情

SPEEDAの場合

今年1年間

リファクタリングコミット数 / コミット総数

= 293 / 6610

= 4.43%

 

Page 14: リファクタリングの実情

SPEEDAの場合

4年間で4倍に増加

Page 15: リファクタリングの実情

やったこと

・自分がリファクタリングをする

・若い人を巻き込む

・ペアプロ

・コードレビュー

・pull-request

Page 16: リファクタリングの実情

有名OSSと比べてみよう

Page 17: リファクタリングの実情

有名OSSと比べてみようコミット総数 リファクタ数 割合

Mercurial 30977 441 1.42

Angular.js 11124 521 4.68

rails 57803 471 0.81

docker 19099 149 0.78

react 7682 19 0.25

elasticsearh 38170 567 1.49

Page 18: リファクタリングの実情

人でソートしてみる

Page 19: リファクタリングの実情

人でソートしてみる

Page 20: リファクタリングの実情

見えてきた傾向

・チームメンバーに一人リファクタリングをよくやる人がいると他のメンバーもリファクタリ

ングをするようになる

・外国籍のエンジニアはリファクタリングする

・若い(20代)のエンジニアはリファクタリングする

・ハイスキルでも外部パートナーはリファクタリングしない