problem with class method

[AmirAB]

Hi experts. I am new to Python but understand programming and have a fair idea about it. I need your help for following.

I am trying to execute the following code but getting an error. Both error and code provided below.

1 2 3 4 5 6 7 8 9 10 11 12

class Circle():     pie=3.14     def __init__(self,radius=1):         self.radius=radius         self.area=pie*radius*radius                         def Circum(self):         return(2*self.pie*self.radius)   mycircle=Circle() mycircle.Circum()     # I get error here when calling this function

Error:
TypeError: Circum() takes 0 positional arguments but 1 was given

[buran]

hm, that is strange. The actual error I get is

Error:
Traceback (most recent call last):
  File "/home/boyan/sandbox/classes/enemy.py", line 11, in <module>
    mycircle=Circle()
  File "/home/boyan/sandbox/classes/enemy.py", line 5, in __init__
    self.area=pie*radius*radius
NameError: name 'pie' is not defined

you can fix that by changing it to self.pie
then the code will run, but in order to see the output of Circum, you need to add print()
e.g. print(mycircle.Circum())

[snippsat]

As your radius is a default argument,then so can pie be.
It can also be be a class attribute with self.pie as @buran suggests.

1 2 3 4 5 6 7 8

class Circle:     def __init__(self, radius=1, pie=3.14):         self.radius = radius         self.pie = pie         self.area = self.pie * self.radius * self.radius       def circum(self):         return(2 * self.pie * self.rad

Use:

1 2 3 4 5 6 7 8

>>> mycircle = Circle() >>> mycircle.circum() 6.28   >>> # Can change radius,pie is the same as it always has been >>> mycircle = Circle(2) >>> mycircle.circum() 12.56

Also look at PEP-8,for small changes that i made.

[AmirAB]

As your radius is a default argument,then so can pie be. It can also be be a class attribute with self.pie as @buran suggests.

1	class Circle: def __init__(self, radius=1, pie=3.14): self.radius = radius self.pie = pie self.area = self.pie * self.radius * self.radius def circum(self): return(2 * self.pie * self.rad

Use:

1	>>> mycircle = Circle() >>> mycircle.circum() 6.28 >>> # Can change radius,pie is the same as it always has been >>> mycircle = Circle(2) >>> mycircle.circum() 12.56

Also look at PEP-8,for small changes that i made.

Great ... :)
Thanks a lot Bros...it feels awesome getting so much help..
Bundle of thanks both of you @buran, @snippsat