I am trying to loop through records in a db and pull out a particular
field. I have no problem with the code when I run it in the console;
however when I try to transfer the code to my Rails environment it
fails.
The code I''m using in my console is:
show = Schedule.find(:all)
n = show.size - 1
0.upto(n) { |i| puts show[i].venue }
The above code runs perfectly. To adapt this code for Rails, I am doing
the following:
(IN THE CONTROLLER)
class ScheduleEditController < ApplicationController
def index
@show = Schedule.find(:all, :order => ''date ASC'')
end
end
(IN THE VIEW)
<ul>
<%= render :partial => ''shows'' %>
</ul>
(IN THE PARTIAL)
<% n = @show.size - 1 %>
<li><%= 0.upto(n) { |i| puts @show[i].date } %></li>
The result I get is this:
<ul>
<li>0</li>
</ul>
The result I am looking for is:
<ul>
<li>2</li>
<li>3</li>
<li>1</li>
</ul>
Yes, I am obviously a noob. Please help shed light on where I''ve gone
wrong before I tear all my hair out.
--
Posted via http://www.ruby-forum.com/.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---
On 27 Mar 2008, at 19:28, Sara Me wrote:> (IN THE PARTIAL) > <% n = @show.size - 1 %> > <li><%= 0.upto(n) { |i| puts @show[i].date } %></li> >When you have a n erb output block (<%= ... %>) what appears in the view is what the expression evaluates to. calls to puts inside there are immaterial n.upto evaluates to n, hence the result you''re getting You need something like <% 0.upto(n) do %> <li><%= @show[i].date %> </li> <% end %> That''s not a very idiomatic way of doing it though. More idiomatic would be <% @show.each do |show| %> <li><%= show.date %> </li> <% end %> or using a partial (although it might be a bit overkill in this particular case) Fred> > The result I get is this: > <ul> > <li>0</li> > </ul> > > The result I am looking for is: > <ul> > <li>2</li> > <li>3</li> > <li>1</li> > </ul> > > Yes, I am obviously a noob. Please help shed light on where I''ve gone > wrong before I tear all my hair out. > -- > Posted via http://www.ruby-forum.com/. > > >--~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Sara Me wrote:> (IN THE PARTIAL) > <% n = @show.size - 1 %> > <li><%= 0.upto(n) { |i| puts @show[i].date } %></li>The #puts method won''t output into the view... In the view, put: <%= render :partial => ''shows'', :collection => @show %> This will cause the partial to run for each item in the @show list. This saves you from iterating over them yourself. Then, in the partial, put: <li><% shows.date %></li> There will be a local variable of the same name as the partial (shows in this case) for each item in the collection. For your iteration in your console session, you could more easily do: show = Schedule.find(:all) show.each {|s| puts s.venue} It is very rare in Ruby code to ever need to refer to indices at all. -- Posted via http://www.ruby-forum.com/. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Sara Me wrote:> I am trying to loop through records in a db and pull out a particular > field. I have no problem with the code when I run it in the console; > however when I try to transfer the code to my Rails environment it > fails. > > The code I''m using in my console is: > > show = Schedule.find(:all) > n = show.size - 1 > 0.upto(n) { |i| puts show[i].venue } > > The above code runs perfectly. To adapt this code for Rails, I am doing > the following: > > (IN THE CONTROLLER) > class ScheduleEditController < ApplicationController > def index > @show = Schedule.find(:all, :order => ''date ASC'') > end > end > > (IN THE VIEW) > <ul> > <%= render :partial => ''shows'' %> > </ul> > > (IN THE PARTIAL) > <% n = @show.size - 1 %> > <li><%= 0.upto(n) { |i| puts @show[i].date } %></li> > > > The result I get is this: > <ul> > <li>0</li> > </ul> > > The result I am looking for is: > <ul> > <li>2</li> > <li>3</li> > <li>1</li> > </ul> > > Yes, I am obviously a noob. Please help shed light on where I''ve gone > wrong before I tear all my hair out. >First change your @show to @shows (it has more than one, so for readability, make it plural) Second, change your partial to _show, not _shows. Next In your view, do this... <ul> <%= render :partial => ''show'', :collection => @shows %> </ul> And then your partial can just be... <li><%= show.date %></li> No need to to the upto loop. passing :collection to render will iterate through items and re-render the partial for each. -- http://www.5valleys.com/ http://www.workingwithrails.com/person/8078 --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Jon Garvin wrote:>> > First change your @show to @shows (it has more than one, so for > readability, make it plural) > Second, change your partial to _show, not _shows. > > Next > In your view, do this... > > <ul> > <%= render :partial => ''show'', :collection => @shows %> > </ul> > > > And then your partial can just be... > <li><%= show.date %></li> > > > No need to to the upto loop. passing :collection to render will iterate > through items and re-render the partial for each.YESSSS!!!!!! Thank you so much. I understand it now. -- Posted via http://www.ruby-forum.com/. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---