指スマ(いっせーので)

Question

前提

前回、長すぎてアドバイスをいただき短くしました！

ただ、このコードだとCPUの番の際ランダムでコンピューターが出す本数よりも合計のほうが小さいということが発生します。その解消の方法を考えてたんですがいい方法が思いつきません；；　答えでなくてもアドバイスでも全然いいので教えてください。

実現したいこと

・CPUの番の際ランダムでコンピューターが出す本数よりも合計のほうが必ず大きくする）
・なんでもいいのでアドバイス（もっと短くする方法があれば）

該当のソースコード

#関数のインポート
import random
import time

#指の本数の定義
people_fin = 2
com_fin = 2

#関数作成
def people_yubisuma():
    global people_fin
    global com_fin
    print("あなたの番！")

    print("何本出すか入力してね。")
    people_stand = int(input("あげる本数"))

    print(" ")

    print("合計全員で何本あげるか予想してね。")
    people_all = int(input("みんなとの合計"))

    print("ゆびすま、、")
    time.sleep(1)
    
    com_stand = random.randint(0, com_fin)

    if com_stand + people_stand == people_all:

        print("CPUは", com_stand, "本あげたよ。")
        print("あたり。一本のけてね。")

        people_fin -= 1

    if com_stand + people_stand != people_all:

        print("CPUは", com_stand, "本あげたよ。")
        print("残念。")


def com_yubisuma():
    global people_fin
    global com_fin
    print("CPUの番！")


    print("何本出すか入力してね。")
    people_stand_up = int(input("あげる本数"))


    print("ゆびすま、、")
    time.sleep(1)
    
    com_stand_up = random.randint(0, com_fin)
    com_all = random.randint(0, com_fin + people_fin)

    if com_stand_up + people_stand_up == com_all:

        print("CPUは", com_stand_up, "本あげたよ。")
        print("CPUは", com_all, "本といったよ。")
        print("あたので一本のけるよ。")

        com_fin -= 1

    if com_stand_up + people_stand_up != com_all:

        print("CPUは", com_stand_up, "本あげたよ。")
        print("CPUは", com_all, "本といったよ。")
        print("残念。")



#関数動かす
while True:
    print(" ")
    print(" ")

    people_yubisuma()


    if people_fin != 0 and com_fin != 0:
        print(" ")
        print(" ")
        com_yubisuma()

        if people_fin == 0:
            print(" ")
            print(" ")
            print("あなたの勝ち！")
３            break
        if com_fin == 0:
            print(" ")
            print(" ")
            print("CPUの勝ち！")
            break


    if people_fin == 0:
        print(" ")
        print(" ")
        print("あなたの勝ち！")
        break
    if com_fin == 0:
        print(" ")
        print(" ")
        print("CPUの勝ち！")
        break

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

googlecolaboratory使用。
anacondaでも大丈夫です(多分)。

Comments

[bsdfan]

if people_fin == 0: と if com_fin == 0: のブロックが、
if people_fin != 0 and com_fin != 0: の中と外で2回書かれていますが、中のほうは要りますか？

[Konya]

回答が遅くなってすみません。
いりませんねｗ　ifをそれぞれで考えてましたw
順次処理ではいりませんね！　意見感謝です！

Answer 1

    com_stand_up = 1#random.randint(0, com_fin)
    com_all = 1#random.randint(0, com_fin + people_fin)

上記の「random.randint(0, com_fin + people_fin)」を0始まりでなく、自分の出す指の数=com_stand_up にすればいいのでは？

    com_stand_up = 1#random.randint(0, com_fin)
    com_all = 1#random.randint(com_stand_up, com_fin + people_fin)

ちなみに終了値ももう一息工夫したほうがよさそうですけどね。

Comments

[Konya]

ほんとですねw　難しく考えすぎたかもしれませんｗ
TakaiYさん、いつもありがとうございます。ほんとに助かります；；

終了値のもう一工夫とは具体的にどう工夫すればいいのでしょうか？
個人的には問題なさそうに思えるのですがどういうことなんでしょうか？
よかったら教えてください

[TakaiY]

終了値は、com_fin + people_fin になってますよね。
これ、2本ずつだとして、最大4本ということになりますが、com が 1本も出していなかった場合、最大は2にしたほうがいいですよね。
ということです。

[Konya]

あと、もう一つ質問できたのでよかったら教えてください。

追加でinputで自分の指の本数以上もしくは、全員の合計以上のときにやり直させらるようにしたいです。この場合、さらif関数を細かくしないといけないのでしょうか？

[Konya]

逆のことですか！？！
気づかなかったw　ほんとですね。
理解しました。ありがとうございます！
if関数で何とかなりそうな気もするのでちょっと頑張ってみます。無理だったら質問にまた入れると思います。

[TakaiY]

今の構造のままでやるのであれば、
print("あなたの番！")
print("何本出すか入力してね。")
people_stand = int(input("あげる本数"))
ここの部分をwhileでかこって、 矛盾の無い入力値になるまで入力させることになるでしょう。

ただ、そうすると、com側にも同じような処理を入れることになりますね。 さらにたとえば中断機能を入れたいときには... というようにだんだんごちゃごちゃしたプログラムになってくると思います。

ここで一度、機能を追加していく前にプログラムの構造を見直すのをお勧めします。
- ユーザの入力を関数化する。 (ここに値のチェックを入れれば1箇所で済む)
- 他にも共通の処理があれば関数化する。
- 勝敗の判定はちょっとわかりにくいので見直したほうがいいと思います。

[TakaiY]

「if関数で何とかなりそうな気もする」
if関数不要です。
com が予測として言える範囲の 最低は 自分が出した本数ですが、 最高はどのような本数でしょうか？ という質問です。 自分の指の本数、 相手の指の本数、 自分の出す本数 が使えますよね。
やってみればわかります。

[Konya]

わかりました！ありがとうございます！