ハングマンというゲームのプログラムを動かしたい

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
独学プログラマーのP138~139に載っているハングマンというゲームのプログラムを動かす練習をしています。
目的としては正解だった場合ゲーム終了、不正解が積み重なってハングマンの絵が完成した時、あなたの負け！と表示した上で正解の単語を表示したいです。

発生している問題・エラーメッセージ

Runしようとすると動かなかったり、動いたとしても出るはずの答えの"cat"が出なかったりと思うような動きをしません
(前者は私の機械の不調の可能性も拭えません)
エラーメッセージ
無し

該当のソースコード

コード
def hangman(word):
    wrong = 0
    stages = ["",
              "__________         ",
              "|                  ",
              "|         |        ",
              "|         0        ",
              "|        /|＼      ",
              "|        / ＼      ",
              "|                  "
             ]
    rletters = list(word)
    board = ["_"] * len(word)
    win = False
    print("ハングマンへようこそ!")
    
    while wrong < len(stages) - 1:
        print("\n")
        msg = "一文字を予想してね"
        char = input(msg)
        if char in rletters:
            cind = rletters.index(char)
            board[cind] = char
            rletters = "$"
        else:
            wrong = wrong + 1
        print(" ".join(board))
        e = wrong + 1
        print("\n".join(stages[0:e]))
        if "_" not in board:
            print("あなたの勝ち！")
            print(" ".join(board))
            win = True
            break
    if not win:
        print("\n".join(stages[0:wrong+1]))
        print("あなたの負け！正解は ().".format(word))


hangman("cat")

試したこと

41行目のhaangman("cat")を弄ってると反応が変わりまして動いたり動かなかったりしたので意味もなく空行を空けて一つ下にしたり逆に一つ上げたりしましたが本当にこんな事が原因で変わってるとは思えず辞めました。

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

anaconda3(64-bit)
windows10
Jupyter Notebook

Comments

[tiitoi]

コードは markdown で書いてください

[Phenomenon2]

訂正いたします

[y_waiwai]

このままではコードが見れないので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください

[Phenomenon2]

ご丁寧に返答感謝いたします。
浅学ながら修正させていただいたのでご覧頂けると幸いです。

Answer 1

動いたとしても出るはずの答えの"cat"が出なかったりと思うような動きをしません

とりあえずフォーマット書式指定の括弧が間違ってますね。

- print("あなたの負け！正解は ().".format(word))
+ print("あなたの負け！正解は {}.".format(word))

他にも rletters = "$" と代入している点や当てた場合に print(" ".join(board)) と表示していない点がおかしいように思えます。

それを直したら動くようになりました。

def hangman(word):
    wrong = 0
    stages = ['',
              '___________________',
              '|         |        ',
              '|         |        ',
              '|         0        ',
              '|        /|＼      ',
              '|        / ＼      ',
              '|                  ']
    letters = list(word)  # 正解の文字
    board = ['_'] * len(word)  # 入力された文字を記録する配列
    win = False  # 勝ったかどうか

    print('ハングマンへようこそ!')
    while wrong < len(stages) - 1:
        print('\n')
        msg = '一文字を予想してね'
        char = input(msg)

        if char in letters:
            cind = letters.index(char)
            board[cind] = char
            print(' '.join(board))
        else:
            wrong = wrong + 1
            print(' '.join(board))
            e = wrong + 1
            print('\n'.join(stages[0:e]))
        if '_' not in board:
            print('あなたの勝ち！')
            print(' '.join(board))
            win = True
            break
    if not win:
        print('\n'.join(stages[0:wrong + 1]))
        print('あなたの負け！正解は {} です。'.format(word))


hangman('cat')

改善案

コードで冗長な点を修正した改善案も貼っておきます。

1 while, for が break で途中で抜けたかどうかを確認するには、

while .. else, for .. else が使える。

while文の終了時の処理(while...else) - 繰り返し - Python入門

これで win フラグを排除できる。

2 str はリストと同じように扱える

str の文字列はリストと同じように扱えるので letters = list(word) としなくてもよい。

3. char が文字列に存在するかどうか

str.find(char) で存在すれば最初に見つかったインデックス、そうでない場合 -1 がかえるので、これで判定できる。

def hangman(word):
    stages = ['___________________',
              '|         |        ',
              '|         |        ',
              '|         0        ',
              '|        /|＼      ',
              '|        / ＼      ',
              '|                  ']
    wrong = 0  # 間違えた回数
    board = ['_'] * len(word)  # 入力された文字を記録する配列

    print('ハングマンへようこそ!')
    while wrong < len(stages):
        print('========================================')
        char = input('一文字を予想してね: ')

        char_pos = word.find(char)
        if char_pos >= 0:
            board[char_pos] = char
        else:
            wrong += 1
            print('\n'.join(stages[:wrong]))  # 間違えた回数行だけ表示する。

        print(' '.join(board))  # 正解できている文字を表示する。

        if '_' not in board:
            print('あなたの勝ち！')
            break
    else:
        print('あなたの負け！正解は {} です。'.format(word))


hangman('cat')

Comments

[Phenomenon2]

無事動きました！ありがとうございます！