Tag: transaction

Good first effort!

On currentBudget:

One of the things that is easy to tell you about is Python’s context managers docs.

The context manager lets you write code in a different context and takes care of preparing that context and cleaning it up. In your case, there’s a context to deal with opened files 🙂

So, instead of

def currentBudget():
    fileTransaction = open("transactions.csv", "r")
    info = fileTransaction.readlines()
    fileTransaction.close()
    # ...

it is recommended that you do

def currentBudget():
    with open("transactions.csv", "r") as f:
        info = fileTransaction.readlines()
    # ...

The context manager makes sure the file is properly closed even if the code raises an error.
Generally, inside the with statement you write the least amount of code possible, so that the context can get cleaned up ASAP.

Also, if you are dealing with csv files you might want to take a look at the csv library Python provides: csv docs.

---

On organisation:

I really liked that you grouped related prints in functions to call repeatedly! One thing you might consider is factoring the different actions you take, depending on the userInput variable, into separate functions. That way, the body of your main function is cleaner and easier to read, and then you have a separate function for each functionality.

On user input:

Clever usage of .lower() to handle capitalised input 😉

You are doing relatively basic I/O, with the user just typing a single letter to pick an action. One thing you might want to do is

userInput = input("nEnter your Choice: ").trim().lower()

This removes leading/trailing whitespace (with .trim).
Additionally, you may want to also add (0) at the end of the line if it is acceptable for the user to write a whole word but you only need to use the first letter to distinguish. This doesn’t make much sense while the actions are a), b), etc, but would make sense if you renamed the options to reflect what they actually do.

Good luck 🙂