2 个问题:Python Pickle .dat 附加到列表和 tkinter checkbutton 问题

2 issues: Python Pickle .dat append to list and tkinter checkbutton issue

我在这个应用程序上遇到了很多问题,因为我还不够好,但我几乎完成了它,只想完成它,这样我就可以继续进行一些稍微较低级别的项目。

这是一个 tkinter 待办事项应用程序。

您可以将任务添加到列表框

对于每个 Task,都有一些关联的属性,其中包括:````self.value = vandself.connectivity = c. The hierarchy of the tasks displayed in the listbox is determined by the value of val_var```(例如,值越高,在列表中显示的位置就越高)。

任务和相关属性由用户在创建另一个任务时的输入决定。

Task 附加到列表 task_list 并且在用户向列表中添加了超过 1 个任务后,下一次添加任务时,用户将可以选择检查以某种方式连接的现有任务。

列表已排序,因此具有最高值 (val_var) 的任务显示在列表框的顶部,具有最低值的任务显示在列表框的底部。 您可以“保存任务”,稍后启动应用程序,然后您可以“加载任务”。

问题 1:

从已保存的 .dat 文件加载 任务后,它会按照保存时的顺序显示在列表框中。 但是,如果您现在想添加另一个任务,至少会发生两件不希望发生的事情:

  1. 现在加载到列表框中的任务现在不会在添加新任务时显示为复选按钮。
  2. 当您添加另一个任务时(同样是加载 .dat 文件后)列表框将删除刚刚加载的内容,列表框将只显示新添加的任务。

我对能够从 .dat 文件加载 Tasks 实例然后将它们附加到 task_list 因此它们是当前 session/instance 的一部分感兴趣应用程序的一部分,但我不知道该怎么做。

问题 2:

在已将任务添加到列表框中的给定会话中,可以使用“删除任务”按钮将它们从列表框中删除。 列表框中的选定任务已删除,但与从task_list中删除的任务不同。

要测试我的意思,可以向列表框中添加几个任务,然后在添加后删除一个。请注意,在尝试创建另一项新任务时,刚刚从列表框中删除的任务仍将显示为复选按钮 - 然而,另一项 刚刚删除的任务现在已消失为检查按钮。

对于这些问题的任何帮助将不胜感激。

代码如下:

from tkinter import Tk, Frame, Button, Entry, Label, OptionMenu, Toplevel, StringVar, Checkbutton, DoubleVar
import tkinter.messagebox 
import pickle

root = Tk()

task_list = []

class Task:
    def __init__(self, n, i, h, v, c): 
        self.name = n
        self.impact = i
        self.hours = h
        self.value = v
        self.connectivity = c

def open_add_task():
    taskwin = Toplevel(root)

    #NAME
    titlelabel = Label(taskwin, text='Title task concisely:').grid(column=1, row=0)
    name_entry = Entry(taskwin, width=40, justify='center')
    name_entry.grid(column=1, row=1)

    #IMPACT
    impactlabel = Label(taskwin, text='Impact').grid(column=1, row=2)
    imp_var = StringVar(value=0)
    OptionMenu(taskwin, imp_var, *range(0, 10+1)).grid(column=1, row=3, sticky='ns')

    #HOURS(required)
    hourlabel = Label(taskwin, text='Whole hours \n required').grid(column=1, row=16)
    hour_entry = Entry(taskwin, width=4, justify='center')
    hour_entry.grid(column=1, row=17)

    #CONNECTIVITY
    C_lab = Label(taskwin,text="Connectivity to other tasks").grid(column=1, row=18)
    placement=19
    vars = [] # list to hold the DoubleVar used by Checkbutton
    for task in task_list:
        # add a DoubleVar to the list
        vars.append(DoubleVar())
        # use the task.value as the "onvalue" option
        Checkbutton(taskwin, text=task.name, variable=vars[-1], onvalue=task.value, offvalue=0).grid(column=1, row=placement, sticky="w")
        placement+=1

    def add_task():
        if name_entry.get() != '': # If textbox inputfield is NOT empty do this

            #CONNECTIVITY
            connectivity = sum(v.get() for v in vars)/10 +1 #if no connectivity the rest below is multiplied by 1
            #VALUE
            val_var = ((((int(imp_var.get())/5) + 1) * (connectivity)+(float(hour_entry.get())/10))) #-(float(hour_entry.get())/20) #-hours fra højere rangerende opgaver skal minusses urgency_var # c = 1+task1(int(imp_var.get())+(int(man_var))+task2(repeat)+task3(repeat)


            task_list.append(Task(name_entry.get(), imp_var.get(), hour_entry.get(), val_var, connectivity))
            reload()
            taskwin.destroy()
        else:
            tkinter.messagebox.showwarning(title='Whoops', message='You must enter a task')

    Add_button = Button(taskwin, text='Add', command=add_task).grid(column=1, row=placement, sticky="ew")
    placement+=1

def reload():
    task_list.sort(key=lambda a: a.value, reverse=True)
    listbox_tasks.delete(0, tkinter.END)

    for x in task_list:
        listbox_tasks.insert(tkinter.END, x.name)

def delete_task():
    try:
        task_index = listbox_tasks.curselection()[0]
        listbox_tasks.delete(task_index)
        tasks = listbox_tasks.get(0, listbox_tasks.size())
        pickle.dump(tasks, open('Todo.dat', 'wb'))
        del task_list[0]
    except:
        tkinter.messagebox.showwarning(title='Error', message='You must select a task to delete')

def save_tasks():
    tasks = listbox_tasks.get(0, listbox_tasks.size())
    pickle.dump(tasks, open('Todo.dat', 'wb'))

def load_tasks():
    try:
        tasks = pickle.load(open('Todo.dat', 'rb'))
        listbox_tasks.delete(0, tkinter.END)
        for task in tasks:
            listbox_tasks.insert(tkinter.END, task)
    except:
        tkinter.messagebox.showwarning(title='Error', message='You have no tasks')

# Create UI
your_tasks_label = Label(root, text='Your tasks:', font=('roboto',11, 'bold'), justify='center')
your_tasks_label.pack(pady=5)


scrollbar_tasks = tkinter.Scrollbar(root)
scrollbar_tasks.pack(side=tkinter.RIGHT, fill=tkinter.Y)

listbox_tasks = tkinter.Listbox(root, height=10, width=45, font=('', 11, 'bold'), fg=('grey'), justify='center') # tkinter.Listbox(where it should go, height=x, width=xx)
listbox_tasks.pack(padx=5, pady=5)

listbox_tasks.config(yscrollcommand=scrollbar_tasks.set)
scrollbar_tasks.config(command=listbox_tasks.yview)

#BUTTONS
New_Task_Button = Button(root, text='New Task', width=42, command=open_add_task)
New_Task_Button.pack()

button_delete_task = Button(root, text='Delete task', width=42, command=delete_task)
button_delete_task.pack()


button_save_tasks = Button(root, text='Save tasks', width=42, command=save_tasks)
button_save_tasks.pack()

button_load_tasks = Button(root, text='Load tasks', width=42, command=load_tasks)
button_load_tasks.pack(pady=5)

root.mainloop()

你的问题很简单。您需要保存 Task class 的对象,而不是保存列表框内的字符串。

那是说你永远不应该像你那样给出裸露的 except 子句,总是指定你想要捕获的异常。如果你不这样做,你会发现很难找到确切的问题。

例如在您的代码块中:

try:
     tasks = pickle.load(open('Todo.dat', 'rb'))
     listbox_tasks.delete(0, tkinter.END)
     for task in tasks:
         listbox_tasks.insert(tkinter.END, task)
except:
     tkinter.messagebox.showwarning(title='Error', message='You have no tasks')

这里的异常发生在找不到文件的时候。但是现在如果有一个空文件,如果一切顺利并且没有引发异常,即使文件中没有任务也不会显示该消息。更合适的做法是检查文件中是否有任何内容,然后显示消息。

我也经常看到你改写东西。例如这里:

def delete_task():
    ...
    tasks = listbox_tasks.get(0, listbox_tasks.size())
    pickle.dump(tasks, open('Todo.dat', 'wb'))
    ...

你真的需要在save函数下写同样的东西吗?

这是您更正后的代码:

...
task_list = []

class Task:
    def __init__(self, n, i, h, v, c): 
        self.name = n
        self.impact = i
        self.hours = h
        self.value = v
        self.connectivity = c

...

def reload():
    task_list.sort(key=lambda a: a.value, reverse=True)
    listbox_tasks.delete(0, tkinter.END)

    for x in task_list:
        listbox_tasks.insert(tkinter.END, x.name)


def delete_task():
    try:
        task_index = listbox_tasks.curselection()[0]
        listbox_tasks.delete(task_index)
        task_list.pop(task_index)
        save_tasks()
        
    except IndexError:
        tkinter.messagebox.showwarning(title='Error', message='You must select a task to delete')


def save_tasks():

    with open('Todo.dat', 'wb') as pk:
        pickle.dump(task_list, pk)


def load_tasks():
    global task_list
    try:
        with open('Todo.dat', 'rb') as pk:
            task_list = list(pickle.load(pk))

        reload()
            
    except Exception as e:  # FileNotFound Error
        print(e)
        tkinter.messagebox.showwarning(title='Error', message='You have no tasks')

# Create UI
your_tasks_label = Label(root, text='Your tasks:', font=('roboto',11, 'bold'), justify='center')
your_tasks_label.pack(pady=5)

...