メソッドに引数を入れたいのですが、うまく出来ません。

Question

Rubyです。以下のクイックソートで、メソッドに引数を使いたいのですが、うまくいきません。どなたかRubyに詳しい方教えていただけると幸いです。

# クイックソート
def quick_sort
    
    return self if self.size < 2
    
    
    pivot = self.shift
    
    
    smallers = []
    biggers = []
    self.each do |num|
        if num < pivot
            smallers << num
        else # num >= pivot
            biggers << num
        end
    end
    
    # smallersとbiggersをそれぞれソートして連結。
    smallers.quick_sort + [pivot] + biggers.quick_sort
    
end
a = [9,8,7,46,5,14,3,2,91]
p a.quick_sort

上記のコードでも動くのですが、def quick_sort(a)で配列aを引数に持ってきたいのですが、うまく出来ません。恐らくselfとの併用がうまくいっていないのではと思っています。Rubyに詳しい方ご指導いただけると幸いです。

Comments

[otn]

どのように書き換えて、何が問題なのかを質問文に追記してください。

Answer 1

意図は　Arrayを拡張するのではなく、独立したmethodにしたい、ということでしょうか？
でしたら
def qsort(a)
でやるとして、

1. self を全て　a　に変える
2. 最後の行を

quick_sort(smallers) + [pivot] + quick_sort(biggers)
にする
3) 呼び出しを p quick_sort(a) にする

Answer 2

オープンブラスについて勉強するとよいです。

• まつもと直伝　プログラミングのオキテ 第21回　オープンクラスとRuby on Rails

https://tech.nikkeibp.co.jp/it/article/COLUMN/20080619/308756/

質問文にある ソートメソッドは、 Array クラスに そのメソッドを追加するものとして書かれています。
(呼び出し法が a.quick_sort となっていますから。

次のようにすると、質問文のコードは動作しました。

# クイックソート
class Array
  def quick_sort
    return self if self.size < 2

    pivot = self.shift

    smallers = []
    biggers = []
    self.each do |num|
      if num < pivot
        smallers << num
      else # num >= pivot
        biggers << num
      end
    end

    # smallersとbiggersをそれぞれソートして連結。
    smallers.quick_sort + [pivot] + biggers.quick_sort
  end
end

a = [9, 8, 7, 46, 5, 14, 3, 2, 91]
p a.quick_sort