Changing dictionaries of functions - Python

Question

So I've been creating a fairly basic rpg in my spare time, but I've reached a stumbling block. I want to make it so that only certain functions are accessible at a certain time by changing a dictionary of commands every time the player goes into/exits battle. However, the loop I set up for searching through the dictionary keys doesn't seem to work for any commands except for the ones initially written.

Main file:

    from commands import *

    Commands = {
        "travel": Player.travel,
        "explore": Player.explore,
        "help": Player.help,
        }

    p = Player()

    while (john_hero.health > 0):
        line = raw_input("=> ")
        args = line.split()
        if len(args) > 0:
            commandFound = False
            for c in Commands.keys():
                    if args[0] == c[:len(args[0])]:
                            Commands[c](p)
                            commandFound = True
                            break
            if not commandFound:
                    print "John's too simple to understand your complex command."

command.py

            class Player:
                def __init__(self):
                    self.state = "normal"
                    john_hero = John()
                    self.location = "Town"
                    global Commands
                    Commands = {
                            "attack": Player.attack,
                            "flee": Player.flee,
                            "help": Player.help
                            }
                def fight(self):
                    Player.state = "fight"
                    global Commands
                    enemy_a = Enemy()
                    enemy_name = enemy_a.name
                    print "You encounter %s!" % (enemy_name)

*Note: The loop was taken from someone else's code. I'm using it since I'm creating the game mostly for learning purposes.

@arunkumar Ah, I've just added it. In retrospect it was rather foolish of me not to include it. — IndustrialPioneer
– IndustrialPioneer, Commented Mar 24, 2013 at 4:55

Community · Accepted Answer · 2017-05-23 12:11:06Z

1

It seems that your code in command.py trying to modify a global variable that is defined in Main file, in other words, something like this: Global Variable from a different file Python

This doesn't work because your code now has two Commands variables, one in the scope of command.py, one in the scope of Main file. Rather than trying to make two files share a global variable (which is a rather terrible idea IMO), I suggest you make Commands an attribute of Player:

class Player:
    def __init__(self):
        ...
        self.Commands = {
            "attack": Player.attack,
            "flee": Player.flee,
            "help": Player.help
        }

edited May 23, 2017 at 12:11

CommunityBot

11 silver badge

answered Mar 24, 2013 at 9:55

xuanji

5,1173 gold badges28 silver badges35 bronze badges

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

Comments

John · Accepted Answer · 2013-03-23 23:57:24Z

0

I'd do something like

commands = {"travel":{"cmd":Player.travel, "is_available":True}}
for key in commands:
    if commands[key]["is_available"]:
        print "do stuff"

but as pointed out by @arunkumar it is going to be difficult to answer this question without more of the code.

answered Mar 23, 2013 at 23:57

John

13.8k7 gold badges53 silver badges104 bronze badges

Collectives™ on Stack Overflow

Changing dictionaries of functions - Python

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related