Почему код не работает?

Question

[Богдан]

money = 0
health = 100
def player_addhealth(addhealth):
    health + addhealth
    print("[DBG] Health added! now health = " + str(health))
def player_sethealth(sethealth):
    health = sethealth
    print("[DBG] Health set! now health = " + str(health))
def player_removehealth(removehealth):
    health - removehealth
    print("[DBG] Health removed! now health = " + str(health))
def player_addmoney(addmoney):
    money + addmoney
    print("[DBG] Money added! now money = " + str(money))
def player_setmoney(setmoney):
    money = setmoney
    print("[DBG] money set! now money = " + str(money))
def player_removemoney(removemoney):
    money - removemoney
    print("[DBG] Money removed! now money = " + str(money))
def player_showmoney():
    print("Money: " + str(money))
def player_showhealth():
    print("Health: " + str(health))
def player_debuginfo():
    player_showmoney()
    player_showhealth()
    print("[DBG] Info on screen!")
print(money)
print(health)
player_addmoney(10)
print(money)

не выдает ошибку, но половина функций не работает или работает странно.

Answer 1

[Сергей Горностаев]

Арифметические операторы не изменяют операнды. Они возвращают значение, которое нужно чем-то присвоить.

Comments

[Богдан]

А как тогда прибавить?
Пробовал money = money + addmoney, UnboundLocalError: local variable 'money' referenced before assignment

[Алан Гибизов]

Богдан, создайте другой вопрос и там приведите код, который выдаёт unboundlocalerror.
См.п.3.7

[Богдан]

Спасибо, разобрался.

Answer 2

[antonwx]

+= и -=, вместо просто + и -.

Comments

[Богдан]

UnboundLocalError: local variable 'money' referenced before assignment

[Сергей Горностаев]

Богдан, а это уже другой вопрос, про области видимости.

[Богдан]

Сергей Горностаев, Нужно сделать global?

[Сергей Горностаев]

Богдан, это решит проблему, но так делать не стоит, глобальные переменные - зло.

[Богдан]

Сергей Горностаев, Я так понял это не единственный выход, но как решить проблему по другому, я так и не понял.

[Сергей Горностаев]

Богдан, использовать чистые функции или объект.

Answer 3

[Владимир Куц]

А что вы ожидаете от кода?
В единственной функции, которую вызываете, вы выполняете арифметическое действие, но ничему его не присваиваете, потом просто печатаете значение money, который у вас равен 0
Ну и в итоге вновь печатаете money, которое, ожидаемо, - равно 0

Возможно, - вы просто слишком многого хотите от своего кода? ))