Python Nested Loop Problems With Validation

Question

I'm creating a work log where a user can input a task or lookup a task by dates.

When looking up task by date, the user is presented with a list of dates. The user can then select from the list by entering a number. The list of tasks should then display for that specific date.

I'm just having a problem if the user enters a number not in the list of dates. You can see else statement at the end of method which is commented out---which had been causing a few issues. Everything works fine otherwise. So question--how do i provide error message if input from user is not in index list without printing out the message every time it loops through the list?

def search_by_date(self):
    for i, d in enumerate(self.tasklist.dates()):
        enum_list = [(i+1,d) for i,d in enumerate(self.tasklist.dates())]
        print(i+1, ':', d)
    while True:
        datereq = input("Select Number To See Tasks For A Date: ").strip()
        try:
            datereq = int(datereq)

        except ValueError:
            print("Invalid Entry")
            continue

        else:
            for i, d in enum_list:
                for task in self.tasklist.task_list:
                    if datereq == i:
                        if task.date == d:
                            print("Date :", task.date,
                                  " Task:", task.task,
                                  " Minutes:", task.minutes,
                                  " Notes: ", task.notes
                                  )
                            continue

                    #else:
                        #print("Invalid Entry. Please try again")
                        #continue

continue should probably be break. The continue statement has no effect here, it doesn't skip anything. — Jean-François Fabre
– Jean-François Fabre ♦, Commented Mar 25, 2017 at 20:18

Julien · Accepted Answer · 2017-03-25 20:25:55Z

2

Is this what you're looking for?

def search_by_date(self):
    for i, d in enumerate(self.tasklist.dates()):
        enum_list = [(i+1,d) for i,d in enumerate(self.tasklist.dates())]
        print(i+1, ':', d)
    while True:
        datereq = input("Select Number To See Tasks For A Date: ").strip()
        try:
            datereq = int(datereq)

        except ValueError:
            print("Invalid Entry")
            continue

        else:
            found = False
            for i, d in enum_list:
                for task in self.tasklist.task_list:
                    if datereq == i and task.date == d:
                        found = True
                        print("Date :", task.date,
                              " Task:", task.task,
                              " Minutes:", task.minutes,
                              " Notes: ", task.notes
                              )
            if not found:
                print("Invalid Entry. Please try again")

edited Mar 25, 2017 at 20:25

answered Mar 25, 2017 at 20:18

Julien

5,8295 gold badges36 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

Jean-François Fabre Over a year ago

that's strange: it will print "invalid entry" whatever the result is.

Julien Over a year ago

Yes, without his data objects I can't actually run the code :) One sec as I fix

John Rogerson Over a year ago

Hi Julien--yes when i try this it prints all the right results, but prints the Invalid Entry line at the end. For an actual invalid entry it is working correctly. Hmm..

Jean-François Fabre Over a year ago

I thought of an else statement matching the for loop but there are 2 for loops. you'll have to go with a flag telling that it's found.

Jean-François Fabre Over a year ago

nitpick: don't use == True or != True. Just use if found: and if not found

|

Collectives™ on Stack Overflow

Python Nested Loop Problems With Validation

1 Answer 1

9 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

9 Comments

Your Answer

Sign up or log in

Post as a guest

Related