渋谷.rb[:20150318] でLTしたい用

1. プロジェクトで
Rubocopを使って
自動コードレビュー
してみた話
Appirits 吉田啓紀

2. 🌟自己紹介

3. 【会社】アピリッツ
• ゲーム系
• 式姫シリーズ
• アルフヘイムの魔物使い
• 受託系
• EC開発
• ASP提供
!
【個人】@cyber_yoshida
• バンドとか
• ウェブサービス作り
• 書籍管理サービスつくってます
!
【このスライド】
• Azusa Colors 使いました
• http://memo.sanographix.net/post/113681262780

4. こういう問題
ありませんか？

5. コードの書き方が
みんなバラバラ！

6. 統一したい！

7. コーディングルール
作ろうか・・？

8. でも面倒くさい

9. そこで
Rubocop

10. Rubocop
って何ができる？

11. 「メソッド長いよ」

12. 「メソッドが複雑だよ」

13. 「メソッド名良くないよ」

14. 「空白/改行余計だよ」

15. 「こう書けるよ」

16. などなど

17. これ、
コーディングルールに
ならない？

18. やってみた！

19. Rubocop
を使う
∼プロジェクト編∼

20. こんな
プロジェクト
です

21. ECシステムの開発

22. ３∼５ヶ月くらい

23. ５∼７人くらい

24. • 新卒
• 派遣の方
• Ruby1.9に慣れてる人
• Rails2, 3に慣れてる人
など

25. こんな
環境です

26. • Ruby2.1, Rails4.1
• Rubocop 0.24
(最新は 0.29)
• Redmine
• GitLab
• Jenkins

27. メモ
!
MethodLength
Max: 30
!
ClassLength:
Max: 300

28. かんたんな
運用フロー

29. コードを書く

30. GitLabで
マージリクエスト

31. テストが走る

32. テストが走る
Rubocop も走る

33. OK ならマージ可能

34. 詳細は
別スライドに！

35. !
!
!
!
!
!
!
!
!
!
!
!
!
!
!
http://www.slideshare.net/cakeyoshida/
ss-31147447

36. わかったこと

37. ☀️ 良い点

38. キレイに書こう、
と意識できる

39. コードを統一できる

40. 後から
リファクタリング
しやすい

41. 新しい構文を知る
良い機会

42. for example:
!
{ :foo => "bar" } => { foo: 'bar' }
!
[:hoge, :piyo] => %i( hoge piyo )
!
raise => fail
!
before_filter => before_action

43. ⛅️ 課題

44. メンバーとの衝突

45. Aさん
「Rubocopに合わせる
とかえって汚くなる！」
「実装遅れる！」

46. 無意味な
リファクタリング

47. Bさん
「メソッドが長い？
わかりました
分割します」

48. Before
def set_customer
@customer = Customer.new
@customer.first_name = params[:customer][:first_name]
@customer.last_name = params[:customer][:last_name]
@customer.hoge = params[:customer][:hoge]
...
...100行ほど...
...
@customer.valid?
end
※注意：この例は、例として分かりやすくするために誇張したものです

49. After
def set_customer
@customer = Customer.new
set_customer1
set_customer2
set_customer3
@customer.valid?
end
!
def set_customer1
@customer.first_name = params[:customer][:first_name]
@customer.last_name = params[:customer][:last_name]
@customer.hoge = params[:customer][:hoge]
...
end
!
def set_customer2
...
end
!
!
def set_customer3
...
end
※注意：この例は、例として分かりやすくするために誇張したものです

50. Rubocopに怒られない
ね、キレイでしょ？

51. どうしてこうなった。。。

52. Rubocopは神ではなく
あくまでツール

53. まとめ

54. Rubocop、
導入の価値あり

55. ただし

56. メンバーの理解と
協力は必須

57. 「やらされる」
では効果がない！

58. おまけ

59. Rubocop、
個人でも使ってます

60. が

61. 初期設定のままだと

62. リファクタリング
強制ギプス感すごいです

63. おすすめ！

64. おわり