変数に配列名を代入してRoRでソートする

Question

絶対RoR初心者ここでは、ループ内で複数のリーグをレンダリングしようとしていますが、毎回div_＃をインクリメントしています。 div_1やdiv_2をソートするのは難しいですが、div_nameは正しく機能しません。なぜなら、正しい内容を持っていても、配列として見えるようにする必要があります。

<% div_1 = Array.new 
    div_1 << { :Name => 'Rob', :Played => '2', :Won => '1', :Lost => 1, :Points => 4} 

    div_2 = Array.new 
    div_2 << { :Name => 'Gavin', :Played => '2', :Won => '1', :Lost => 1, :Points => 4} 

    for i in (1..2) 

    i = i.to_s 
    div_name = "div_" + i 

    div_name.sort_by { |position| position[:Points] }.reverse!.each do |position| %> 

<%= position[:Name] %> 

Answer 1

問題の多くがここにあります

div_1 = Array.new 
div_1 << { :Name => 'Rob', :Played => '2', :Won => '1', :Lost => 1, :Points => 4} 

div_1は今ハッシュある単一の要素を持つ配列です。要素が1つしかない場合、配列は必要ありません。

for i in (1..2) 

このループのブロックはどこですか？その文の後、iはまだ定義されていませんので、あなたは

i = i.to_s 

を呼び出すときにはNameErrorを取得します。

div_name = "div_" + i 

でもi == 1場合、div_nameは値を持つ文字列「div_1」、およびないあなたは上記の定義div_1変数のコピーまたはインスタンスになります。

div_name.sort_by { |position| position[:Points] }.reverse!.each do |position| %> 

今、あなたはそれが意味をなさないので、それに応答しない文字列、上sort_byを呼び出すようにしようとしています。

<%= position[:Name] %> 

この範囲では、positionという名前の変数は定義されていません。

また、<% %>タグ内のビュー内にたくさんのロジックを入れていると、そのコードをコントローラのように別の場所に移動する必要があるという兆候です。あなたが定義し、コントローラ内のハッシュの配列として@positionsを計算して、ビューで何か行うことができます：私はそれを必要とするように指導のための

<% @positions.each do |position| %> 
    <%= position[:name] %> 
<% end %>