Python3で初めて作った数当てゲームの評価、添削をお願いします。

Question

前提・実現したいこと

Python3で初めて作った数当てゲームの評価、添削をお願いします。

発生している問題

気をつけた方がいい箇所が自力では見つけきれないので、出来るだけ頑張って書きました、添削をできればお願いします。

該当のソースコード

import random
# import os

VAL_MIN = 1
VAL_MAX = 100


def ask_number():
    while True:
        n = 0
        s = input("回答:")
        if s.isdigit():
            n = int(s)
        if VAL_MIN <= n <= VAL_MAX:
            return n

        print("{}以上{}以下の整数を入力してください\n".format(VAL_MIN, VAL_MAX))


def ask_yes_no():
    print("もう一度遊びますか？y/n")
    while True:
        ans = input()
        if ans in ('y', 'n'):
            return ans

        print("※y(yes)かn(no)を入力してください！\n")


def play():
    print("{}~{}のランダムな整数を生成しました、正しい数字を当ててください。\n".format(VAL_MIN, VAL_MAX))
    num = random.randint(VAL_MIN, VAL_MAX)
    try_count = 0

    while True:
        n = ask_number()
        try_count += 1

        if n == num:
            print("おめでとうございます、正解です!")
            print("チャレンジ回数{}、次もがんばってください。".format(try_count))
            return

        if n > num:
            print("答えより大きいです、もう一度当ててみましょう！\n")
        else:
            print("答えより小さいです、もう一度当ててみましょう！\n")


def clear_screen():
    print("\033[2J")     # for ANSI terminal
    # # os.system('cls') # for windows


def game():
    while True:
        play()
        if ask_yes_no() == 'n':
            print("遊んでいただいてありがとうございました、またのご利用お待ちしております！")
            return
        clear_screen()


game()

試したこと

while文の使い方の工夫(continueやbreak)、全体のループ

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

バージョン：Python3.7.7
エディタ：PyCharm CE

Comments

[hoshi-takanori]

無効な数（100 より大きかったりマイナスだったり）を入れても「0以上100以下の数字を入力してください」が出ませんね。

[HIROKIdesuga]

hoshi-takanoriさん、修正の依頼ありがとうございます！更新します！

[hoshi-takanori]

無限ループにならなかった。ざんねん…。添削します。

[HIROKIdesuga]

危ういところでした、なんとか回避です！

Answer 1

• このコードは flake8 というツールでの書式チェックは警告が 0 です。

• 機能毎に関数分割してます。

• 画面クリアは 改行 100 回出力とは異なる方法で書いてみました。
  (windows と linux 系では異なる処理になってしまってますが)

• その他 いろいろ 小さな変更をしています。
  (最小値が 0 なのか 1 なのか のバグ? も修正しています。

wc.py

import random
# import os

VAL_MIN = 1
VAL_MAX = 100


def ask_number():
    s = input("回答:")
    while True:
        if s.isdigit():
            n = int(s)
        if VAL_MIN <= n <= VAL_MAX:
            return n

    print("{}以上{}以下の整数を入力してください\n".format(VAL_MIN, VAL_MAX))


def ask_yes_no():
    print("もう一度遊びますか？y/n")
    while True:
        ans = input()
        if ans in ('y', 'n'):
            return ans

        print("※y(yes)かn(no)を入力してください！\n")


def play():
    print("{}~{}のランダムな整数を生成しました、正しい数字を当ててください。\n".format(VAL_MIN, VAL_MAX))
    num = random.randint(VAL_MIN, VAL_MAX)
    try_count = 0

    while True:
        n = ask_number()
        try_count += 1

        if n == num:
            print("おめでとうございます、正解です!")
            print("チャレンジ回数{}、次もがんばってください。".format(try_count))
            return

        if n > num:
            print("答えより大きいです、もう一度当ててみましょう！\n")
        else:
            print("答えより小さいです、もう一度当ててみましょう！\n")


def clear_screen():
    print("\033[2J")     # for ANSI terminal
    # # os.system('cls') # for windows


def game():
    while True:
        play()
        if ask_yes_no() == 'n':
            print("遊んでいただいてありがとうございました、またのご利用お待ちしております！")
            return
        clear_screen()


game()

追記
ask_number はバグがありました。
(数字以外や 範囲外の値を入力したときの挙動)
次のようにしたほうが良さそうです。

def ask_number():
    s = input("回答:")
    while True:
        if s.isdigit():
            n = int(s)
            if VAL_MIN <= n <= VAL_MAX:
                return n
        print("{}以上{}以下の整数を入力してください\n".format(VAL_MIN, VAL_MAX))
        s = input()

Comments

[HIROKIdesuga]

katoyさんコメントありがとうございます！
flake8を実際入れてみて使ってみました、とても便利です！
wc.pyをプレイしてみましたが、入力部分で何を入れても先に進まなかったので、自分で直してみました。
ソースコードを更新しますので、よろしかったら添削お願いいたします。

[HIROKIdesuga]

正確には文字列や範囲外の数字を入れた場合先に進まなかったので、
関数ask_numberの部分を自分なりに直してみました、よろしかったら添削お願いします。

[katoy]

ask_number はバグってましたね。
回答に少し訂正した ask_number をかきました。(テストはしていません)

本来は メソッド毎に テストコードを書くべきです。
python での テストコードの書き方も研究してみてください。
(Ctrl-C や Ctrl-D, ctrl-Z などを入れたときにどんな挙動をさせたいかも考慮するとよいです)

[HIROKIdesuga]

テストコードですね、重要と聞くのでしっかり書いていきます。
katoyさん、添削評価ありがとうございました！

Answer 2

ネストとcontinueの多用が気になったので、関数単位で分けてみました。
1-100の整数を受け取るまでのif文と数の判定のif文が並んでいた部分も、可読性のために入力受け取りを分けました。

from random import randint


def guess_number() -> None:
    num = randint(1, 100)
    count = 0
    print("1~100のランダムな整数を生成しました、正しい数字を当ててください。\n")
    while True:
        user_input = get_user_input()
        count += 1
        if user_input == num:
            print("おめでとうございます、正解です!")
            break
        elif user_input > num:
            print("答えより大きいです、もう一度当ててみましょう!\n")
        else:
            print("答えより小さいです、もう一度当ててみましょう!\n")

    print(f"チャレンジ回数{count}、次もがんばってください。\n")


def get_user_input() -> int:
    raw_input = input("回答: ")
    if raw_input.isdigit() and 1 <= (int_input := int(raw_input)) <= 100:
        return int_input
    print("1以上100以下の整数を入力してください\n")
    return get_user_input()


def main():
    while True:
        guess_number()

        while (b := input("もう一度遊びますか?y/n ")) not in "yn":
            continue
        if b == "y":
            print("-" * 10)
        else:
            break

    print("遊んでいただいてありがとうございました、またのご利用お待ちしております!")


if __name__ == "__main__":
    main()

Comments

[HIROKIdesuga]

solzardさん、添削ありがとうございます！
この場合の、機能を関数別に分ける方法をよく理解できました！
その中でひとつ疑問があるのですが、関数get_user_input()でreturn int_inputのint_inputが
Local variable 'int_input' might be referenced before assignment
というエラーを表示させています。
これは変数bと何が違って、どういう対処をしたらいいですか？
一応関数get_user_input()の最初にint_input=""と付け加えたら消えたのですが、それで大丈夫でしょうか？

[退会済みユーザー]

int_input := int(raw_input)としてif文中でセイウチ演算子を使って定義し、その評価が真のときのみreturn int_inputと使われるので実は問題は起きません。
もし気になるようでしたら、
if raw_input.isdigit() and 1 <= (int_input := int(raw_input)) <= 100:
return int_input
の部分を、従来のように
if raw_input.isdigit() and 1 <= int(raw_input) <= 100:
return int(raw_input)
と書き換えてくださっても構いません。

:= のセイウチ演算子は昨秋のPython3.8で追加された新機能なので、PyCharmではまだ対応できていないようです。
今回のように、直前に宣言しているなどで問題ないと自信があるときは、私は"might be referenced before assignment"のエラーは無視しています。
Intellijのバグリポートに報告はされているので、将来的には文脈に応じて判断されるようになるでしょう。

セイウチ演算子についてあまりご存じないようでしたら、下記の記事が分かりやすいかと思います。
https://atsuoishimoto.hatenablog.com/entry/2019/09/03/110508

[HIROKIdesuga]

solzardさん、詳細な解説ありがとうございます！
セイウチ演算子についても、初めて聞きました。
まだまだ自分の勉強不足な部分が多いので、インプットの量を増やしてかつアウトプットもいいバランスで行えたらなと考えています。
solzardさん、大変勉強になりました、ありがとうございます。

Answer 3

python はあまり詳しくないので、一般的なプログラミングの観点から添削します。

import random

flag = "y"
while flag == "y":

flag はループ中で代入してないので、ずっと "y" のままですね。そういう場合は単純に while True: としましょう。
参考: Pythonのwhile文によるループ処理（無限ループなど） | note.nkmk.me

    num = random.randint(1,100)
    count = 0
    a = 0
    print("1~100のランダムな数字を生成しました、正しい数字を当ててください。\n")
    while a != num:
        a = input("回答:")
        a = int(a)
        while (a > 100) | (a < 0):
            print("0以上100以下の数字を入力してください\n")
            a = input("回答:")
            a = int(a)
            continue

この部分、元は

        while a > 100 | a < 0:
            print("0以上100以下の数字を入力してください\n")
            continue

となっていて、| がビット和のために絶対に成り立たない条件になってましたが、or を使っていれば無効な数（マイナスまたは 101 以上）の場合には無限に「0以上100以下の数字を入力してください」が出力されるはずでした。ここは while ではなく if を使って単純に

        if a > 100 or a < 0:
            print("0以上100以下の数字を入力してください\n")
            continue

とすれば良かったと思います。なお、ビット和については後で書きます。

        if a > num:
            print("答えより大きいです、もう一度当ててみましょう！\n")
        elif a == num:
            print("おめでとうございます、正解です")
            break
        else:
            print("答えより小さいです、もう一度当ててみましょう！\n")

「大きい」「正解」「小さい」という判定の順序が気になります。先に正解を判定して、後から「大きい」「小さい」を判定する方が分かりやすいと思います。

        count += 1
    print("チャレンジ回数{}、次はもっとがんばりましょう".format(count))

もしも 1 回で正解すると、「チャレンジ回数0」と表示されますが、それでいいのかな？

    print("もう一度遊びますか？y/n")
    b = 0

未入力を表す b の初期値ですが、0 よりも None または "" がいいと思います。

    while (b != "y") & (b != "n"):

論理積（かつ）や論理和（または）は and や or を使いましょう。
参考: Pythonの論理演算子and, or, not（論理積、論理和、否定） | note.nkmk.me

| や & はビット和やビット積と言って、論理値（True や Flase）ではなく 2 進数の計算になります。また、演算子の優先順位も違うので括弧が必要になったりしますね。
参考: Python ビット演算 超入門 - Qiita

        b = input()
        print("y(yes)かn(no)を入力してください\n")
        continue
    if b == "n":
        break

print("遊んでいただいてありがとうございました、またのご利用お待ちしております！")

全体的には読みやすくて綺麗なコードだと思います。がんばってください。

Comments

[HIROKIdesuga]

たくさんの添削ありがとうございます！
while文やif文の使い分けなど、いろんな点において気をつけることを知ることができました。
添削していただいた問題点に加え、入力値の判別もより厳密に修正しました。
よろしかったら最終チェックしていただけるとありがたいです。