Skip to content

Conversation

@tomkooij
Copy link

@tomkooij tomkooij commented May 7, 2017

In een facebook groep (Vaksteunpunt natuurkunde) is gelinkt naar: Inleiding programmeren: grafieken.

Moraal van het verhaal: Fantastisch materiaal dat (sporadisch) ook op het VWO wordt ingezet.

Dus vooral: COMPLIMENTEN!

Toch valt mij wel iets op, en ik kan het niet laten om feedback te geven in de vorm van een PR.

In jullie materiaal gebruiken jullie python lijsten op een manier die doet denken aan C/FORTRAN/BASIC/Pascal, dwz een loopje met met een array index (for index in range(len(lijst)) en dat soort dingen:

Dan krijg je zoiets (dit is gekopieerd uit inleiding programmeren lijsten):

metingen_science_park = [12.7, 18.8, 24.9, 14.5, 19.0]
metingen_science_park.append(20.5)
for positie in range(0, len(metingen_science_park)):
    print positie, 
    print "de meting was", metingen_science_park[positie], "graden"

Ik neem aan dat er een didactisch doel achter zit (stap voor stap aanleren wat een lijst is). Maar ik heb een andere mening: Het loopen met array indices is gebaseerd op "onze" manier van programmeren (uit C/FORTRAN/BASIC) en dat moeten we niet meer aanleren.

Didactisch gezien is het altijd beter ongewenste patronen gewoon helemaal niet aan te leren, en direct met het gewenste patroon aan te leren:

Bovenstaande code is m.i. geen python. Het is C/BASIC/Pascal in python syntax. In python schrijf je:

for positie in metingen_science_park:
     print("De meting was %d graden" % positie)

Ik heb geen idee of dit soort feedback welkom is, ik hoor het graag.

Gebruik python lijsten (numpy arrays) in plaats van het C/FORTRAN/BASIC achtige loops over array indices.
@stgm
Copy link
Contributor

stgm commented May 8, 2017

Ja, daar zit wat in Tom. We gaan dit eens bespreken en kijken of we dit zo kunnen aanpassen. Ondertussen laat ik de PR open staan :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants