Download to read offline
![for example:
!
{ :foo => "bar" } => { foo: 'bar' }
!
[:hoge, :piyo] => %i( hoge piyo )
!
raise => fail
!
before_filter => before_action](https://image.slidesharecdn.com/rubocop-150318034918-conversion-gate01/85/Rubocop-42-320.jpg)
![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
※注意：この例は、例として分かりやすくするために誇張したものです](https://image.slidesharecdn.com/rubocop-150318034918-conversion-gate01/85/Rubocop-48-320.jpg)
![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
※注意：この例は、例として分かりやすくするために誇張したものです](https://image.slidesharecdn.com/rubocop-150318034918-conversion-gate01/85/Rubocop-49-320.jpg)
![[コデアル株式会社] Ruby on rails 開発環境設定勉強会(株式会社シンクスマイル主催)](https://cdn.slidesharecdn.com/ss_thumbnails/rubyonrails-140527200620-phpapp02-thumbnail.jpg?width=640&height=640&fit=bounds)
プロジェクトでRubocopを使って自動コードレビューしてみた話
- 1.
- 2.
- 3. 【会社】アピリッツ • ゲーム系 • 式姫シリーズ •アルフヘイムの魔物使い • 受託系 • EC開発 • ASP提供 ! 【個人】@cyber_yoshida • バンドとか • ウェブサービス作り • 書籍管理サービスつくってます ! 【このスライド】 • Azusa Colors 使いました • http://memo.sanographix.net/post/113681262780
- 4.
- 5.
- 6.
- 7.
- 8.
- 9.
- 10.
- 11.
- 12.
- 13.
- 14.
- 15.
- 16.
- 17.
- 18.
- 19.
- 20.
- 21.
- 22.
- 23.
- 24. • 新卒 • 派遣の方 •Ruby1.9に慣れてる人 • Rails2, 3に慣れてる人 など
- 25.
- 26. • Ruby2.1, Rails4.1 •Rubocop 0.24 (最新は 0.29) • Redmine • GitLab • Jenkins
- 27.
- 28.
- 29.
- 30.
- 31.
- 32.
- 33.
- 34.
- 35.
- 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.
- 46.
- 47.
- 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.
- 51.
- 52.
- 53.
- 54.
- 55.
- 56.
- 57.
- 58.
- 59.
- 60.
- 61.
- 62.
- 63.
- 64.