配列を順番に処理するメソッドの色んな書き方

f:id:ryoutaku_jo:20190113142620p:plain

【結論】
・eachやmapなど、配列の要素毎に処理を
　繰り返すメソッドは、省略した書き方がある

・do...endの記述は、中カッコ{}で代用できる

・{|変数| object.method}の代わりに
　(&:method)と記述することも可能

・但し、一行で書く方法は、可読性が悪くなるので、
　複数行に亘る処理を書く場合は、「do...end」推奨

【目次】

ワンライナーで書き直せという問題
do...endの記述は、中カッコ{}で代用できる
(&:method)と記述することも可能
可読性の問題
所感
《今日の学習進捗》

【本題】

ワンライナーで書き直せという問題

今日、スクールでこういった問題が出ました。

・問題
以下をワンライナー（一行）で記述してください

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

処理の流れは、下記の通り。

１：中身が[1, 2, 3, 4, 5]の配列オブジェクトarrayを生成
２：mapで、要素毎に下記で指定する処理を実行。
　　要素は変数elと定義して、処理中に取得できる様にする。
３：「if el.odd?」で、取得した要素(el)が奇数(odd?)か判定。
４：奇数(true)なら、elを返す。
奇数で無い(false)なら、何も返さない(nil)
５：３〜４までを全ての要素毎に繰り返す
６：mapにより、５の返り値を集めた配列を作成。
７：end.compact!により、６で作成した配列から
nilの要素を取り除いて、新しい配列を作成。

参考
ref.xaio.jp

後置ifは覚えていたのですが、
繰り返し文をワンライナーで書く方法が思いつきませんでした。

do...endの記述は、中カッコ{}で代用できる

そして答えが、こちらでした（一例）

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, 2, 3, 4, 5].select{ |el| el.odd?}