Rails 4 передает данные с контроллера на модель attr_accessor

Question

Хорошо, я очень смущен и немного застрял здесь, я пытаюсь передать данные в свою модель через attr_accessor, но я не могу найти правильный путь. Вот моя установка до сих пор:

class ApplicationController < ActionController::Base 
    before_filter :current_league 

    protected 

    def current_league 
    @current_league ||= Conf.all.order('updated_at ASC').last.league 
    end 
end 

class HerosController < ApplicationController 
    def index 
    @heros = Hero.all.order(:name) 
    end 
end 

class Hero < ActiveRecord::Base 
    attr_accessor :current_league 

    def some_method 
    puts current_league 
    end 
end 

<% @heros.each do |hero| %> 
    <tr> 
    <td><%= hero.some_method %></td> 
    </tr> 
<% end %> 

Теперь, как мне установить @current_league внутри моей модели? Я знаю, что у меня может быть attr_accessor внутри моей модели, но это относится только к экземпляру этой модели, но действие индекса не создает экземпляр, насколько мне известно. Может быть, кто-то может указать мне в правильном направлении. Заранее спасибо.

Answer 1

Переменная @heros - это коллекция экземпляров Hero. Вы можете пропустить их и установить, если хотите.

class HerosController < ApplicationController 
    def index 
    @heros = Hero.all.order(:name).each do |hero| 
     hero.current_league = current_league 
    end 
    end 
end 

Хотя он работает, я не нахожу этот ответ таким элегантным. Конечно, я не знаю, что вы делаете в полной мере, но на основе кода здесь я бы создал составной объект. Что-то вроде этого:

class HeroInLeague 

    attr_reader :league, :hero 

    def initialize(league, hero) 
    @league = league 
    @hero = hero 
    end 

    def some_method 
    # ... 
    end 

end 

Затем вы можете создавать эти объекты внутри контроллера:

class HerosController < ApplicationController 
    def index 
    @heros_in_league = Hero.all.order(:name).map { |hero| 
     HeroInLeague.new(current_league, hero) 
    } 
    end 
end 

Теперь вы создали место для методов идти, которые связаны с комбинацией героев и лиг. Почему это важно? Ну, с предыдущим подходом вы, вероятно, закончили бы методы Гера, которые не имеют никакого смысла, когда нет текущей лиги. (например, метод some_method). Это делает класс Hero немного беспорядочным. Теперь вы создали место для размещения some_method и всех его друзей.

Вы можете использовать delegators сделать интерфейс HeroInLeague немного более дружественным, так что вам не придется делать hero_in_league.hero.foo, но можно назвать hero_in_league.foo напрямую.