tkinterでまるばつゲームを作ったが謎のバグが起きました

Question

pythonのtkinterでまるばつゲームを作ったのですがなぜか右下の枠にしかマークされません。
参考にしたサイトの自分が入力したところのみコピペしてみたところ正常にできていました。
自分のコードにはホーム画面とまるばつゲーム用の画面を切り替えれるようにし
参考にしたサイトからは勝敗の判定が抜けています。
それと見比べてみても原因がわかりませんでした。

#!/usr/bin/env python
# -*- coding: utf8 -*-
import tkinter as tk
import tkinter.ttk as ttk


def gomarubatu():
    frame_marubatu.tkraise()

def gohome():
    frame_home.tkraise()


def init_data():
    global init, data
    marubatu_data = [[init for i in range(3)] for j in range(3)]

turn = 1
def marubatu_bt_mark(marubatu_pt_bt):
    global marubatu_data, turn
    if marubatu_pt_bt.widget == bt_marubatu[j][i]:
        if turn == 1:
            marubatu_pt_bt.widget["text"] = "〇"
            marubatu_data[j][i]=1

        if turn == -1:
            marubatu_pt_bt.widget["text"] = "×"
            marubatu_data[j][i]=-1
    turn *= -1




if __name__ == "__main__":
    #ホームフレーム
    #メインウィンドウの設定
    root = tk.Tk()
    root.title("HOME")
    root.geometry("250x400")
    #メインウィンドウのグリッドを1x1にする
    root.grid_rowconfigure(0, weight=1)
    root.grid_columnconfigure(0, weight=1)
    #フレームの作成と設置
    frame_home = ttk.Frame(root)
    frame_home.grid(row=0, column=0, sticky="nsew", pady=20)

    #メインウィンドウのウィジェットの設定と配置
    lb_home = ttk.Label(frame_home, text="HOME")
    bt_gomarubatu = ttk.Button(frame_home, text="まるばつゲーム", command=gomarubatu)

    lb_home.pack()
    bt_gomarubatu.pack()



    #まるばつフレーム
    #まるばつフレームの作成と設置
    frame_marubatu = ttk.Frame(root)
    frame_marubatu.grid(row=0, column=0, sticky="nsew", pady=20)

    #まるばつウィンドウの基本ウィジェットの設置と設定
    lb_marubatu = ttk.Label(frame_marubatu, text="まるばつゲーム")
    bt_backhome = ttk.Button(frame_marubatu, text="ホームに戻る", command=gohome)
    lb_marubatu.pack()
    bt_backhome.pack()

    #まるばつゲーム用ボタンを配置するフレームの準備
    frame_marubatu_game = ttk.Frame(frame_marubatu)
    frame_marubatu_game.pack()

    #まるばつゲーム用ボタンの設定
    w=5
    h=5
    bt_marubatu = [[tk.Button(frame_marubatu_game, width=w, height=h,)for i in range(3)]for j in range(3)]

    for i in range(3):
        for j in range(3):
            bt_marubatu[j][i].grid(column=j, row=i)





    #メインフレームを全面にする
    frame_home.tkraise()

    init = 0
    marubatu_data = [[init for i in range(3)] for j in range(3)]

    root.bind("<1>", marubatu_bt_mark)

    root.mainloop()

参考にしたサイト

Comments

[8524ba23]

この質問で提示されたコードと参考サイトのコードは全く同じですか？

[kinopi]

いいえ、参考にしただけでホーム画面を追加したり名前を変えたりしていて、判定はまだ書いていません。

[8524ba23]

「判定」とはなんのことでしょうか？
参考サイトのコードのうち、どの部分をどういった目的で修正したのかを具体的に記載すると回答得られやすくなるかと思います。

[__horito]

以下の質問のようにエラー文と実行環境の情報が欲しいです。
https://teratail.com/questions/290593

[1T2R3M4]

>見比べてみても原因がわかりません
もう少し良く見比べられたらいかがでしょうか。

Answer 1

ボタンが押されたときに動作する関数で、i と j を使っていますが、この値が適切に設定されていないからでしょう。

def marubatu_bt_mark(marubatu_pt_bt):
    global marubatu_data, turn
    if marubatu_pt_bt.widget == bt_marubatu[j][i]:
        if turn == 1:
            marubatu_pt_bt.widget["text"] = "〇"
            marubatu_data[j][i]=1

        if turn == -1:
            marubatu_pt_bt.widget["text"] = "×"
            marubatu_data[j][i]=-1
    turn *= -1

「 if marubatu_pt_bt.widget == bt_marubatu[j][i]:」の判定にどんな意図があるのかわからないので改善のアドバイスはできません。
また、たぶん、すでにマルバツが入っていたとしても上書きされちゃいますね。