質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.57%

pythonでの正しいインデントの仕方、外側ループの抜け方について

受付中

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 895

babbleman

score 39

以下のコードを見てください。
五目並べプログラムを作っており、どこかに五目並べてあるかを判定するメソッドを書いているのですが(関係ない部分は省略しています)
まず最初に、盤面上に合法手がない場合には、game_endをTRUEとして、それでこのメソッドを終わらせたいです。ただ、self.end=Trueのところと同じインデントレベルでbreakしても以下の処理がどうも実行されているようでした。
二つ目に、多重ネストで書いているのですが、8方向をくまなく調べるところで、もしどこかで
if counter>=RENJU:がTRUEとなったときには、このループを一気に抜け出してメソッドそのものを切り上げたいです。ただ、今回は一般化として、どこにbreak文を記述すれば任意の場所のループを抜けられるかを教えてほしいです。
最後に、もし上二つに両方引っかからなかったら、game_end=falseをreturnしたいというものですが、
やはりこれも上二つをどのようにうまくbreakするかになると思います。そのやり方がわかりません。お願いします。

それと、ちょっとした質問ですが、もし後からこれをif文の中身として扱いたいとなったときに、それより下にある行の全てのインデントを修正する必要が出てきます。自分はやり方がわからず一行一行主導でやっているのですが、一気に合わせるやり方は存在するでしょうか?

それと、このプログラムは、盤面の外から出ない範囲で、という意味で
if 0<=y1<=SIZE-1 and 0<=x1<=SIZE-1:
を記述していますが、もう少しうまい方法はあるでしょうか?
こちらの方は、気が向いたらで構いません、教えていただけたら嬉しいです。
質問が沢山ありますが、よろしくお願いいたします。

class Board():

    def end_check(self,turn):
        if len(self.search_availablepos())==0:
            self.end=True

        #隣接8方向をくまなくチェック

        y,x=numpy.where(self.board==turn)
        for i,j in zip(y,x):
            for p in direction:
                k,m=p[0],p[1]
                counter=0
                for pp in range(-(RENJU-1),RENJU):
                    y1=i+pp*k
                    x1=j+pp*m
                    if 0<=y1<=SIZE-1 and 0<=x1<=SIZE-1:
                        if self.board[y1][x1]==turn:
                            counter+=0
                        else:
                            counter=0
                    else:
                        continue
                if counter>=RENJU:
                    print(turn,"WINS!")
                    break
        return False
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

+5

まず最初に、盤面上に合法手がない場合には、game_endをTRUEとして、それでこのメソッドを終わらせたいです。

もしどこかでif counter>=RENJU:がTRUEとなったときには、このループを一気に抜け出してメソッドそのものを切り上げたいです。

最後に、もし上二つに両方引っかからなかったら、game_end=falseをreturnしたいというものですが、


次行でreturn (仕様に沿ったbool値)で関数を終了したらよいかと。
returnにて任意の位置で関数を抜けることができます。

一気に合わせるやり方は存在するでしょうか?

まともなテキストエディタを使いましょう。
たとえばWindows上のサクラエディタであれば行範囲選択したあとにShift+TaborTabキー入力でインデントレベルを一括で上げ下げできます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.57%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る