search

minprog / adventure

3 stars 5 forks source link

Veranderen van databestanden bij adventure kan leiden tot crashes/falende checks #4

Open Jelleas opened 4 years ago

Jelleas commented 4 years ago

Zo vertrouwde een uitwerking erop dat er een lege regel stond aan het einde van synonyms.dat, maar dat stond natuurlijk niet zo bij check50.

Misschien een idee om even te checken of de data bestanden van de student, anders zijn dan de databestanden bij de checks?

Jelleas commented 4 years ago

Yup echt even checken! Debuggen is onmogelijk als de databestanden niet overeenkomen

bjorn-out commented 4 years ago

Afgezien van minprog/cs50x#42 zijn er geen afwijkingen (hiervoor zijn nu PR's aangemaakt). De databestanden in check50 hebben ook lege regels aan het eind.

Jelleas commented 4 years ago

Studenten passen de bestanden weleens aan :)!

Misschien slim om, als de bestanden bestaan, ze mee te nemen in de check. Dan vervolgens in de eerste check even na te gaan of deze niet anders zijn dan de bestanden bij de checks zelf.

bjorn-out commented 4 years ago

Oh, dat bedoelde je daarmee. Ja, dat kan inderdaad. Dan moeten ze wel ook mee ingeleverd worden via de website (anders zijn die bestanden niet beschikbaar)

Jelleas commented 4 years ago

Mijn idee zou zijn om ze enkel mee te nemen als ze bestaan, ofwel in .cs50.yaml

check50:
    files:
        - !require "adventure.py"
        - !require "room.py"
        - !include "SmallAdv.dat"
        - !include "TinyAdv.dat"

Als ze bestaan, kunnen we even kijken of ze afwijken en vervolgens op de afwijking wijzen.