E' un corretto modo di scrivere codice agile?

Community IT
1

Scusa per il brutto oggetto ma non mi veniva niente di meglio.
In sostanza ho due modelli.
User e Delivery.
class Ruser < ActiveRecord::Base
has_many :deliveries, :dependent => :destroy

class Delivery < ActiveRecord::Base
belongs_to :ruser

Nella show action di User ho aggiunto:

<%= render :partial => ‘deliveries/form’ %>

In modo che visualizzando lo user possa anche avere la possibilita’ di
associarli una delivery.
Il form di delivery e’:

<%= form_for(@delivery) do |f| %>
<% if @delivery.errors.any? %>


<%= pluralize(@delivery.errors.count, “error”) %> prohibited
this delivery from being saved:

  <ul>
  <% @delivery.errors.full_messages.each do |msg| %>
    <li><%= msg %></li>
  <% end %>
  </ul>
</div>

<% end %>

<%= f.label :delivered_at %>: <%= l Date.today %>

<%= f.label :notes%>
<%= f.text_area :notes %>

<%= f.hidden_field :ruser_id, :value => @ruser.id %>

<%= f.submit %>
<% end %>

Come vedere ho aggiunto un hidden field in modo da associargli lo user.
E’ corretto come modo di procedere?
O c’e’ un modo piu in stile rails di fare queste cose.

2

Mi rendo conto che il tutto e’ relativo ma il dubbio e’ se tenere e
gestire delivery come modello a se stante quindi con i suoi
controller, le sue actions, create, ecc., e le sue views oppure
gestirlo legato al modello user, in quest’ultimo caso ad esempio avrei
il form per delivery nella show di user e la action create di delivery
nel controller di user.

2010/9/23 Mauro [email protected]: