文字の更新が上手くいかない。

Question

クトゥルフ神話TRPGのキャラメイクツールをtkinterで作っているのですがサイコロを振るプログラムがどうもうまくいきません。

実現したいこと

・ROLLを押したときに該当箇所のみが変わるようにしたい。
・ALL ROLLを押したときにすべて変わるようにしたい

前提

ボタンを押してもtkinter上の文字が変わらず、最後の8番目のボタンを押したときと、ALL ROLLを押したときだけしか変わりません。しかも数字が同じです。リスト上では変わっています。

該当のソースコード

import tkinter as tk
from tkinter import ttk
import random

root=tk.Tk()
root.geometry('400x600')
root.title('クトゥルフ神話ツール')
flame=tk.Frame(root)
flame.grid(row=0,column=0)

def roll(i):
    def x():
        status_amount=0
        if i==2 or i==5:
            status_amount=random.randint(8,18)
            amount_status[i]=status_amount
        else:
            status_amount=random.randint(3,18)
            amount_status[i]=status_amount
        number_set()
        point_label=tk.Label(flame,text=amount_status[5]*10+amount_status[7]*20,
                             font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')
        point_label.grid(row=8,column=2)
    
    return x

def number_set():
    global amount_status
    for i in range(8):
        var.set(amount_status[i])
        amount_label=tk.Label(flame,textvariable=var,font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')
        amount_label.grid(row=i,column=2)
    print(amount_status)

def roll_all():
    def x():
        #global var
        for i in range(8):
            status_amount=0
            if i==2 or i==5:
                status_amount=random.randint(8,18)
                amount_status[i]=status_amount
            else:
                status_amount=random.randint(3,18)
                amount_status[i]=status_amount
        number_set()
        point_label=tk.Label(flame,text=amount_status[5]*10+amount_status[7]*20,
                        font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')
        point_label.grid(row=8,column=2)
    
    return x
    
  


status=['STR','CON','SIZ','DEX','APP','INT','POW','EDU']
amount_status=[0,0,0,0,0,0,0,0]
var=tk.IntVar()
for i in range(8):
    var.set(amount_status[i])
    label=tk.Label(flame,text=status[i],font=('HG丸ｺﾞｼｯｸM-PRO',30))
    button=tk.Button(flame,text='ROLL',font=('HG丸ｺﾞｼｯｸM-PRO',15),command=roll(i))
    label.grid(row=i,column=0)
    button.grid(row=i,column=1)
    amount_label=tk.Label(flame,textvariable=var,font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')
    amount_label.grid(row=i,column=2)
ginou_label=tk.Label(flame,text='技能ポイント',font=('HG丸ｺﾞｼｯｸM-PRO',20))
ginou_label.grid(row=8,column=1)
all_push=tk.Button(flame,text='ROLL ALL',font=('HG丸ｺﾞｼｯｸM-PRO',15),command=roll_all())
all_push.grid(row=8,column=0)

root.mainloop()

試したこと

IntVerを使ってみましたがうまくいきません。

Answer 1

よい回答ついているのでおまけです。

動作確認していて作ってしまったのがこちらです。
考えかたは回答と同じです。

• それぞれのラベルごとに作ったIntVarをamount_statusに格納する
• 乱数を設定するときに、対応するIntVarを変更する
• 値を使うときはget()メソッドを使う

import tkinter as tk
from tkinter import ttk
import random

root=tk.Tk()
root.geometry('400x600')
root.title('クトゥルフ神話ツール')
flame=tk.Frame(root)
flame.grid(row=0,column=0)

def roll(i):
    def x():
        global amount_status
        status_amount=0
        if i==2 or i==5:
            status_amount=random.randint(8,18)
            amount_status[i].set(status_amount)
        else:
            status_amount=random.randint(3,18)
            amount_status[i].set(status_amount)
        point_label=tk.Label(flame,text=amount_status[5].get()*10+amount_status[7].get()*20,
                             font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')

        point_label.grid(row=8,column=2)

    return x

def roll_all():
    def x():
        global amount_status
        for i in range(8):
            status_amount=0
            if i==2 or i==5:
                status_amount=random.randint(8,18)
                amount_status[i].set(status_amount)
            else:
                status_amount=random.randint(3,18)
                amount_status[i].set(status_amount)
        point_label=tk.Label(flame,text=amount_status[5].get()*10+amount_status[7].get()*20,
                        font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')
        point_label.grid(row=8,column=2)

    return x


status=['STR','CON','SIZ','DEX','APP','INT','POW','EDU']

amount_status = []
for i in range(8):
    amount_status.append(tk.IntVar())

for i in range(8):
    label=tk.Label(flame,text=status[i],font=('HG丸ｺﾞｼｯｸM-PRO',30))
    button=tk.Button(flame,text='ROLL',font=('HG丸ｺﾞｼｯｸM-PRO',15),command=roll(i))
    label.grid(row=i,column=0)
    button.grid(row=i,column=1)
    amount_label=tk.Label(flame,textvariable=amount_status[i],font=('HG丸ｺﾞｼｯｸM-PRO',30),foreground='#ff0000')
    amount_label.grid(row=i,column=2)

ginou_label=tk.Label(flame,text='技能ポイント',font=('HG丸ｺﾞｼｯｸM-PRO',20))
ginou_label.grid(row=8,column=1)
all_push=tk.Button(flame,text='ROLL ALL',font=('HG丸ｺﾞｼｯｸM-PRO',15),command=roll_all())
all_push.grid(row=8,column=0)

root.mainloop()

Answer 2

58行目

var=tk.IntVar()

でインスタンス化されたvarを各ウィジットに紐づけしているため、すべて同じ数字が表示されてしまいます。amount_statusにあるそれぞれの内容を表示したい場合は、1つ1つに対応するtk.IntVarのインスタンスを作成して各ウィジットに紐づけてあげる必要があります。

断片的な形での提示で申し訳ないのですが、以下のようにしてあげるとよいかと思います。

var_list = list(tk.intVar() for i in range(8))
...
var_list[i].set(amount_status[i])
...
tk.Label(..., textvariable=var_list[i], ...)

Comments

[PythonNINJYA]

ナルホド・・・でもなぜ、質問にあるプログラムではリスト(amount_status)の最後の要素が入ってくるのでしょうか？

[PythonNINJYA]

あ、varの中にです。

[yukkeorg]

回答が遅くなりました。
29行目や59行のforループを抜けるときには、amount_statusの一番最後(８番目)の値がvar.set()で設定されるからです。