python ヒット&ブロー たくさん出てくる

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
pythonで数当てゲーム、ヒット&ブローを作っています。
結果を出力したいのですが、なぜか同じ結果が必要以上に出てしまいます。
原因が分からないので、教えて欲しいです。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0
ヒット3
ブロー0

該当のソースコード

#coding:utf-8
import random

a=[random.randint(0,9),
   random.randint(0,9),
   random.randint(0,9),
   random.randint(0,9),]
print(str(a[0])+str(a[1])+str(a[2])+str(a[3]))


isok=False
while isok == False:
    b=input("数を入れてね")
    if len(b) !=4:
        print("４桁の数を入力してください")
    else:
        kazuok=True
        for i in range(4):
            if (b[i]<"0")or(b[i]>"9"):
                print("数ではありません")
                kazuok=False
                break
            if kazuok:
                isok=True
print(b[0])
print(b[1])
print(b[2])
print(b[3])

hit=0
for i in range(4):
    if a[i] == int(b[i]):
            hit=hit+1
blow=0
for j in range(4):
    for i in range(4):
        if(int(b[j])==a[i])and(a[i]!=int(b[i]))and(a[j]!=int(b[j])):
            blow=blow+1
            break
        
        print("ヒット"+str(hit))
        print("ブロー"+str(blow))

        if hit==4:
            print("当たり!")
            break

試したこと

googleで調べた
なぜこうなるのかを考えた

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

ここにより詳細な情報を記載してください。

Comments

[ppaul]

https://teratail.com/questions/358693
で、インデントについて勉強しなさいというアドバイスがありましたが、勉強しましたか？

[Daregada]

ヒット&ブローのルールでは、4桁の数字が全部違ってないといけませんよね。

[ChainsawMeat]

ご心配ありがとうございます
できる限り勉強してきました

[BeatStar]

まずは『問題文を定義せよ』ですね。
普通、Hit&Blowと言えば伝わりますが、いわゆる『ローカルルール』のような感じで独特の定義があるときがあるので。

Answer 1

blow=0 の下にあるループ処理の中に結果を表示する処理が入っているのが原因です。
このため、blowの状態を確認するたびに結果の文字列が表示されてしまっているのです。

結果の表示は、hit数のカウント、blow数のカウントの処理が両方終った後に入れましょう。

Comments

[ChainsawMeat]

両方、breakを使って終わらせようとしてみましたが
終わらず、また同じ様な結果になってしまいます
カウントの処理を両方終わらせるには、どうすればいいでしょうか？

[TakaiY]

breakを使って終らせるのでまありません。

質問にも「インデントについて勉強しなさい」とありますが、インデント=制御の範囲の設定がおかしいのです。

print("ヒット"+str(hit))
print("ブロー"+str(blow))

の処理が上のループの中に入っているのが問題=インデントが適切でない ので、そこを直しましょう。

[ChainsawMeat]

ありがとうございます。
インデントを調節したら、ちゃんと作動しました！

Answer 2

答え自体はほぼ他の方の回答です。そのフォロー的なものとして書いておきます。
質問者さんがつまづいている本質的な部分がわかったので。
(そのため、その方の回答を読んでなんとなくわかったので、この回答をBAに選ぶぐらいならその方の回答をBAにしてください)

とりあえず、ご自分のコードを読んでください。

コードを読むコツは、一行レベルで読み、その行が何をしているかを考えながら読むです。

ただし、Pythonはインデントがものをいう言語です。

Pythonはインデントの数でブロックを表します。
日本語の文章で表現すると『段落』のようなものでしょうか。

たとえば、

for i in range( 0, 3 ):
     print( “Hello” )
print ( “Python!” )

はどうなるでしょうか。

結果はHelloを3回繰り返して、最後に一回だけPython! と表示されるはずです。

Hello の方のインデントとPyhton!の方のインデントの数が違うので、
for文の処理はHelloを表示する一行だけになります。

こういうふうにインデントの数で意味が変わってきます。
それを他の人たちが指摘しているのです。

で、話を戻して、ご自分のコードを読んでみてください。

今回問題となっている部分だけを抽出してみましょう。

# 4回くりかえし、変数jに整数を入れる
for j in range(4):
    # 4回繰り返し、変数iに整数を入れる
    for i in range(4):
        # 条件に一致すれば
        if(int(b[j])==a[i])and(a[i]!=int(b[i]))and(a[j]!=int(b[j])):
              # blowを+1する
             blow=blow+1
             # 内側のループから抜ける
            break
        
        # ヒット数等を表示
        print("ヒット"+str(hit))
        print("ブロー"+str(blow))
        
        # ヒットが4なら
        if hit==4:
            # 「当たり！」と表示
            print("当たり!")
            # 外側のループからも抜ける
            break

これを擬似コードというものとして切り出しましょう。
日本語でも英語でもなんでもいいので、自分が読み書きできる言語で書き下しましょう。

1. 以下を4回繰り返す
    1.1. 以下を4回繰り返す
         1.1.1. 条件に一致すれば
             1.1.1.1. Blowを+1する
             1.1.1.2. 強制的に(1.1)に移動
         1.1.2. ヒット数などを表示
         1.1.3. ヒット回数が4回なら
              1.1.3.1. 「当たり！」と表示
              1.1.3.2. 強制的に(1)に移動

という感じでしょうか。

(1.1.1)で条件に一致した場合は強制的に(1.1)に移動しますが、条件に一致しない場合はそのまま(1.1.2)に行ってしまいます。

そうなると、条件(ブローの条件)を満たさないと、「ヒット○回」とかみたいな表示をされます。

では、どうすればいいでしょうか。

あとは仕様によるので、ここまでとします。(追記されたら続きを書くかもしれませんが)