リファクタリング練習

リファクタリング

リファクタリングは単にコードを読みやすくするだけではない
チーム開発において見やすいコードとは非常に重要で、チーム開発では他人のコードを解読する時間は非常に多くなる
その時間を短くすることは開発効率の促進に繋がる

array = [1, 2, 3, 4, 5].map do |el| 
  if el.odd?
    el 
  end
end.compact!

このプログラムは配列に格納されている数字から奇数のみを取り出すプログラムです。
これをワンライナーで書き直します。

array = [1, 2, 3, 4, 5].map { |el| el if el.odd? }.compact!
array = (1..5).to_a.delete_if { |el| el.even? }
array = (1..5).to_a.delete_if(&:even?)
array = [1, 2, 3, 4, 5].select{ |el| el.odd?}

mapメソッドは配列から値を一つずつ取り出して変数に格納するメソッド
odd?は奇数を判別するメソッド
delete_ifは条件に合わない値を削除するメソッド
to_aメソッドは対象を配列としてまとめて返すメソッド
even?は偶数を判別するメソッド