コードで2つの配列が正しく並べ替えられないのはなぜですか？

Question

この問題を解決するための私の試みでの更新です。 なぜアレイを逆順にソートするのですか？ロジックは私にとって意味があります。

def combine_array(arr1,arr2) 
arr3 = [] 
arr1.each do |x| 
    arr3.push(x) 
end 

arr2.each do |x| 
    arr3.push(x) 
end 

arr3.each_index do |i| 
    arr3.each_index do |j| 
     if arr3[i] > arr3[j] 
     hold = arr3[i] 
     hold2 = arr3[j] 
     arr3[i] = hold2 
     arr3[j] = hold 
     end 
end 
end 
return arr3 

端

combine_array（[1,5,3]、[6,2,4]）

Click link to see the code and the array in reverse order

Answer 1

1. あなたはarr3[i+1..-1].each_index do |j|にわたって反復し、次に記しますfo arr3[j]ですが、それはおそらく意図していないarr3の正面部分です。 jはarr3[i+1..-1]の中のインデックスであり、arr3ではありません。

2. あなたはarr3を一度だけ通過し、すべての配列をアルゴリズムで1回のパスでソートすることはできません。

Answer 2

あなたはここで、.sortメソッドを使用しない場合は興味があるかもしれないもう一つのアプローチは、次のとおりです。 Sorting an Array in Ruby without using Sort method

Answer 3

これは、ソリューション

def combine_array(arr1,arr2) 
arr3 = [] 
arr1.each do |x| 
    arr3.push(x) 
end 

arr2.each do |x| 
    arr3.push(x) 
end 

arr3.each_index do |i| 
    j = i+1 
    until j == arr3.length do 
     if arr3[i] > arr3[j] 
      hold = arr3[i] 
      hold2 = arr3[j] 
      arr3[i] = hold2 
      arr3[j] = hold 
     end 
     j+=1 
    end 
end 
return arr3 

エンド

です

combine_array（[1,5,3]、[6,2,4]）