CODEWITHSUNDEEP
pythonpysimplegui
divingTobi
divingTobi

Reputation: 2300

Proper design pattern for longer PySimpleGUI scripts

I am working on a script that simplifies my daily work routines. Eventually I added a GUI to it using PySimpleGUI. As I add functionality the event loop continues to grow and I am wondering if this is the right way to go. Imagine the following code:

import PySimpleGUI as sg
layout = [
    [sg.Text("Some Text")],
    [sg.Text("Some Input", size=(1, 10))],
    [
        sg.Button("Ok", key="-buttonOk-"),
        sg.Button("Do Something", key="-buttonDoSth-"),
        sg.Button("Cancel"),
    ],
]

# Create the Window
window = sg.Window("Window Title", layout)
# Event Loop to process "events" and get the "values" of the inputs
while True:
    event, values = window.read()
    if (event == sg.WIN_CLOSED or event == "Cancel"):  
        break
    if event == "-buttonOk-":
        print("ok pressed")
    if event == "-buttonDoSth-":
        print("I am doing something")
window.close()

As I add more buttons, input fields, etc. the if event == '...': part grows, so that it is about 300 lines long by now. Is there a better way to do this? How would this be done in a "real" software package?

Upvotes: 2

Views: 287

Answers (1)

Jason Yang
Jason Yang

Reputation: 13061

You can use a dictionary to map event to function called.

For example

import PySimpleGUI as sg

def do_something(*args, **kwargs):
    print("I am doing something")

def button_ok(*args, **kwargs):
    print("ok pressed")

function = {
    "-buttonOk-": button_ok,
    "-buttonDoSth-": do_something,
}

layout = [
    [sg.Text("Some Text")],
    [sg.Text("Some Input", size=(10, 1))],
    [
        sg.Button("Ok", key="-buttonOk-"),
        sg.Button("Do Something", key="-buttonDoSth-"),
        sg.Button("Cancel"),
    ],
]
window = sg.Window("Window Title", layout)

while True:
    event, values = window.read()
    if (event == sg.WIN_CLOSED or event == "Cancel"):
        break
    elif event in function:
        function[event]()

window.close()

Upvotes: 3

Related Questions