wrong number of arguments - in initialize method

Question

I am not able to figure out this extremely simple error:

xyz_controller.rb:
    @isbn = params[:q]
    @search_type = params[:search_type]
... 
    @prices = Generalsearch.new(:search_term => @isbn, :search_type => @search_type)

generalsearch.rb

    attr_accessor :search_term , :search_type
    def initialize(search_term , search_type)
      self.search_term= search_term
      self.search_type= search_type
    end
...

I keep getting

wrong number of arguments (1 for 2)
app/models/generalsearch.rb:11:in `initialize'
app/controllers/book_controller.rb:47:in `new'
app/controllers/book_controller.rb:47:in `view'

Naren Sisodiya · Accepted Answer · 2011-04-27 14:08:49Z

7

you are passing only one argument ie hash :search_term => @isbn, :search_type => @search_type in Generalsearch.new()

use

Generalsearch.new( @isbn, @search_type)

answered Apr 27, 2011 at 14:08

Naren Sisodiya

7,2882 gold badges27 silver badges35 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Dogbert · Accepted Answer · 2011-04-27 14:08:47Z

4

You have to use, as you're accepting 2 params on the initialize function, not a hash of of params.

@prices = Generalsearch.new(@isbn, @search_type)

answered Apr 27, 2011 at 14:08

Dogbert

224k43 gold badges419 silver badges417 bronze badges

Comments

Andrew Grimm · Accepted Answer · 2011-04-27 23:53:59Z

0

If you want to use

Generalsearch.new(:search_term => @isbn, :search_type => @search_type)

Then you can have in the initialize method

def initialize(options)
  # You can also use options[:search_term], 
  # but fetch lets you know if the key doesn't exist
  self.search_term= options.fetch(:search_term) 
  self.search_type= options.fetch(:search_type)
end

answered Apr 27, 2011 at 23:53

Andrew Grimm

82.3k58 gold badges210 silver badges355 bronze badges

Collectives™ on Stack Overflow

wrong number of arguments - in initialize method

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related