Rails: variables Partager entre le contrôleur et la vue

Question

Possible en double:
Passing only two variables between controller and view - best practice?

Il est mon action:

def list 
    @codes = Code.order("created_at") 
    @languages = Language.order('name').collect {|l| [l.name, l.coderay]} 
    end 

Il est mon avis (j'ai enlevé quelques lignes):

<% @codes.each do |code| %> 
    <div class="code"> 
     <%= link_to code.title, :action => 'show', :id => code.id %> 
     <% if code.author %> 
      @<%= code.author %> 
     <% end %> 
    </div> 
<% end %> 
<%= render :partial => 'shared/error_messages', :locals => {:object => @code} %> 
<%= form_for :code, :url => {:action => 'create' }, :html => {:multipart => true} do |f| %> 
    <%= f.text_field :title %><br /> 
    <%= f.text_area :content %><br> 
    <%= f.select(:language, @languages, {:selected => 'text'}) %> 
    <%= f.text_field :author %><br> 
    <%= f.submit "Submit code" %> 
<% end %> 

Il y a 3 variables: @codes (liste des articles), @code (poste actuel, utilisé dans une autre action) et @languages.

Mon IDE écrit:

Dans la plupart des deux variables d'instance doivent être partagées entre le contrôleur et vue
Cette inspection met en garde s'il y a plus de deux variables d'instance partagées entre un contrôleur et une vue. Un contrôleur doit gérer uniquement une variable d'instance, plus une seconde pour la variable current_user .

Habituellement, je partage plus de variables entre Controller et View (en PHP), parfois 10+.
Comment cela se fait-il dans Rails?

Answer 1

Vous pouvez enregistrer en faisant des langues une aide d'une instance var:

def languages 
    Language.order('name').collect {|l| [l.name, l.coderay]} 
end 

Answer 2

C'est simplement une bonne pratique de Rails.

Mais je lirais un peu plus sur Rails Routing. Comprendre le fonctionnement du routage Rails devrait vous donner une meilleure idée de la façon de structurer votre code.

http://guides.rubyonrails.org/routing.html

J'ai modifié votre code un peu, non testé. Mais j'espère que vous avez de bonnes idées.

Controller:

def new 
    @code = Code.new 
    @codes = Code.order("created_at") 
end 

def create 
    @code = Code.new(params[:code]) 
    if @code.save? 
     # Do your thing. 
    else 
     # render your :new action passing your @code variable 
    end 
end 

Vue:

<% @codes.each do |code| %> 
    <div class="code"> 

    # Use Rails Routing - In console, type rake routes to get list of routes.  
    <%= link_to code.title, code_path(code.id) %> # example. 

    <% if code.author %> 
     <%= code.author %> 
    <% end %> 
    </div> 
<% end %> 

    <%= render 'shared/error_messages', :object => @code %> 

    <%= form_for @code, :html => {:multipart => true} do |f| %> 
    <%= f.text_field :title %><br /> 
    <%= f.text_area :content %><br> 

    # language_list = helper method. 
    <%= f.select(:language, language_list, {:selected => 'text'}) %> 
    <%= f.text_field :author %><br> 
    <%= f.submit "Submit code" %> 
    <% end %>