Python: nested loops not behaving how I want them to

Question

I need to write a program that calculates the users pay. The part I'm stuck on is where I have to ask for the amount of hours the user worked for each day of the week (which is repeated depending on how many weeks they specified). This is what I have so far:

daysofweek = ["Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday", "Sunday"]

day = 0
while day < weeks: #user specified variable above 0
  for i in range(len(daysofweek)):
    while True:
      daysofweek[i] = input("Enter the number of hours for week 1 " + daysofweek[i] + ": ")
      try:
        hours = float(daysofweek[i])
        if hours < 0 or hours > 24:
          print("Invalid: Enter a number between 0 and 24")
          continue
        break
      except ValueError:
        print("Invalid: Enter a number between 0 and 24")
day = day + 1

Sorry for the wall of code (some of it is probably irrelevant to my issue). So basically this iterates through the list well the first week but then the day names are replaced with the numbers that are entered. Plus I'm not sure how to store these variables that are entered so I can calculate the pay afterwards. Sorry if this is asking too much, just completely stuck. Thanks!

"but then the day names are replaced with the numbers that are entered" - Yes, that happens when you reassign the elements. Why, what did you want it to do with the entered values? — TigerhawkT3
– TigerhawkT3, Commented Apr 7, 2016 at 9:09
yeah i know why it was happening but have no idea how to change it. I need to add them up and multiply them by another input. — DanThistle
– DanThistle, Commented Apr 7, 2016 at 9:13
yeah, that's what you told to Python to do. daysofweek[i] = input('x') replaces values in daysofweek. What did you intended to do? — Łukasz Rogalski
– Łukasz Rogalski, Commented Apr 7, 2016 at 9:24

aluriak · Accepted Answer · 2016-04-07 09:17:49Z

2

You could use a dict, associate the day and the number of hour:

daysofweek = ["Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday", "Sunday"]
hours = {}
for day in daysofweek:
    hour = -1
    while hour < 0 or hour > 24:
        hour = input("Enter the number of hours for week 1 " + day + ": ")
    hours[day] = hour

Then, you can manipulate the day and the associated amount of work:

from collections import defaultdict
payperday = defaultdict(lambda: 10, {'Sunday': 20})
print(sum(payperday[day] * hours[day] for day in daysofweek))

Adding the weeks can be done by adding an integer in dictionnary key, and a loop:

for week in range(nb_week):
    for day in daysofweek:
        hour = -1
        while hour < 0 or hour > 24:
            hour = input("Enter the number of hours for week " +str(week) + " " + day + ": ")
        hours[week, day] = hour

Then you can ask for the number of hour worked for a particular week & day:

hours[2, 'Sunday']

Note that some more idiomatic code is enough to get simpler code: usage of for day in daysofweek instead of the C-style for i in range(len(daysofweek)), or usage of dictionnary for association of data, for instance.

edited Apr 7, 2016 at 9:17

answered Apr 7, 2016 at 9:10

aluriak

5,9572 gold badges30 silver badges42 bronze badges

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

2 Comments

DanThistle Over a year ago

I have this now: daysofweek = ["Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday", "Sunday"] hours = {} while day < week: for day in daysofweek: hour = -1 while hour < 0 or hour > 24: hour = input("Enter the number of hours for week 1 " + day + ": ") hours[day] = hour day = day + 1 But it goes through the days of the week infinitely regardless of the week value

aluriak Over a year ago

You have to increment week. Also, day < week seems to be a non-sense.

Collectives™ on Stack Overflow

Python: nested loops not behaving how I want them to

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related