Skip to content

Clean up and improve readability of expense.py #537

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
aazainjan wants to merge 4 commits into
base: main
Choose a base branch
from
+11 −1
Open

Clean up and improve readability of expense.py #537

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
bca2c7e
Fix SQL DELETE query to use parameterized query
aazainjan Jan 13, 2026
2b7cc1e
Add file-level docstring with project description and dependencies
aazainjan Jan 13, 2026
c8076fa
Display total expenses below the table
aazainjan Jan 13, 2026
339a508
Cleaned and refactored expense.py for readability
aazainjan Jan 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 11 additions & 1 deletion Expense Tracker/expense.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
"""
DebEx - Expense Tracker GUI
Description: Track, filter, delete, and export expenses.
Dependencies: tkinter, tkcalendar, sqlite3.
"""
import datetime
import sqlite3
from tkcalendar import DateEntry
Expand Down Expand Up @@ -36,7 +41,7 @@ def remove_expense():
surety = tb.askyesno('Are you sure?', f'Are you sure that you want to delete the record of {values_selected[2]}')

if surety:
connector.execute('DELETE FROM ExpenseTracker WHERE ID=%d' % values_selected[0])
connector.execute('DELETE FROM ExpenseTracker WHERE ID=?', (values_selected[0],))
connector.commit()

list_all_expenses()
Expand Down Expand Up @@ -289,6 +294,11 @@ def export_to_csv():
table.place(relx=0, y=0, relheight=1, relwidth=1)

list_all_expenses()

# Show total expenses
total_amount = sum([row[4] for row in connector.execute('SELECT * FROM ExpenseTracker')])
Label(root, text=f"Total Expenses: {total_amount}", font=("Georgia", 14)).place(x=950, y=10)

# Finalizing the GUI window
root.update()
root.mainloop()