search

jhoorneman / bestRouteFinder

0 stars 1 forks source link

Resultaat wordt niet live uitgelegd #2

Open snyke7 opened 3 years ago

snyke7 commented 3 years ago

Iedereen kan nu een code review doen en akkoord gaan, maar de find_winner methode zal nu alleen in 1 keer uitprinten welke route gewonnen heeft.

Zou het niet nice zijn als het (heel Amerikaans) een spektakel wordt, met tussenstanden die ook worden uitgeprint en zo?

PS: En mist de logica nu niet dat als een route meer dan de helft van de stemmen heeft, die automatisch wint? Al maakt dat niet uit voor het resultaat

jhoorneman commented 3 years ago

Ja dat stond idd nog op de to do list. Opavote, waar dit hele gebeuren door geïnspireerd is, laat dat heel mooi zien met een reeks plotjes Maar dat is veel werk, dus tussenstandjes in tekst is ook al nice.

We houden op dit moment nog niet op als 1 optie de meerderheid heeft inderdaad. Voor snelheid maakt het niets uit, want dit is ws instantaan berekend. Maar het voorkomt een hoop ties, dus misschien toch goed om te implementeren.