実行結果が想像していたものと異なる

Question

以下の名前データがあります

gname=["ジョージ","ジョン","ポール","リンゴ"]
sname=["ハリスン","レノン","マッカートニー","スター"]
beatles=["ジョン・レノン","ポール・マッカートニー","ジョージ・ハリスン","リンゴ・スター"]

gname,snameの要素を4個ずつランダムに抽出し、それらを組み合わせて名前を生成します。生成した名前がbeatlesに存在すれば正解とし、4人の名前が同時に全て正解になるまで繰り返すプログラムです。このプログラムですが、実行結果において名前が4つ表示されるはずが3つしか表示されなかったり、正誤の判定がおかしかったり（名前はあっているのに正解数にカウントされていなかったり、その逆もあり）します。プログラムを見直してみましたがわかりません。原因は何でしょうか？

※このプログラムは少し長いため、大まかな内容を理解するだけでもかなりの時間を要するかと思います。時間のある方は以下のプログラムを見て回答していただければ幸いです。

該当のソースコード

import random
gname=["ジョージ","ジョン","ポール","リンゴ"]
sname=["ハリスン","レノン","マッカートニー","スター"]
beatles=["ジョン・レノン","ポール・マッカートニー","ジョージ・ハリスン","リンゴ・スター"]
g=random.sample(gname,4)            #gnameから重複なく4つ選ぶ
s=random.sample(sname,4)            #snameから重複なく4つ選ぶ
count=0                  #正解数をカウントする変数
b=[0,1,2,3]

for x in range(0,3):
    b[x]=str(g[x])+"・"+str(s[x])   #ランダムに選んだgとsを結合
   
while count < 4:            #4人正解するまで繰り返す
    for x in range(0,3):
        for y in range(0,3):
            if b[x]==beatles[y]: #生成した名前とbeatlesが同じがチエック
                count=count+1    #同じなら正解数を1増やす
        print(b[x])
    if count==0:
        print("不正解です\n")    #一度も正解していなければ不正解と表示
    else:
         print(str(count)+"つ正解です\n")  #正解していれば正解数を表示
    g=random.sample(gname,4)
    s=random.sample(sname,4)         #新たに要素をランダムに抽出し、結合する
    for i in range(0,3):
        b[i]=str(g[i])+"・"+str(s[i])

実行結果 (あくまで一例。出力される名前の組み合わせは毎回変わる)

ポール・レノン
リンゴ・マッカートニー
ジョン・スター
不正解です

ポール・マッカートニー
ジョージ・ハリスン
ジョン・レノン
3つ正解です

ジョン・ハリスン
ポール・スター
リンゴ・レノン
3つ正解です

ジョン・ハリスン
ジョージ・レノン
ポール・マッカートニー
4つ正解です

#　私が望む実行結果 (あくまで一例。出力される名前の組み合わせは毎回変わる)
上の実行結果との違い
・名前の組み合わせは3つではなく4つ表示されている
・正誤の判定がちゃんとしている

ジョージ・レノン
ジョン・ハリスン
ポール・スター
リンゴ・マッカートニー
不正解です

ジョージ・ハリスン
ジョン・マッカートニー
ポール・スター
リンゴ・レノン
1つ正解です

ジョージ・ハリスン
ジョン・レノン
ポール・スター
リンゴ・マッカートニー
2つ正解です

ジョージ・ハリスン
ジョン・レノン
ポール・マッカートニー
リンゴ・スター
4つ正解です

Answer 1

とりあえず怪しそうなのは、所々に見られる range(0, 3) です。
これは 0以上3未満 の範囲を表します。

4人なら、range(0, 4) あるいは単に range(4) を使った方が良いでしょう。

---

ご提示のコードには、正解のカウント数を適切にリセットしていないというバグがまだあります。
これでは通算4回正解すると終了してしまいます。

実行結果も実際そのようになっていますね。

ポール・レノン
リンゴ・マッカートニー
ジョン・スター
不正解です

ポール・マッカートニー   ← 正解1
ジョージ・ハリスン      ← 正解2
ジョン・レノン          ← 正解3
3つ正解です

ジョン・ハリスン
ポール・スター
リンゴ・レノン
3つ正解です

ジョン・ハリスン
ジョージ・レノン
ポール・マッカートニー    ← 正解4
4つ正解です

正解の判定をする際は集合演算を用いた方が楽かと思います。
答えの集合との共通部分を取れば、その要素数が正解数と同じになります。

Comments

[退会済みユーザー]

返信遅くなってすみません。ご回答ありがとうございます。解決しました！