Python - classes and methods

Question

I am trying to create a class called trader that has instance attributes price, action and number, where action is an empty list and number is initially set to 0.

Within this class, i need to define a method called takeAction, so for each price, will set action equal to ‘buy’ if the price is lower than 50, ‘sell’ if the price is greater than 90, and ‘hold’ otherwise. So at each price the list action should be filled with the action that has been taken, i.e. buy, sell or hold, and number should increase by 1 if the action is ‘buy’, decrease by 1 if the action is ‘sell’ or remain constant otherwise.

My code below, does not increment the number by 1, since price is 30 so <50, I expect to increment number by 1, when I access t1.number I get the output 0. Not sure where my logic is failing, any pointer appreciated. Also just new to programming, and learning python so may have made a very obvious mistake!

class trader:
    action=[]
    number=0

    def __init__ (self,price):
        self.price=price
        
        
    def takeAction(self):
        if self.price<50:
            action.append('BUY')
            number+=1

        elif self.price>90:
            action.append('SELL')
            number = number-1

        else:
            action.append('HOLD')
            number
        return action

My instance Variable: t1=trader(30) t1.number output: 0

use self.action and self.number

splash58
– splash58

2023-11-30 16:29:44 +00:00
Commented Nov 30, 2023 at 16:29 — splash58
– splash58, Commented Nov 30, 2023 at 16:29

Jab · Accepted Answer · 2023-11-30 16:30:59Z

0

You should have action and number as instance values not class values. Instantiate them in the init. Then refer to them using self.

class trader:
    def __init__ (self,price):
        self.action=[]
        self.number=0
        self.price=price
        
        
    def takeAction(self):
        if self.price<50:
            self.action.append('BUY')
            self.number+=1

        elif self.price>90:
            self.action.append('SELL')
            self.number-=1

        else:
            self.action.append('HOLD')

        return self.action[-1]

answered Nov 30, 2023 at 16:30

Jab

27.6k22 gold badges81 silver badges117 bronze badges

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

Comments

Sem · Accepted Answer · 2023-11-30 16:31:07Z

0

Correct me if i'm wrong but you have to put the action var and the number var in the init as a self.action and self.number

like this the code worked for me

class trader:


def __init__ (self,price):
    self.price=price
    self.action=[]
    self.number=0
    
    
def takeAction(self):
    if self.price<50:
        self.action.append('BUY')
        self.number+=1

    elif self.price>90:
        self.action.append('SELL')
        self.number = self.number-1

    else:
        self.action.append('HOLD')
        self.number
    return self.action


trade = trader(100)


print(trade.takeAction())

I'm also learing myself OOP on python good luck and I hope I could help!

answered Nov 30, 2023 at 16:31

Sem

156 bronze badges

Collectives™ on Stack Overflow

Python - classes and methods

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related