Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feedback från studenter HT23 #61

Open
AndreasArne opened this issue Aug 18, 2023 · 4 comments
Open

Feedback från studenter HT23 #61

AndreasArne opened this issue Aug 18, 2023 · 4 comments

Comments

@AndreasArne
Copy link
Member

Fyll på med feedback som studenter har gett i redovisningstexter.

@Ylih
Copy link

Ylih commented Sep 2, 2023

Tycker överlag kmom01 var enkel att utföra samtidigt som man fick använda de operationer som gåtts igenom. En bra start på kursen.

@kiwijos
Copy link

kiwijos commented Sep 11, 2023

Kommentar från student angående rättningsprogrammet och plane-uppgiften i kmom01. Håll till godo!

Jag gick igenom mitt program idag och märkte att jag hade gjort ett fel i min plane uppgift. När jag ville lista ut hur många feet över havet planet va så hade jag gångat med km/h med misstag. Det såg inte ert rättningsprogram för att den kollade samma siffra på alla varje test så den missade det felet. Så jag fick godkänt ändå.

@Miyarima
Copy link

Kmom03

Jag tror nått är galet med valideringen ibland!
Vid test av validate_ssn så ges det ibland personnummer som pylint anser är valid, men som jag (och min kod) hävdar är icke-valid.
Detta ex genom att testa pylints genererade perssonnumer på https://www.personnummer.nu/verifiera/
Links to an external site.
Min validering går igenom ca 9 fall av 10 när jag testar om och om igen.
Alla egen-genererade personnummer genom create_ssn går igenom validate_ssn
Exempel på när det blivit fel:
Fails for Marvin2NewMenus
|Testar att anropa menyval 9 via main funktionen i main.py.
|Använder följande som input:
|'7', '030123-4510', '', 'q'
|Använder funktionen 'validate_ssn()' för att validera det genererade personnummret:
|'Valid'
|Personnummret var inte giltigt:
Tänker att detta är tokigt från pylints sida, men jag har haft fel förut! 030123-4510 skall nämligen inte vara ett giltigt personnummer enligt uppgiftens egen algoritm när jag räknar själv, eller då genom personnummer.eu

@FalkenDev
Copy link

Kmom05 - Kommentar från student

Tycker det ofta är oklart vad målet är med uppgifterna. Man gör uppgiften och får ut vad man anser är rätt i menyvalen men sen måste man ändra hela koden när man kör dbwebb test för att dbwebb tycker returnen ska vara formaterad på ett visst sätt eller att printen inte är exakt så som den vill. Och det hade varit okej om det tydligt hade framgått hur allt skulle struktureras men jag tycker att det brister där så får man lägga helgen på att skriva om fullt fungerande kod varje vecka.

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

No branches or pull requests

5 participants