From 6715769c9c82d77a4b74d6c46ff00e0f50af9bd9 Mon Sep 17 00:00:00 2001 From: Yong-Jer Chuang Date: Fri, 26 Apr 2024 14:28:11 +0800 Subject: [PATCH] Refactor config_launcher.py to simplify code and improve readability --- config_launcher.py | 30 ++++++++++-------------------- 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/config_launcher.py b/config_launcher.py index 9644ef4..c933524 100644 --- a/config_launcher.py +++ b/config_launcher.py @@ -183,32 +183,22 @@ def btn_save_act(slience_mode=True): config_dict = get_default_config() language_code = get_language_code_by_name(config_dict["advanced"]["language"]) - is_all_data_correct = True - global combo_language - if is_all_data_correct: - config_dict["advanced"]["language"] = combo_language.get().strip() - # display as new language. - language_code = get_language_code_by_name(config_dict["advanced"]["language"]) - + config_dict["advanced"]["language"] = combo_language.get().strip() + # display as new language. + language_code = get_language_code_by_name(config_dict["advanced"]["language"]) global txt_file_name - filelist = [] - for i in range(15): - filelist.append(txt_file_name[i].get().strip()) - - if is_all_data_correct: - config_dict["list"]=filelist + filelist = [txt_file_name[i].get().strip() for i in range(15)] + config_dict["list"] = filelist # save config. - if is_all_data_correct: - # slience - util.save_json(config_dict, config_filepath) - - if not slience_mode: - messagebox.showinfo(translate[language_code]["save"], translate[language_code]["done"]) + util.save_json(config_dict, config_filepath) + + if not slience_mode: + messagebox.showinfo(translate[language_code]["save"], translate[language_code]["done"]) - return is_all_data_correct + return True def open_url(url): webbrowser.open_new(url)