automatically instantiating objects

Question

players_list = [Ani, Paty, Felix, Alex]

class Player:

def __init__(self, name):
    self.name = name
    self.score = 0
    self.vote = 0
    self.player_hand = []
    self.choice = ''
    self.player_hand = []
            
def player_turn(self):
    print(self.name, "'s turn")


def p_vote(self):
    print(self.name, " voted")

I tried to iterate over the list, but it always gives me an error: NameError: name 'Ani' is not defined

for player in players_list:
    player = Player(str(player))

But doing all the process manually work:

Ani = Player("Ani"), etc

Is there any way that i can automate this process?

Why have you used quotes in the example that works, and not in the one that doesn't? — Open AI - Opting Out
– Open AI - Opting Out, Commented Jul 3, 2020 at 20:10

dewDevil · Accepted Answer · 2020-07-03 20:17:56Z

First of all the thing you should know, the players_list that you have declared are not containing strings, they are being considered as variables which you have not defined anywhere, and therefore the NameError.

Now, if you want to correct this, and if you actually intend to store objects of Player in players_list, then you can do the following:

players_list = ["Ani", "Paty", "Felix", "Alex"]

class Player:

    def __init__(self, name):
        self.name = name
        self.score = 0
        self.vote = 0
        self.player_hand = []
        self.choice = ''
        self.player_hand = []
            
    def player_turn(self):
        print(self.name, "'s turn")
    
    
    def p_vote(self):
        print(self.name, " voted")
        
for i in range(len(players_list)):
    players_list[i]=Player(players_list[i])

This will store Player objects in the list you have declared just the thing that you expect to get.

Bryce Wayne · Accepted Answer · 2020-07-03 20:22:40Z

0

You are having problems with the players not being defined. So players_list = [Ani, Paty, Felix, Alex] will throw an error because the objects Ani, Paty, Felizx, and Alex do not exist.

 class Player:
    def __init__(self, name):
        self.name = name
        self.score = 0
        self.vote = 0
        self.player_hand = []
        self.choice = ''
        self.player_hand = []
            
    def player_turn(self):
        print(self.name, "'s turn")


    def p_vote(self):
        print(self.name, " voted")

Now, we need to iterate through the list.

players_list = ['Ani', 'Paty', 'Felix', 'Alex']
players = [Player(player) for player in players_list]

answered Jul 3, 2020 at 20:22

Bryce Wayne

3515 silver badges18 bronze badges

Comments

mark_s · Accepted Answer · 2020-07-03 21:37:06Z

0

Sounds like you're trying to dynamically create variables - write code that writes code.

You could try to use the exec built-in function.

players = ['Ani', 'Paty', 'Felix', 'Alex']

class Player:
    def __init__(self, name):
        self.name = name

    def p_vote(self):
        print(self.name + " voted.")

for player in players:
    exec( "%s = Player( '%s' )" %(player, player) )

Ani.p_vote()

Although, general internet advice has two points to make:

Be cautious where you use exec.
The Pythonic way is to write out the variables, "explicit is better than implicit."

answered Jul 3, 2020 at 21:37

mark_s

5063 silver badges6 bronze badges

Collectives™ on Stack Overflow

automatically instantiating objects

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related