Repository navigation
Ch10 Variable scope has confusing variable name #196
Description
How could the content be improved?
Ch0 Variable scope section has this code:
initial_fine = 0.25
late_fine = 0.50
def calc_fine(days_overdue):
if days_overdue <= 10:
days_overdue = days_overdue * initial_fine
else:
days_overdue = (days_overdue * initial_fine) + (days_overdue * late_fine)
return days_overdue
There are two different uses of days_overdue, one of which would be measured in days, and one of which would be measured in dollars (pounds for us!). It would be better as something like:
def calc_fine(days_overdue):
if days_overdue <= 10:
total_fine = days_overdue * initial_fine
else:
total_fine = (days_overdue * initial_fine) + (days_overdue * late_fine)
return total_fine
Which part of the content does your suggestion apply to?
Ch10 Variable scope
Activity