thr3ads.net - Rails - Arguments Error [Dec 2008]

If this information is useful, please help other people find it:
Share via:

Ryan Ororie

2008-Dec-30 18:28 UTC

Arguments Error

I''m new to rails so I apologize for my ignorance in advance...
I''m
working through a tutorial here but I think it may be written for a
slightly older version of rails. Trying to make a store app.

I''m getting the following error (with application trace shown):

ArgumentError in StoreController#add_to_cart

wrong number of arguments (1 for 0)

RAILS_ROOT: script/../config/..
Application Trace | Framework Trace | Full Trace

app/models/cart.rb:13:in `initialize''
app/models/cart.rb:13:in `new''
app/models/cart.rb:13:in `add_product''
app/controllers/store_controller.rb:10:in `add_to_cart''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/rails.rb:76:in
`process''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/rails.rb:74:in
`synchronize''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/rails.rb:74:in
`process''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:159:in
`process_client''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:158:in `each''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:158:in
`process_client''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:285:in `run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:285:in
`initialize''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:285:in `new''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:285:in `run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:268:in
`initialize''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:268:in `new''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel.rb:268:in `run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/configurator.rb:282:in
`run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/configurator.rb:281:in
`each''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/configurator.rb:281:in
`run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/bin/mongrel_rails:128:in
`run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/lib/mongrel/command.rb:212:in
`run''
/Library/Ruby/Gems/1.8/gems/mongrel-1.1.5/bin/mongrel_rails:281


Now I *think* this means that I have a problem with my
store_controller.rb file, which looks like this:

class StoreController < ApplicationController

  def index
    @products = Product.find_products_for_sale
  end

  def add_to_cart
    @cart = find_cart
    product = Product.find(params[:id])
    @cart.add_product(product)
  end

  private
  def find_cart
    session[:cart] ||=Cart.new
  end
end

But, I''m not sure - and it does seem a bit strange that that would be
where the error originates from as I didn''t change anything in this
file
from when it was working to when I got the error.

What I did do was create a model called cart_item.rb:

class CartItem
  attr_reader :product, :quantity

  def initialize
    @product = product
    @quantity = 1
  end

  def increment_quantity
    @quantity += 1
  end

  def title
    @product.title
  end

  def price
    @product.price * @quantity
  end
end

And edited the cart.rb model to include a more robust add_product, it
now looks like this:

class Cart
  attr_reader :items

  def initialize
    @items = []
  end

  def add_product(product)
    current_item = @items.find {|item| item.product == product}
    if current_item
      current_item.increment_quantity
    else
      @items << CartItem.new(product)
    end
  end
end

I had it pointed out to me that the line;
current_item = @items.find {|item| item.product == product}

Should be;
current_item = @items.detect {|item| item.product == product}

But this didn''t result in any change... would REALLY appreciate any
insight!
-- 
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
-~----------~----~----~----~------~----~------~--~---

Ryan Ororie

2008-Dec-30 18:32 UTC

head link

Re: Arguments Error

Also, my add_to_cart.rhtml view looks like this:

<h1>The shopping cart</h1>
<ul>
  <% for item in @cart.items %>
    <li><%= cart_item.quantity %> &times; <%= h(item.title)
%></li>
  <% end %>
</ul>

-- 
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
-~----------~----~----~----~------~----~------~--~---

Patrick Doyle

2008-Dec-30 18:44 UTC

head link

Re: Arguments Error

Your error message is:

wrong number of arguments (1 for 0)

That means that somewhere, you are calling a method that doesn''t expect
any
arguments and you are passing an argument to it.  The stack trace give you a
clue:

ArgumentError in StoreController#add_to_cart

app/models/cart.rb:13:in `initialize''
app/models/cart.rb:13:in `new''
app/models/cart.rb:13:in `add_product''
app/controllers/store_controller.rb:10:in `add_to_cart''
One of the methods listed (#add_to_cart, #initialize, #new, or #add_product)
is being called with an argument where it wasn''t expecting one.

StoreController#add_to_cart
-- doesn''t expect any methods, but this is called by the Rails
framework as
an action, and Rails doesn''t pass any arguments to the action methods
Cart#add_product
-- expects 1 argument, you''re calling it with 1 argument in
StoreController#add_to_cart
CartItem#new
-- calls CartItem#initialize with however many arguments it was passed
-- is called with 1 argument in Cart#add_product
CartItem#initialze
-- expects 0 arguments, you passed 1 argument in via CartItem#new

--wpd

--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Ryan Ororie

2008-Dec-30 20:55 UTC

head link

Re: Arguments Error

> StoreController#add_to_cart
> -- doesn''t expect any methods, but this is called by the Rails
framework
> as
> an action, and Rails doesn''t pass any arguments to the action
methods
> Cart#add_product
> -- expects 1 argument, you''re calling it with 1 argument in
> StoreController#add_to_cart
> CartItem#new
> -- calls CartItem#initialize with however many arguments it was passed
> -- is called with 1 argument in Cart#add_product
> CartItem#initialze
> -- expects 0 arguments, you passed 1 argument in via CartItem#new

I''m not really sure what do do with that -- what should I change? Are 
you saying only my CartItem#initialze is wrong?
-- 
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
-~----------~----~----~----~------~----~------~--~---

Patrick Doyle

2008-Dec-30 21:07 UTC

head link

Re: Arguments Error

>
> > CartItem#initialze
> > -- expects 0 arguments, you passed 1 argument in via CartItem#new
>
>
> I''m not really sure what do do with that -- what should I change?
Are
> you saying only my CartItem#initialze is wrong?
>Probably.

You could always add a parameter to your CartItem#initialize method. 
I''m
not sure that you''re heading in the direction you want to be heading,
since
it appears that CartItem is not derived from ActiveRecord::Base -- it''s
not
tied to a database.

You might want to go back and study Rail''s ideas regarding models, how
they
tie to database tables, and how you can initialize them.

--wpd

--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Ryan Ororie

2008-Dec-30 21:15 UTC

head link

Re: Arguments Error

> Probably.
> 
> You could always add a parameter to your CartItem#initialize method. 
> I''m
> not sure that you''re heading in the direction you want to be
heading,
> since
> it appears that CartItem is not derived from ActiveRecord::Base --
it''s
> not
> tied to a database.
> 
> You might want to go back and study Rail''s ideas regarding models,
how
> they
> tie to database tables, and how you can initialize them.
I following along from the book DHH wrote verbatim, but it''s the second
edition. It was my understanding that having a database table for the 
cart was unnecessary because it''s session data and would be avaliable 
based on that user?



-- 
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
-~----------~----~----~----~------~----~------~--~---

Freddy Andersen

2008-Dec-30 22:08 UTC

head link

Re: Arguments Error

Looks like you have two open posts for the same issue... Here is the
answer from the other post...

ahhhh Sorry I see now I was thinking that the CartItem class was an
activerecord class ... The issue is this:
Here you call a new object of CartItem with a product passed
@items << CartItem.new(product)
But here in the initialize you do not have a argument for
initialize...
class CartItem
  attr_reader :product, :quantity
  def initialize
    @product = product
    @quantity = 1
  end
Change it to this
class CartItem
  attr_reader :product, :quantity
  def initialize( product )
    @product = product
    @quantity = 1
  end

I would NEVER store the cartitem(add product to cart) in a session
object for a ecommerce application. Yes there are things that could/
should be stored in the session but not something like adding a
product to your cart... Unless you have a session replication system
for your back-end which then would store the session in either the
database or memcache anywho...

--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Ryan Ororie

2008-Dec-30 22:17 UTC

head link

Re: Arguments Error

> ahhhh Sorry I see now I was thinking that the CartItem class was an
> activerecord class ... The issue is this:
> Here you call a new object of CartItem with a product passed
> @items << CartItem.new(product)
> But here in the initialize you do not have a argument for
> initialize...
> class CartItem
>   attr_reader :product, :quantity
>   def initialize
>     @product = product
>     @quantity = 1
>   end
> Change it to this
> class CartItem
>   attr_reader :product, :quantity
>   def initialize( product )
>     @product = product
>     @quantity = 1
>   end
Interesting, tried that and it''s spitting a "no method for
cart_item
back at me:

 NameError in Store#add_to_cart

Showing app/views/store/add_to_cart.rhtml where line #4 raised:

undefined local variable or method `cart_item'' for 
#<#<Class:0x2602924>:0x26028e8>

Extracted source (around line #4):

1: <h1>The shopping cart</h1>
2: <ul>
3:   <% for item in @cart.items %>
4:     <li><%= cart_item.quantity %> &times; <%=
h(item.title) %></li>
5:   <% end %>
6: </ul>

RAILS_ROOT: script/../config/..
Application Trace | Framework Trace | Full Trace

#{RAILS_ROOT}/app/views/store/add_to_cart.rhtml:4:in 
`_run_rhtml_47app47views47store47add_to_cart46rhtml''
#{RAILS_ROOT}/app/views/store/add_to_cart.rhtml:3:in `each''
#{RAILS_ROOT}/app/views/store/add_to_cart.rhtml:3:in 
`_run_rhtml_47app47views47store47add_to_cart46rhtml''


Any idea?
-- 
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
-~----------~----~----~----~------~----~------~--~---

Freddy Andersen

2008-Dec-30 23:14 UTC

head link

Re: Arguments Error

Where does the cart_item come from? is that in the Store controller?
post the store controller... The view has the @cart object but also
needs the cart_item.. Is that from the session?
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Ryan Ororie

2008-Dec-30 23:27 UTC

head link

Re: Arguments Error

Freddy Andersen wrote:> Where does the cart_item come from? is that in the Store controller?
> post the store controller... The view has the @cart object but also
> needs the cart_item.. Is that from the session?
cart_item.rb is the name of a model file, the one that starts with class 
CartItem from above.

My store_controller.rb looks like this:

class StoreController < ApplicationController

  def index
    @products = Product.find_products_for_sale
  end

  def add_to_cart
    @cart = find_cart
    product = Product.find(params[:id])
    @cart.add_product(product)
  end

  private
  def find_cart
    session[:cart] ||=Cart.new
  end
end
-- 
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
-~----------~----~----~----~------~----~------~--~---

Angel Dinar

2010-May-14 12:43 UTC

head link

Re: Arguments Error

I have the exact same problem.
same book.
same code.
same error.

So, how did you finally solve the problem ?

Thanks
Angel
-- 
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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en.

Rails - Dec 2008 - Arguments Error

Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error

Re: Arguments Error