Problem with task objectoriented-python-2/advanced-objects/setters

You're updating the base price (_price) here.

So, unlike the property method that returns the price with tax calculated, the setter method does not need any calucation. Just copy the argument into the base price.

hi Steven,

it's not that I don't believe you, and yes, I changed the code as you indicated and it passed, but honestly, this does not make sense at all,

running the code that passed does not return a logical result:

class Product: _price = 0.0 tax_rate = 0.12

def __init__(self, base_price):
    self._price = base_price

@property
def price(self):
    return self._price + (self._price * self.tax_rate)

@price.setter
def price(self, price):
    self._price = price #price / (1 + self.tax_rate)

prod = Product(100) print 'prod._price:', prod._price print 'prod.price:', prod.price prod.price = 200 print 'prod._price:', prod._price print 'prod.price:', prod.price

if you set the price via the setter, then this should update the base price deducting the tax rate from the new price, just as Kenneth showed in the video with a similar example,

anyway,

thank you for getting back to me on this topic and for the feedback provided,

cheers, Dieter

hi Steven,

the way I read it, you need to update the price, not the base price, hence the price you set is including tax (which does not require a calculation, I agree), this would mean though that when you set the price you need to update the base price since that would change implicitly,

in the video before, Kenneth is doing a similar exercise, his setter also updates "a base element",

thank you, Dieter

ok, thank you for that note, see you around, Dieter

I actually just actually did this exercise. I just passed it because I looked up an answer(the first time i did so for a quiz), but it really doesn't make sense to me at all. I wish someone could give a good explanation why this works:

@price.setter def price(self, price): self._price = price

and this doesn't:

@price.setter def price(self, price): self._price = price / (1 + self.tax_rate)

for some reason we don't have to specify or modify anything for this example. We just use the parameter's unadulterated value and it magically gives us the price before tax. I mean I wouldn't make such a big deal out of it if it weren't for the fact that there is a huge issue of consistency here. In the video just before this we had to specify that self.diameter was equal to the radius MULTIPLIED by 2.

@radius.setter def radius(self, radius): self.diameter = radius * 2.

with the logic of how we solved the quiz this should work perfectly:

@radius.setter def radius(self, radius): self.diameter = radius

But it doesn't. Am I missing something here?