初心者です。文法の間違いがわかりません

Question

さいころを投げて先に20に到達した人が勝ちというゲームを関数を使って作ったのですがうまくいきません。どこが間違ってますか?必ずyou winになります。

import random
your=0
me=0
def saikoro(player):
    daiz=random.randrange(1,7)
    if(player=='you'):
        input('push enter')
        global your
        your+=daiz
    else:
        global me
        me+=daiz
    return True
def checker(counter):
    if(your or me >=20):
        if(your>=20):
            print('you win')           
        else:
            print('i win')
while(True):
    saikoro('you')
    checker(your)
    saikoro('me')
    checker(me)
            
    
コード

Answer 1

グローバル変数を使わないようにしてみました。

wc.py

import random

def play(name, point):
  daiz = random.randrange(1, 7)
  if player == 0:
    input('push enter')
  return point + daiz

def check_winer(player, point, name):
  print( "  ", name, point)
  if point >= 20:
    print(name + " win")
    return name      


player_infos = {
  0: {'name': 'you', 'point': 0, 'winer_name': 'you'}, 
  1: {'name': 'me', 'point': 0, 'winer_name': 'i'}
}
player = 0
while(True):
  player_infos[player]['point'] = play(player, player_infos[player]['point'])
  if check_winer(player, player_infos[player]['point'], player_infos[player]['winer_name']):
    break
  player = (player + 1) % 2 

実行例

Answer 2

if(your or me >=20):は、if(your >0 or me >=20):と同じことになるので、yourが小さいうちは常にi winですね。

def checker(counter):
    if(your >=20 or me >=20):
        if(your>=20):
            print('you win')           
        else:
            print('i win')

が正しいです。
これだと冗長なので、冗長な部分と無意味な部分(引数と括弧)を削って、

def checker():
    if your>=20:
        print('you win')           
    elif me>=20:
        print('i win')

あと、勝負がついたら、プログラムを終了させた方がよくないですか？

Comments

[massa55]

ありがとうございます。本当に助かります。
この場合ってプログラムの終了をbreakで行うことはできるのですか？

[otn]

whileループの中でbreakすれば可能です。

[massa55]

ありがとうございます。やってみます

Answer 3

グローバル変数に代入するのであれば、global宣言をしてください。

PythonプログラミングFAQ: なぜ変数に値があるのに UnboundLocalError が出るのですか？
https://docs.python.org/ja/3/faq/programming.html#id9

Comments

[massa55]

ありがとうございます。