Need help cleaning up a model method

[Giant Cranes]

Hi,

I need a little help cleaning up the method below. In my .NET days I
would have let this pass, but I am trying to be a good ruby citizen by
keeping things simple.

This is from a seller model which has many vehicles:

def total_retail_price
if @@total_retail_price.nil?
@@total_retail_price = self.vehicles.sumretail_price)
@@total_retail_price = 0 unless @@total_retail_price
end
@@total_retail_price
end

If I didn't have to worry about the seller having 0 vehicles I could
just use:

def total_retail_price
@@total_retail_price ||= self.vehicles.sumretail_price)
end

Does anyone have any suggestions on how I can fix improve on this?

Thanks,
GiantCranes

 

[Giant Cranes]

Apologies, I meant to post this in the Rails group.

 

[Jano Svitok]

what about this?

def total_retail_price
@@total_retail_price ||= self.vehicles.sumretail_price) || 0
end

 

[Giant Cranes]

Thanks to you both for your help