# Is it a good rails agile code?

**URL:** <https://www.ruby-forum.com/t/is-it-a-good-rails-agile-code/195504>\
**Category:** Rails\
**Created:** [September 23, 2010, 11:58pm UTC](https://www.ruby-forum.com/t/is-it-a-good-rails-agile-code/195504 "2010-09-23T23:58:47Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![msan](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/m/3ab097/32.png) [@msan](https://www.ruby-forum.com/u/msan)\
**Post date:** [September 23, 2010, 11:58pm UTC](https://www.ruby-forum.com/t/is-it-a-good-rails-agile-code/195504/1 "2010-09-23T23:58:47Z")

</div>

I have two classes: User and Delivery.

class Ruser \< ActiveRecord::Base  
has\_many :deliveries, :dependent =\> :destroy

class Delivery \< ActiveRecord::Base  
belongs\_to :ruser

In the User show action I have put:

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

So that showing the user I can create a delivery for it.  
Delivery form is:

\<%= 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>

```

 \<% 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 %\>

Is it a good code or there is a better mode to create a delivery for one  
user?