[Python] definition vraag

[flugelisoke]

Hallo coders!


De volgende code:

code:

def IsYearLeap(year):
    if year % 4 != 0:
         return False
    elif year % 100 != 0:
         return True
    elif year % 400 != 0:
         return False
    else:
         return True

def DaysInMonth(year,month):        
    normalmonthslist = [31,28,31,30,31,30,31,31,30,31,30,31]
    leapmonthslist =   [31,29,31,30,31,30,31,31,30,31,30,31]
  
    if IsYearLeap(year) == True:
        return leapmonthslist[:month]
    else:
        return normalmonthslist[:month]

#print(DaysInMonth(2000,5))


def DayOfYear(year,month,day):
    maandlijst = DaysInMonth(year,month)



    if month <1 or month > 12:
        return None
    if day < 1 or day > maandlijst[-1]:
        return None
    del maandlijst[-1]
  
    total = 0
    for i in maandlijst:
        total+=i
    return total + day
      

  
print(DayOfYear(2000,12,31))
print(DayOfYear(2015,1,25))
print(DayOfYear(2015,12,31))

Regel 24
Bij maandlijst = DaysInMonth(year,month)

Zit er een verschil tussen maandlijst en DaysInMonth(year,month) ?

Want als ik bij de vergelijking:

code:

if month <1 or month > 12:
        return None
    if day < 1 or day > maandlijst[-1]:
        return None

in plaats van maandlijst, DaysInMonth(year,month) neerzet, komt er een type error:

TypeError: '>' not supported between instances of 'int' and 'list'

Hoe komt het dat maandlijst wel werkt in een vergelijking ( or < > ) en DaysInMonth(year,month) niet?

[Sendy]

Ben je niet gewoon de [-1] vergeten achter DaysInMonth(y, d)?

[CCJJVV]

Zoals Sendy zegt ben je inderdaad de [-1] achter de function call vergeten.

Je hebt wat dat betreft geluk dat je gebruik maakt van python3. Voer je deze zelfde code uit met behulp van python2 dan was hij er vrolijk voorbij gegaan en had je een mooie bug geïntroduceerd in je programma.

[flugelisoke]

@Sendy @CCJJVV

Als ik lijstmetmaanden[-1] doe werkt de code wel, wanneer ik:

- DaysInMonth(year,month-1)
- DaysInMonth(year,month)
- DaysInMonth(-1)
- DaysInMonth(month)
- DaysInMonth(month-1)

doe, komen ze allemaal uit op een error.

[Woy]

@flugelisoke DaysInMonth(year,month) geeft een lijst als resultaat. Daar moet je dus nog een element uit pakken net zoals je met lijst lijstmetmaanden doet door de indexing operator [] te gebruiken.

Python:

someDay = DaysInMonth(year,month)[-1]

is dus vergelijkbaar met

code:

lijstmetmaanden = DaysInMonth(year,month)
someDay = lijstmetmaanden[-1]

[flugelisoke]

Bedankt, elke keer leer ik weer een stapje meer