I have the following code:
class BookPrice attr_accessor :price def initialize(price) @price = price end def price_in_cents Integer(price*100 + 0.5) end end b = BookPrice.new(2.20) puts b.price_in_cents This all works well and produces 220. But when I replace the second line attr_accessor :price with:
def price @price = price end I get stack level too deep (SystemStackError) error. What's going on? I know I can replace Integer(price*100 + 0.5) with @price instead of the method call price, but I want to keep it the way it is for OOP reasons. How can I make this code work the way it is without attr_accessor?
4 Answers
Your below code
def price @price = price # <~~ method name you just defined with `def` keyword. end Creates never stopable recursion,.
How can I make this code work the way it is without attr_accessor?
You need to write as
def price=(price) @price = price end def price @price end You need to do:
@price = self.price to differentiate between your object attribute price and your method parameter price.
One general reason this can happen is if you have some method that calls itself recursively (i.e. calls itself inside itself, which causes an infinite loop).
That's the problem I had that led to the error message.
In your case that's what's happening here:
def price @price = price end read_attribute is what you are looking for
def price @price = read_attribute(:price) end 
