CODEWITHSUNDEEP
pythonpython-3.xclassvariablesadventure
Chris
Chris

Reputation: 23

Python class variable changes not saving

I am trying to work on a text adventure to help keep my interest while trying to learn the concepts and nuances of Python. I originally followed along to a YouTube tutorial (6 part series, part 1 link provided: https://www.youtube.com/watch?v=MFW8DJ6qsak).

I created the player class as shown:

class Player:
    def __init__(self):
        self.name = ''
        self.jobclass = ''
        self.currenthp = 0
        self.totalhp = 0
        self.currentmp = 0
        self.totalmp = 0
        self.physical_damage = 0
        self.physical_defense = 0
        self.magic_attack = 0
        self.magic_defense = 0
        self.speed = 0
        self.luck = 0
        self.status_effect = []
        self.location ='b2'
        self.game_over = False

myPlayer = Player()

My current issue is trying to assign the player certain stats based on their chosen character job i.e.:

def setup_game():
    ...
    player_jobclass = input("> ")
    myPlayer.jobclass = player_jobclass
    valid_jobs = ['warrior', 'mage', 'thief', 'cleric']
    while player_jobclass.lower() not in valid_jobs:
        print("Please choose a valid class")
        player_jobclass = input("> ")
    if player_jobclass.lower() in valid_jobs:
            myPlayer.jobclass = player_jobclass
            print("Ah, the " +player_jobclass.lower()+ ". Fine choice!\n")

    #Class Starting Stats
    if myPlayer.jobclass is 'warrior':
        self.currenthp = 20
        self.totalhp = 20
        self.currentmp = 10
        self.totalmp = 10
        self.physical_damage = 10
        self.physical_defense = 10
        self.magic_attack = 2
        self.magic_defense = 3
        self.speed = 4
        self.luck = 4
        self.location = 'd1'
    elif myPlayer.jobclass is 'mage':
        self.currenthp = 10
        self.totalhp = 10
        self.currentmp = 20
        self.totalmp = 20
        self.physical_damage = 2
        self.physical_defense = 3
        self.magic_attack = 10
        self.magic_defense = 7
        self.speed = 7
        self.luck = 3
        self.location = 'a1'
    ...
    main_game_loop()

Because after this all runs, I get into the game fine, and all my movements and little actions work fine, however if I try to print the player "stats" via something like:

def show_stats():
hpmpadarmPrint = ("" + str(myPlayer.currenthp) + "/" + str(myPlayer.totalhp) + " hp " + str(myPlayer.currentmp) + "/" + str(myPlayer.totalmp) + " mp " + str(myPlayer.physical_damage) + " ATK " + str(myPlayer.physical_defense) + " ARM \n")
mgclckspdPrint = ("" + str(myPlayer.magic_attack) + " AP " + str(myPlayer.magic_defense) + " MR " + str(myPlayer.speed) + " SPD " + str(myPlayer.luck) + " LCK \n")

All that prints is:
0/0 hp 0/0 mp 0 ATK 0 ARM 0 AP 0 MR 0 SPD 0 LCK

I can tell there is some larger concept that is eluding me as far as assigning the values as even the self.location = 'b2' MUST be there or the game won't run as myPlayer is not actually being assigned the values I believe it should. This also prevents my change_text_speed() function from working and also ... you get it.

Guess main question would be: how to keep the new values once assigned?

Upvotes: 2

Views: 3414

Answers (2)

user1532172
user1532172

Reputation:

self and myPlayer are two different objects. There are two fixes you could make. First, replace self with myPlayer when setting the stats.

if myPlayer.jobclass is 'warrior':
    myPlayer.currenthp = 20
    myPlayer.totalhp = 20
    myPlayer.currentmp = 10
    ...

The second and in my option, better approach is to set the stats in __init__. It would look something like this.

if self.jobclass is 'warrior':
    self.currenthp = 20
    self.totalhp = 20
    self.currentmp = 10
    ...

To make the code even better, read about inheritance and make a class for each jobclass. You called it job class after all.

Upvotes: 3

Mureinik
Mureinik

Reputation: 311053

is checks if two variables point to the exact same object. In cases like your's, when inputting a user strings, you'll most probably have two different string objects with the same value. E.g.:

>>> x = input("")
ab
>>> x
'ab'
>>> x == "ab"
True
>>> x is "ab"
False

To make a long story short - use ==, not is when you check the players jobClass.

Upvotes: 1

Related Questions