[初心者]Pythonのスコープについて2点質問（バグ・変数と関数の扱い）

Question

前提・実現したいこと

Pythonの基礎勉強のため、ドラクエのようなゲームを作ろうと思っています。

[3]銀行 を選ぶと500円もらえる設定し、1000 + 500円で所持金は 1500円になる予定ですが、
下記の通り1000円のままになってしまいます。

［質問］
1)本件のバグの原因は何でしょうか?

2)今後内容拡大に伴い関数が増えていくと、コメント # グローバル変数 の箇所のように
様々な部分で global ~ が無数に登場する気がするのですが、
基本的なやり方として別の方法があるのでしょうか？

それとも global ~ はある程度多く使うものでしょうか？

変数と関数の基礎的な扱いがまだよく分かっておらず、初歩的な質問ですがよろしくお願いします。

実行結果

誰と話す?[1]村人[2]武器屋[3]銀行 3
現在の所持金1000円
ここは銀行です
これをきみにあげよう
500円を手に入れた！
現在の所持金1000円
誰と話す?[1]村人[2]武器屋[3]銀行

該当のソースコード

# 所持金
money = 1000

# NPC
npc = {'villager': {'name': "村人",  'quote': "ここは最初の村です"},
       'armsshop': {'name': "武器屋", 'quote': "ここは武器屋だ"    },
       'banker'  : {'name': "銀行",   'quote': "ここは銀行です"    }}

# 現在のステータス表示
money_status = "現在の所持金" + str(money) + "円"

def status():
    print(money_status)

# 銀行イベント
def banker_event():
    print("これをきみにあげよう")
    get_money(500)

def get_money(x):
    global money # グローバル変数
    print(str(x) + "円を手に入れた！")
    money += x

# NPCを選択
choice_txt = "誰と話す?"
npc_choice = choice_txt + \
             "[1]" + npc['villager']['name'] + \
             "[2]" + npc['armsshop']['name'] + \
             "[3]" + npc['banker']['name']

def act():
    act_number = int(input(npc_choice))

    if act_number == 1:
        print(npc['villager']['quote'])
    elif act_number == 2:
        print(npc['armsshop']['quote'])
    elif act_number == 3:
        print(npc['banker']['quote'])
        banker_event()

# 実行
def main():
    for _ in range(5):
        status()
        act()

main()

補足情報（FW/ツールのバージョンなど）

OS:Mac
Editor:Atom

Answer 1

1)本件のバグの原因は何でしょうか?

money自体は確かに更新されています。
しかし、money_statusの値がmoneyの変化に伴って更新されることはありません。

関数statusを改良すればバグは解消するでしょう。

def status():
    print("現在の所持金" + str(money) + "円")

2)今後内容拡大に伴い関数が増えていくと、コメント # グローバル変数 の箇所のように

様々な部分で global ~ が無数に登場する気がするのですが、
基本的なやり方として別の方法があるのでしょうか？

クラスを作ります。
そうすれば、変数の影響範囲が縮まりますので。

それとも global ~ はある程度多く使うものでしょうか？

バグが蔓延する要因になるので、むしろ一切使わない方法を検討すべきです。

書いてみた

雑な作りですが、クラスを使った例です。

main.py

from player import Player
from non_player import choose_event


hero = Player(money=1000)
for _ in range(5):
    event = choose_event()
    
    hero.print_status()
    hero.process_event(
        event()
    )

    print()

player.py

from event import EventType


class Player:
    def __init__(self, money):
        self._money = money
        self._action_table = {
            EventType.DO_NOTHING: lambda: None,
            EventType.GET_MONEY: self._add_money,
        }
        
    def process_event(self, event):
        self._action_table[event.event_type](*event.args)
        
    def print_status(self):
        print(f'現在の所持金{self._money}')
        
    def _add_money(self, value):
        print(f'{value}円を手に入れた！')
        self._money += value

non_player.py

from event import Event, EventType


class _NonPlayer:
    def __init__(self, name, event):
        self.name = name
        self.event = event

        
def _event_stub():
    print('特に用はないよ')
    return Event(EventType.DO_NOTHING)

def _banker_event():
    print('ここは銀行です')
    print('これをきみにあげよう')
    return Event(EventType.GET_MONEY, 500)


npcs = [
    _NonPlayer('村人', _event_stub),
    _NonPlayer('武器屋', _event_stub),
    _NonPlayer('銀行屋', _banker_event),
]

def choose_event():
    print('誰と話す?', end='')
    for i, npc in enumerate(npcs, start=1):
        print(f'[{i}]{npc.name}', end='')
        
    num = int(input(' '))
    return npcs[num-1].event

event.py

from enum import Enum, auto


class EventType(Enum):
    DO_NOTHING = auto()
    GET_MONEY = auto()


class Event:
    def __init__(self, event_type, *args):
        self.event_type = event_type
        self.args = args

実行例

>python main.py
誰と話す?[1]村人[2]武器屋[3]銀行屋 1
現在の所持金1000
特に用はないよ

誰と話す?[1]村人[2]武器屋[3]銀行屋 2
現在の所持金1000
特に用はないよ

誰と話す?[1]村人[2]武器屋[3]銀行屋 3
現在の所持金1000
ここは銀行です
これをきみにあげよう
500円を手に入れた！

誰と話す?[1]村人[2]武器屋[3]銀行屋 2
現在の所持金1500
特に用はないよ

誰と話す?[1]村人[2]武器屋[3]銀行屋 3
現在の所持金1500
ここは銀行です
これをきみにあげよう
500円を手に入れた！

Comments

[go0x]

迅速な回答ありがとうございます。

>クラスを作ります。
>そうすれば、変数の影響範囲が縮まりますので。

クラスについて読んだり書いたりしているのですが、なかなか活用法が腑に落ちないできないのでよろしければ本件かその他の例でもう少し具体的に教えてもらえませんでしょうか？

[go0x]

サンプルありがとうございます！たった数行のコードでこんなに違うんですね...

全部読み込めてないのでじっくり勉強させてもらいます