search

jhoorneman / bestRouteFinder

0 stars 1 forks source link

Static method voor maken van voters #1

Open PiotrBenedysiuk opened 3 years ago

PiotrBenedysiuk commented 3 years ago

https://github.com/jhoorneman/bestRouteFinder/blob/08f06223361be04a888e51da478d938e4605a2f8/voter.py#L7

Dit moet een static method zijn, netter is wel om het geen onderdeel te maken van voter. Bovendien, je kan @dataclass gebruiken.

Andere zaak: als je de typing module gebruikt dan kan je ook aangeven wat er in de (bijvoorbeeld) lists zit, zoals typing.List[int].

jhoorneman commented 3 years ago

Ja, die method was gemaakt toen voter nog de enige class was. Het plan is nu het uitlezen van de google form te verplaatsen naar Election.

Typing module ken ik idd. We hebben die niet gebruikt zodat je de hele code kunt runnen zonder dat je modules nodig hebt, maar misschien komen we daar nog op terug.

jhoorneman commented 3 years ago

DataClass kende ik nog niet trouwens, maar ziet er wel nice uit.

PiotrBenedysiuk commented 3 years ago

typing zit ingebouwd in python tegenwoordig

jhoorneman commented 3 years ago

Oh ik zie het ja. Dataclass ook. Dat is nice! Dan gaan we die wel gebruiken.