upgrading...

Upgrading�Rails?�It�is�a�good�time�to�refactor�

your�codeP.S.V.R

Powered�by�Rabbit�2.1.2

Rails�4.1�was�released�last�week!

1/39

Why�is�it�a�good�time�to�refactor�your�code?�(1)

Trust�me:�if�you�had�time�for�upgrading,�you�would�have�time�for�refactoring

2/39


��

some�upgraded�behaviors�require�refactoring�after�all

3/39


��

You�wan�to�implement�old�functions�via�new�features

4/39

RoadmapⅠ.�Community�Guidelines

Ⅱ.�Code�coverage

Ⅲ.�Dependencies�updating

Ⅳ.�Density�of�Comments

Ⅴ.�Cyclomatic�complexity

Ⅵ.�Static�code�analyzer

Ⅶ.�Commit�messages

5/39

Ⅰ.�Community�Guidelines�(1)

ruby-style-guide

A�community-driven�Ruby�coding�style�guide

https://github.com/bbatsov/ruby-style-guide

6/39

Ⅰ.�Community�Guidelines�(2)ex.�Comments�section

7/39

Ⅰ.�Community�Guidelines�(3)

rails-style-guide

A�community-driven�Rails�3�&�4�style�guide

https://github.com/bbatsov/rails-style-guide

8/39

Ⅰ.�Community�Guidelines�(4)ex.�Routing�section

9/39

Ⅱ.�Code�coverage�(1)simplecov

Code�coverage�for�Ruby�1.9+�with�a�powerful�configuration�library�and�automatic�merging�of�coverage�across�test�suites

gem�install�simplecov

10/39

Ⅱ.�Code�coverage�(2)

11/39

Ⅱ.�Code�coverage�(3)coveralls.io

https://coveralls.io/

gem�ʻcoverallsʼ,�require:�false

12/39

Ⅱ.�Code�coverage�(4)

13/39

Ⅲ.�Dependencies�updating�(1)

gemnasium

https://gemnasium.com/

14/39

Ⅲ.�Dependencies�updating�(2)

15/39

Ⅳ.�Density�of�Comments�(1)

cf.�http://staff.unak.is/andy/StaticAnalysis0809/metrics/dc.html

16/39

Ⅳ.�Density�of�Comments�(2)inch

Documentation�measurement�tool�for�Ruby,�based�on�YARD.

gem�install�inch

17/39


18/39


inch-pages

https://github.com/rrrene/inch-pages

19/39


20/39

Ⅴ.�Cyclomatic�complexity�(1)

developed�by�Thomas�J.�McCabe,�Sr.�in�1976

For�this�strongly�connected�control�flow�graph,�Cyclomatic�complexity�=�3�is�the�number�of�linearly�independent�cycles�that�exist�in�the�graph,�i.e.�those�cycles�that�do�not�contain�other�cycles�within�themselves.

21/39


22/39


��

ex.�Cyclomatic�complexity�=�1

23/39


��


24/39


��


25/39


��

ex.�Cyclomatic�complexity�=�826/39

Ⅴ.�Cyclomatic�complexity�(7�-�before)

��

27/39

Ⅴ.�Cyclomatic�complexity�(8�-�after)

��

��

28/39

Ⅴ.�Cyclomatic�complexity�(9�-�after)

��

29/39

Ⅵ.�Static�code�analyzer�(1)

rubocop

gem�install�rubocop

rubocop�-R�>�rubocop.txt

30/39


31/39


codeclimate

https://codeclimate.com/

32/39


33/39

Ⅶ.�Commit�messages�(1)good�commit�message�example

34/39

Ⅶ.�Commit�messages�(2)bad�commit�message�examples

35/39


36/39


37/39

How�to�write�good�commit�messages?

Use�English

Use�the�imperative,�present�tense:�“change”,�not�“changed”�or�“changes”;�“Fix�bug#1234”�is�better�than�“Fixed�bug�#1234”

Use�50-char�title�and�Use�72-character�columns�for�word-wrapping.

Read�more:�https://github.com/torvalds/linux/pull/17#issuecomment-5654674 38/39

Download�the�slides

https://github.com/pmq20

39/39Powered�by�Rabbit�2.1.2

upgrading...

Entertainment & Humor