PySimpleGUIでの、Tableクリック時のKeyErrorを回避する方法を教えてください。

Question

前提・実現したいこと

初の質問です、よろしくお願いいたします。
PySimpleGUIを使用し、毎秒スクレイピング結果を取得し、スクレイピング先に更新があった場合にデータに反映させるプログラムを作成しています。

現在はTableの行をクリックするとKeyErrorが発生してしまいますが、実現したいことはTableの行をクリックすることで、選択した行のデータを取得できるようにしたいと考えています。

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

*** tk version 8.6.9 detected.... patching ttk treeview code ***
*** tk version 8.6.9 detected.... patching ttk treeview code ***
Traceback (most recent call last):
  File "C:\Dropbox\programs\pyguisample.py", line 77, in <module>
    job()
  File "C:\Dropbox\programs\pyguisample.py", line 56, in job
    check_list.append(check_dic[value[0]])
KeyError: 'TABLE'

Tableの行をクリックすると、KeyErrorでプログラムが強制終了してしまいます。
eventの指定が誤っているのだと考えていますが、試したものの上手くいかなかったため、KeyErrorを回避するにはどのように指定すればよいかを教えていただきたいです。

なお、if event in '-timeout-':を削除して毎秒更新を行わないようにした場合には、Tableの行をクリックした場合にもエラーが発生しないようです。
(自動更新は実装したいと考えているので、テーブルクリックと自動更新を両立する方法を教えていただきたいです。)

該当のソースコード

import PySimpleGUI as sg

sg.theme("LightGrey2")
check_dic = {
    '1': 'A',
    '2': 'B'
}

#tableの内容。特に意味は無し。
header = ['項目1',"項目2","項目3","項目4","項目5"]
header2 = ['test','test2']
list1 = [
    ['1a', "1b", '1c', '1d', '1e'],
    ['2a', "2b", '2c', '2d', '2e'],
    ['3a', "3b", '3c', '3d', '3e']]
testable = [
    ['1a', "1b"],
    ['2a', "2b"],
    ['3a', "3b"]]

#左側のtableを含む、メインプログラム
col1 = [    [sg.Text('1行目', enable_events = True)],
            [sg.Checkbox(item[1], key = item[0])for item in check_dic.items()],
            [sg.Button('Reset')],
            [sg.Button('キャンセル', size=(15, 1))],
            [sg.Table(list1, headings=header, key="TABLE",
            background_color = '#dddddd',
            alternating_row_color = '#ffffff',
            display_row_numbers = True,
            def_col_width = 25,
            num_rows=30,
            auto_size_columns=False, vertical_scroll_only=False)]]

#サブプログラム(右側のtableです。)
col2 = [    [sg.Table(testable, headings=header2, key="TABLE2",
            background_color = '#dddddd',
            alternating_row_color = '#ffffff',
            display_row_numbers = False,
            def_col_width = 15,
            num_rows=40,
            auto_size_columns=False, vertical_scroll_only=False)]]


layout = [
    [sg.Column(col1), sg.Column(col2)]
]    


window = sg.Window('サンプルプログラム', layout , resizable = True, finalize=True)

#timeoutを使用し、毎秒jobが実行されるようになっています。
def job():
    check_list = []
    for value in values.items():
        if value[1]:
            check_list.append(check_dic[value[0]])
    
    #チェックの内容に応じて行を追加する処理。
    if 'A' in check_list:
        list1.insert(0, ['4a', '4b', '4c', '4d', '4e'])

    if 'B' in check_list:
        list1.insert(0, ['5a', "5b", '5c', '5d', '5e'])

    window['TABLE'].update(values = list1)

while True:
    event, values = window.read(timeout = 1000, timeout_key = '-timeout-')
    if event == sg.WIN_CLOSED or event == 'キャンセル':
        break
    if event == "Reset":
        list1 = []
        window['TABLE'].update(values = list1)

    #10秒に一度
    if event in '-timeout-':
        job()

window.close()

試したこと

10/20
job()の内容に誤りがあるようで、

    if event in '-timeout-':
        list1.insert(0, ['4a', '4b', '4c', '4d', '4e'])
        window['TABLE'].update(values = list1)

にしたところ、毎秒更新を持続させたままKeyErrorを回避し、Tableをクリックすることができました。
この状態でKeyErrorを発生させている原因を調査し、元の動作を実現できる方法を引き続き調査してみます。

(~投稿日)
・TableのKeyの変更、削除：関係なし
・eventへの「if event == "TABLE"：」追加：関係なし
参考サイト：https://note.com/hotta3216/n/nb7beddc305d4

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

Windows10
Python3.9
他に必要な情報があればご連絡ください。

Answer 1

エラー原因:

check_dic = {
    '1': 'A',
    '2': 'B'
}

# job() 関数内

    for value in values.items():
        if value[1]:
            check_list.append(check_dic[value[0]])

check_dic[value[0]] が KeyError を出しています。
values の内容はタイマーで実行される時は '1', '2' のみですが、
テーブルクリック時には 'TABLE', 'TABLE2' が渡ります。

check_dic 辞書には TABLE 及び TABLE2 が無い為 KeyError

大元の原因は、変数 values をjob()関数から参照している点で、
このvalues は、ユーザ操作により内容が変わります。

while True:
    event, values = window.read(timeout = 1000, timeout_key = '-timeout-')

目的の挙動が解らないので、解決方法は示せませんが
job() 関数内の for value in values.items() の values が意図通りかどうかを確認して見て下さい。

ループ内で書き換わる変数は、グローバル変数として運用しない方が良いです。
頻繁に書き換わる＆影響範囲が広く、不具合の原因になりやすい傾向があります。

Comments

[tongs1341]

ありがとうございます。
メッセージを受けて、ひとまず解決することが出来ました。
ご指摘の通り、job()内でチェックボックスの内容を取得させる処理とTableクリック時にKeyを取得する処理が競合していたようです。

ここで行っていた動作としては、「チェックボックスにチェックが入っている値を取得し、自動実行させる処理を値に応じて変動させる」というものでした。
修正内容としては、今回の問題になっているcheck_listへの入力をボタン処理としてwhile_true内に移動させ、それを実行するためのボタンを一つ作成したことで解決しました。
チェックを入れてからボタンを押す必要が出たので二度手間にはなりますが、結果的にループ内処理を減らすことができ、快適になったと思います。
将来的には、チェックが付いたり外れたりしたときに自動でcheck_listの内容が変動するようにすれば、さらに効率化できそうだと考えられます。

この度はありがとうございました。助かりました。