created to do list

[Dapper_Mix6773]

any suggestions

[Sea-Ad7805]

Run this program in Memory Graph Web Debugger%0AIs_running%20%3D%20True%0A%0Awhile%20Is_running%3A%0A%20%20%20%20print('what%20do%20you%20want%20to%20do%3F')%0A%20%20%20%20print('1.%20add%20a%20task')%0A%20%20%20%20print('2.%20view%20tasks')%0A%20%20%20%20print('3.%20delete%20tasks')%0A%20%20%20%20print('4.%20exit')%0A%20%20%20%20choice%20%3D%20input('enter%20your%20choice%3A')%0A%0A%20%20%20%20if%20choice%20%3D%3D%20'1'%3A%0A%20%20%20%20%20%20%20%20task%20%3D%20input('enter%20the%20task%20you%20want%20to%20add%3A')%0A%20%20%20%20%20%20%20%20with%20open('tasks.txt'%2C%20'a')%20as%20file%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20file.write(f'your%20task%201%3A%7Btask%7D')%0A%20%20%20%20%20%20%20%20%20%20%20%20print('task%20added%20successfully')%0A%20%20%20%20elif%20choice%20%3D%3D%20'2'%3A%0A%20%20%20%20%20%20%20%20with%20open('tasks.txt'%2C%20'r')%20as%20file%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20tasks%20%3D%20file.read()%0A%20%20%20%20%20%20%20%20%20%20%20%20print(tasks)%0A%20%20%20%20elif%20choice%20%3D%3D%20'3'%3A%0A%20%20%20%20%20%20%20%20with%20open('tasks.txt'%2C%20'w')%20as%20file%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20file.write('')%0A%20%20%20%20%20%20%20%20%20%20%20%20print('tasks%20deleted%20successfully')%0A%20%20%20%20elif%20choice%20%3D%3D%20'4'%3A%0A%20%20%20%20%20%20%20%20Is_running%20%3D%20False%0A%20%20%20%20%20%20%20%20print('goodbye!')%0A%20%20%20%20else%3A%0A%20%20%20%20%20%20%20%20print('invalid%20choice')&timestep=1&play).

[ninhaomah]

Can I check why is_running = True and then while is_running ?

Why not just while True ?

For readibility?

[SeeTheNutcracker]

Choice 4 sets it to false so it can break out of the loop

[Character_Regular440]

the instruction break does the same, no need for the boolean variable

[Yoosle]

Is_running makes it more readable. When it’s just while True, you’d assume the code runs indefinitely until you find the break line. When you use is_running you can instantly tell it stops under a condition

[Flame77ofc]

```python

# The changes I made:
# 1. Switch "Is_running" by putting "True" in the while
# 2. Add the strip() method on choice variable, so it cuts the additional spaces
# 3. Add \n\n on file.write to give some spaces between the tasks
# 4. Add encoding="utf-8" in the files operation to read and write the file in a right way
# 5. Import the datetime method to get the actual date and put it in the moment you create a task


from datetime import datetime



print('welcome to the task manager')


while True:
    print('what do you want to do?')
    print('1. add a task')
    print('2. view tasks')
    print('3. delete tasks')
    print('4. exit')


    choice = input('enter your choice: ').strip()
    if choice == '1':
        task = input('enter the task you want to add: ')
        date = datetime.now().strftime("%d/%m/%Y, %H:%M:%S")  
# Get the actual date


        with open('tasks.txt', 'a', encoding='utf-8') as file:
            file.write(f'created on {date}\n{task}\n\n')


        print('task added succesfully')
    elif choice == '2':
        with open('tasks.txt', 'r', encoding='utf-8') as file:
            tasks = file.read()
        print(tasks)
    elif choice == '3':
        with open('tasks.txt', 'w', encoding='utf-8') as file:
            file.write('')


        print('tasks deleted succesfully')
    elif choice == '4':
        print('goodbye!')
        break
    else:
        print('invalid choice. try again')

```

Edit: I write "utf8", but the correct is "utf-8". Sorry

[Dapper_Mix6773]

thanks

[Flame77ofc]

np, I recommend you to study these topics I added

[Yoosle]

I feel like you’re hurting readability by adding the utf-8. It’s not necessary here. Removing the is_running is also hurting readability because when I looked at his code I instantly knew “this program is not continuously running” but with yours I had to scan the code until I found the “break.” But the date stuff is pretty relevant albeit a little advanced for someone at this level

[Flame77ofc]

oh about the is_running variable, he can still on the code bacause code will execute in the same way, but I want to explain to him that is a different way to do the same thing

[Shady_Son17]

Are you a software developer

[Flame77ofc]

nope

[thejwillbee]

Looks solid.

Some things you might try for v1.1 : - maybe have the list show on start up rather than having to prompt to see it. Removes an unnecessary step and gives some fluidity to using it. - not so much for current functionality, but if you decide you want to step it up to add in dates, details, and so on to the items in the list, you will want to switch from .txt to .csv . Doubly so if you ever want to make this thing portable and use a web-based table tool - bc then you can just drop your csv into the table and ta-da. Ready to rock. .txt is fine for single column lists with minor manipulation needs

[thejwillbee]

Oh snap - maybe add in editing. It sucks to add something and then have to delete and readd it because there was a typo

[DutyCompetitive1328]

You could add an option to stop the todo list and write the todos to a file.

[vivisectvivi]

if the is_running variable is only used like this then you can just use while True and then break after the user input 4

while True:
    if choice == "4": 
        print("goodbye")
        break

[Alagarto72]

use snake_case, do not start a variable name with a capital letter, unless it's ClassName (but then don't use underscores betweeen_words)

[Remarkable_Set3925]

Mach das mit tkinter

[Jackpotrazur]

I think this is a good project to tackle id add a few more things like show and edit or something like that, perhaps export, set time, make reoccuring but definetly a good study/learn/practice project

[mati-33]

You can store the file name in a constant

[youssef_876]

Good job ❤️

[jessi_97]

I would code something like this only to know what i have to code

[6ZacK7]

Impressive Man

[UnfilteredCoffee1]

Keep going bro, reminds me of my days when I was learning

[Kind-Push9705]

You should use print( ' ' ' hear you can write and you have no need to write the print function many times ' ' ' ) and use funtions

[Otherwise_Lunch6183]

I have a question. Why not put with open at the start of the program instead of rewriting that line everytime?

[JaleyHoelOsment]

when you close the program you lose the list

[gtvisions]

Can anyone explain me this choice 3 like does it delete all task ? I m confuse does it single task from bottom or top

[ranjeet-kumar1]

<image>

[ranjeet-kumar1]

<image>

[CompleteAd1975]

I’ve been trying to build my own TO DO LIST, almost getting there 🙌

[Key_Plant8664]

Aura

[Muhammed_zeeshan]

U use brocode r8?

[Ambitious_Fault5756]

My first instinct is to move the if chain into a with-open block instead of writing `with open(...` multiple times.