Rubyのリファクタリング方法について

Question

こんにちは。

Ruby2,Rails4の環境でアプリケーションを開発しています。
リファクタリングについて質問させてください。
以下のようなコードがあり、配列で渡された値の中で
一番大きい値と二番目に大きい値か、
一番小さい値と二番目に小さい値か
のどちらかを取得したいです。
どちらかちいうのはフラグで判断します。

以下のコードでも動くのですが、冗長な部分が多くもっときれいに書けるとは思うのですが、どのように書けばよく分からず。。。

良い案あるかたお教えいただけないでしょうか?

    #一番目に大きい値と、二番目に大きい値の配列を取得する。
    for i in -25..-1 do
      if upper_flag==1
        if first_param[1] < sharping_befor_param[i][1]
          second_param=first_param
          first_param=sharping_befor_param[i]
        elsif second_param[1] < sharping_befor_param[i][1]
          second_param=sharping_befor_param[i]
        end
    #一番目に小さい値と、二番目に小さい値の配列を取得する。
      else
        if first_param[1] > sharping_befor_param[i][1]
          second_param=first_param
          first_param=sharping_befor_param[i]
        elsif second_param[1] > sharping_befor_param[i][1]
          second_param=sharping_befor_param[i]
        end
      end
    end

以上よろしくお願いします。

Answer 1

既にでている回答を参考に、次のコードをかいてみました。

def sored_take_n(ary, n = 1, order = :asc)
  sorted =
  case order
  when :asc
    ary.sort
  when :desc
    ary.sort.reverse!
  else
    fail ArgumentError.new("order = #{order}")
  end

  sorted.take(n)
end

ary = [10, 9, 1, 2, 5, 4, 3]
p ary.sort

p sored_take_n(ary, 2)
p sored_take_n(ary, 2, :desc)

p sored_take_n([0], 2)
p sored_take_n([], 2)

p sored_take_n(ary, 2, :xxx)

実行結果：

$ ruby ccc.rb
[1, 2, 3, 4, 5, 9, 10]
[1, 2]
[10, 9]
[0]
[]
ccc.rb:9:in `sored_take_n': order = xxx (ArgumentError)
	from ccc.rb:24:in `<main>'

Answer 2

rubyよく知らないんですけど
こんな感じでどうでしょう。

def take2(arr,assending)
    if(assending)
        return arr.sort.take(2)
    else
        return arr.sort {|a, b| b <=> a }.take(2)
    end
end

arr=[7,8,9,1,2,3,4,5,6]

puts take2(arr,true)
puts take2(arr,false)

take2という名前はとりあえずでつけたんで
もっと実体を表したいい名前つけて下さい。
「2個」というのもパラメータ化していいと思います。

Answer 3

その仕様で書き直すとこんな感じでしょうか。要素が数十個、数百個なら全体をソートするのが簡単です。
数百万個あるなら、お書きのようなループでしょうね。

first_param, second_param = 
if upper_flag==1
   sharping_befor_param.sort_by{|x| -x[1]}
else
   sharping_befor_param.sort_by{|x| x[1]}
end

なお、フラグに数値を使うのは良くないです。二値なら、true/false を使いましょう。