Setting initial instance parameter using a class method

Question

I have this code:

import random

class Player:

    def __init__(self):
        self.first_name = set_first_name()

    def set_first_name(self)
        List = open("player/first_names.txt").readlines()
        self.first_name = random.choice(List)

As you can see, I would like to set first name randomly from a text file. But I receive this error:

def set_first_name(self) ^ SyntaxError: invalid syntax

I assume it is not possible to call a class method within the initialisation of a class instance. At least not the way I am doing it. Could sombody give me a quick hint? I suppose there is an easy fix to this.

Thanks

You missed : after def set_first_name(self)

Yaroslav Surzhikov
– Yaroslav Surzhikov

2017-10-14 09:14:06 +00:00
Commented Oct 14, 2017 at 9:14 — Yaroslav Surzhikov
– Yaroslav Surzhikov, Commented Oct 14, 2017 at 9:14
Oh man. Thanks and sorry for the spam.

Maximilian A. Graves
– Maximilian A. Graves

2017-10-14 09:18:01 +00:00
Commented Oct 14, 2017 at 9:18 — Maximilian A. Graves
– Maximilian A. Graves, Commented Oct 14, 2017 at 9:18

Yaroslav Surzhikov · Accepted Answer · 2017-10-14 09:24:54Z

1

First of all as was allready mentioned - you missed : in define line. Second: even if you fix that - you will get NameError because set_first_name is not in global scope. And for last - set_first_name doesn't returns anything, so you will get first_name as None.

Assuming that, right version of your code should look like:

import random

class Player:

    def __init__(self):
        self.first_name = self.set_first_name()

    @staticmethod
    def set_first_name():
        List = open("player/first_names.txt").readlines()
        return random.choice(List)

edited Oct 14, 2017 at 9:24

answered Oct 14, 2017 at 9:19

Yaroslav Surzhikov

1,6081 gold badge13 silver badges16 bronze badges

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

1 Comment

Maximilian A. Graves Over a year ago

Thanks Yaroslav. I did your changes. After adding "self" as argument to set_first_name(), then it works perfectly. Thanks!

Stuart Buckingham · Accepted Answer · 2017-10-14 09:14:04Z

1

Your method is not a class method, you are simply missing the semi-colon from the end of your def line of the set_first_name method.

answered Oct 14, 2017 at 9:14

Stuart Buckingham

1,86419 silver badges28 bronze badges

2 Comments

Stuart Buckingham Over a year ago

No problem. Please upvote and accept as answer if it helped.

Stuart Buckingham Over a year ago

Also, beware of using the term 'class method'. It refers to a specific way of calliing a method. Info is here stackoverflow.com/q/12179271/4047084 if you're interested.

Collectives™ on Stack Overflow

Setting initial instance parameter using a class method

2 Answers 2

1 Comment

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related