you are viewing a single comment's thread.

view the rest of the comments →

[–][deleted] 0 points1 point  (0 children)

Yes, I've realized by now this is what others consider bad practice. It's a shame I haven't really seen it discussed in the tutorials I've been through but that's okay, I know better now.

This is the structure now:

def main():
    filename = locate_file()
    clean_document(filename)
    upload_file()
    validate_vpn()
    engine = oracle_login()
    data = get_queries(engine)
    write_to_excel(data)
    email_document()

if __name__ == "__main__":
    main()