Python3でランダムリスト生成インプット版を作成してみました、評価や添削をお願いします。

Question

前提・実現したいこと

作成したソースコードの構文ミスの指摘や、もっと汎用性のある書き方に添削していただきたいです。

該当のソースコード

from random import randint

def get_user_input():
    while True:
        n = -1
        x = input("要素数が0以上50以下のリストを作れます(要素は0~100の間でランダムに生成します)\n要素を何個にしますか:")
        if x.isdigit():
            n = int(x)
            if 0 <= n <= 50:
                return n
        print("0以上50以下の整数を入力してください！\n\n")


def random_list():
    user_input = get_user_input()
    x = []
    for i in range(user_input):
        x.append(randint(0, 100))
    return x

def list(a):
    return a

def random_list_max(a):
    if not a:
        return "なし"
    max_value = a[0]
    for item in a:
        if item > max_value:
            max_value = item
    return max_value

def list_len(a):
    l = len(a)
    return l

def random_list_min(a):
    if not a:
        return "なし"
    min_value = a[0]
    for item in a:
        if item < min_value:
            min_value = item
    return min_value

def add(a):
    sum = 0
    for item in a:
        sum += item
    return sum

def play():
    l = random_list()
    print(list(l))
    print("要素は{}個".format(list_len(l)))
    print("合計は{}".format(add(l)))
    print("最大値は{}".format(random_list_max(l)))
    print("最小値は{}".format(random_list_min(l)))

def game():
    while True:
        play()
        get_user_request = input("もう一度生成しますか?y/n:")
        while get_user_request not in "yn":
            get_user_request = input("\n\nもう一度生成しますか？\nyかnを入力してください:")
            continue
        if get_user_request == "y":
            print("\n" * 10)
        else:
            break
    print("またのご利用をお待ちしております！")

if __name__ == "__main__":
    game()

試したこと

手段を関数別に分けて、調節しやすいようにしました。

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

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

Comments

[退会済みユーザー]

何らかの基準(例えば仕様やコーディング規約)がなければ、レビューはできません。
何を求めた質問でしょうか？

[hoshi-takanori]

「コードレビュー」タグをつけたらいいと思います。

[HIROKIdesuga]

te2jiさん、hoshi-takanoriさん指摘ありがとうございます。
この場合のコーディング規約というのは、一般的にpep8のことで大丈夫でしょうか？
それ以外で特にこれを基準とする、というようなコーディング規約は今回の場合特にないです。

Answer 1

ざっくりいきます。

def get_user_input():
    while True:
        n = -1
        x = input("要素数が0以上50以下のリストを作れます(要素は0~100の間でランダムに生成します)\n要素を何個にしますか:")
        if x.isdigit():
            n = int(x)
            if 0 <= n <= 50:
                return n
        print("0以上50以下の整数を入力してください！\n\n")

このままだと汎用性がないので、入力を求めるメッセージや、値の範囲を引数で渡せるようにしたらいいと思います。

def random_list():
    user_input = get_user_input()
    x = []
    for i in range(user_input):
        x.append(randint(0, 100))
    return x

これも、乱数の個数や範囲を引数にして、get_user_input の呼び出しは play に移動した方が汎用性が上がります。

def list(a):
    return a

この関数は何のためのものですか？

def random_list_max(a):
    if not a:
        return "なし"
    max_value = a[0]
    for item in a:
        if item > max_value:
            max_value = item
    return max_value

random_list_max の対象は必ずしも random list である必要はないので、次の list_len のように、単に list_max という名前がいいと思います。

def list_len(a):
    l = len(a)
    return l

def random_list_min(a):
    if not a:
        return "なし"
    min_value = a[0]
    for item in a:
        if item < min_value:
            min_value = item
    return min_value

関数の並び順ですが、random_list_max と random_list_min は似たような関数なので、近くに置いた方がいいと思います。（間に list_len が挟まると違和感があります。）
また、random_list_max と random_list_min を統一できないか考えてみましょう。（方法はいろいろあります。）

def add(a):
    sum = 0
    for item in a:
        sum += item
    return sum

関数名は list_sum がいいかな。
list_len, random_list_max, random_list_min と並べた時に統一感のある名前にしましょう。

def play():
    l = random_list()
    print(list(l))
    print("要素は{}個".format(list_len(l)))
    print("合計は{}".format(add(l)))
    print("最大値は{}".format(random_list_max(l)))
    print("最小値は{}".format(random_list_min(l)))

def game():
    while True:
        play()
        get_user_request = input("もう一度生成しますか?y/n:")
        while get_user_request not in "yn":
            get_user_request = input("\n\nもう一度生成しますか？\nyかnを入力してください:")
            continue
        if get_user_request == "y":
            print("\n" * 10)
        else:
            break
    print("またのご利用をお待ちしております！")

get_user_request は関数名に見えます。変数名は user_input とか user_request の方がいいし、そこまで重要な変数じゃないのでもっと単純な名前でもいい気がします。

Comments

[HIROKIdesuga]

hoshi-takanoriさん、添削していただきありがとうございます。
変数名の名前の付け方と引数の使い方、必要性について理解が増えました。
それと質問なんですが、get_user_inputの呼び出しがrandom_list()ではなくplay()の時の方が汎用性が増すのはなぜでしょうか？

[hoshi-takanori]

random_list の配列の要素数を get_user_input を呼び出すのではなく引数で受け取ることで、要素数が決まっている乱数の配列を作るためにも使えるようになって、汎用的になるからです。

[HIROKIdesuga]

コメントありがとうございます。
理解しました、random_list()をget_user_input()以外の関数と組み合わせる考えをしていませんでした。
ご教授ありがとうございます。