Bottom Page

Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
 tkinter - global variable not working

I am using Tkinter and doing GUI so that there is a 'save', 'save as', and 'open' button, along with a text box.
I have got everything working except the 'save' button. I Have tried to set the name from filedialog.askopenfilename(), to a global variable to use in my 'save' function.

I have written my save function so that if the file name already exists, save over that, if not do the save_as function. It works fine when I need to save to an existing file, but when I enter straight into the text box and hit 'save' (NOT save as), it does not run the save_as function, but instead tells me that 'filename' is not defined ?

Is it saying that there is no filename because I didn't open anything ?
Is there a different want to do this that I haven't thought about ?

I cant see what I am doing wrong ? Please help!

def open_file():
    global filename
    filename = filedialog.askopenfilename()
    ask_file= open(filename)
    return filename

def save_as_file():
    save_file = filedialog.asksaveasfile( mode = "w", defaultextension = '.txt')
    save_f = text.get(0.0, END)
        showerror(title = "Save Error")

def save_file():
    if os.path.exists(filename):
        save_text = text.get(0.0, END)
        file_save = open(filename, 'w')

You never close the file when you read it, so writing to an already open file will be troublesome.
You'd be better off using a class for this:
import tkinter as tk
import tkinter.filedialog as fd
import tkinter.messagebox as tm
import sys

class  FileManager:
    def __init__(self, parent):
        self.parent = parent
        self.filename = None
        self.file_data = None

    def create_gui(self):
        self.mainframe = tk.Frame(self.parent, bd=6, relief=tk.RIDGE)
        self.mainframe.grid(column=0, columnspan=5, row=0, sticky='nsew')
        self.mainframe.grid_rowconfigure(0, weight=1)
        self.mainframe.grid_columnconfigure(0, weight=1)

        self.text = tk.Text(self.mainframe, bd=2, bg='#CEF6EC', width=113, relief=tk.RAISED)
        self.text.grid(row=0, column=0)

        frame2 = tk.Frame(self.parent, bd=2, padx=2, pady=2, relief=tk.RAISED)
        frame2.grid(row=1, rowspan=2, column=0, columnspan=5, sticky='ew')
        frame2.grid_rowconfigure(0, weight=1)
        frame2.grid_columnconfigure(1, weight=1)

        self.open_button = tk.Button(frame2, text='Open', padx=2, pady=2, bd=2, 
                                     relief=tk.RAISED, command=self.open_file)
        self.open_button.grid(row=0, column=0)

        self.save_button = tk.Button(frame2, text='Save', padx=2, pady=2, bd=2, 
                                     relief=tk.RAISED, command=self.save_file)
        self.save_button.grid(row=0, column=1)

        self.quit_button = tk.Button(frame2, text='Quit', padx=2, pady=2, bd=2,
                                     relief=tk.RAISED, command=self.quit)
        self.quit_button.grid(row=0, column=2)

    def get_filename(self):
        self.filename = fd.askopenfilename()
        print(f'filename: {self.filename}')
    def open_file(self):
        if self.filename:
            with open(self.filename, 'r') as fp:
                self.file_data =
            tm.showerror('read_file', 'File has not been selected')
    def load_text_widget(self):
        if self.file_data:
            self.text.insert(tk.INSERT, self.file_data)
            tm.showerror('load_text_widget', 'No file data')

    def save_file(self, from_text=True):
        if from_text:
            text_data = self.text.get(0.0, tk.END)
            if len(text_data):
                with open(self.filename, 'w') as fp:
                tm.showerror('save_file', 'No text to save')

    def save_as_file():
        # for you to  finish 

    def quit(self):

def main():
    root = tk.Tk()
    fm = FileManager(root)

if __name__ == "__main__":

Top Page

Possibly Related Threads...
Thread Author Replies Views Last Post
  python file(.py) not working with my tkinter project DeanAseraf1 9 1,164 Mar-22-2020, 10:58 PM
Last Post: ifigazsi
  Tkinter scaling windows conten to or with its size not working Detzi 5 452 Jan-12-2020, 12:42 PM
Last Post: Detzi
  [Tkinter] Mouse click event not working on multiple tkinter window evrydaywannabe 2 603 Dec-16-2019, 04:47 AM
Last Post: woooee
  Global Variable in Event Function? What am I doing wrong here? p_hobbs 1 349 Nov-13-2019, 02:50 PM
Last Post: Denni
  [Tkinter] Unable to Access global Variable when switching between frames tziyong 1 443 Nov-03-2019, 01:08 AM
Last Post: balenaucigasa
  [Tkinter] Extrakt a Variable from a closed tkinter window hWp 5 664 Aug-23-2019, 09:01 PM
Last Post: woooee
  [Tkinter] sleep(n) not working inside tkinter mainloop roger31415 2 1,326 Jul-14-2019, 06:57 PM
Last Post: wuf
  Variable not sharing same value between two different functions Python 2.7 Tkinter albert 0 725 Aug-31-2018, 10:45 AM
Last Post: albert
  Tkinter, button doesnt working ;// Dzonzi 3 3,734 Jul-18-2018, 11:14 AM
Last Post: buran
  printing option menu variable in label in Tkinter SmokerX 1 3,229 Jan-18-2018, 07:36 PM
Last Post: SmokerX

Forum Jump:

Users browsing this thread: 1 Guest(s)