Beoordeel mijn werk (naamloting)

Op het eerste ogenblik ziet het er netjes uit. Echter merk ik dat de knop om namen toe te voegen een "Forbidden" cursor geeft als je een veld niet hebt ingevuld, maar werkt toch. Ik zou die cursor gewoon weghalen dan.

Misschien is het leuk om een kort en bondig praktijkvoorbeeldje op je site te zetten.

Verder kan je de hele JS vast wel minifieën, er staat een hoop onnodig commentaar in.

Misschien een handig idee: Controleer op dubbele namen, en laat dan niet loten. Op die manier hebben we ooit eens in de familie per ongeluk een dubbele naam gehad tijdens lootjestrekken met Sinterklaas, met verbaasde gevolgen alom ;-)

Ziet er leuk uit, maar qua techniek wil je misschien iets doen met een minifier, de javascript is gigantisch zoals het nu is. Qua CSS: ik snap dat je bootstrap gebruikt, maar om nu zoveel zinloze CSS te hebben voor zoiets simpels gaat mij veel te ver. Minification speelt ook hier.

Ik heb het even getest met 2 namen, alleen krijg ik de namen telkens om en om als resultaat (als je meerdere lotingen doet). Weet niet of dit nou puur toeval is.

bijvoorbeeld na 10 lotingen zijn de oneven lotingen : "Jip" en de even lotingen: "Janneke". 50/50 dus.

@Midas klopt. Er kan nooit achter elkaar dezelfde winnaar zijn. Uit tests is gebleken dat dit niet handig en gewenst is.

@Ben er was een bug die ontstond tijdens de minify van de JavaScript, dit is inmiddels verholpen en alles is geminified

@Arien Er kunnen dubbele namen ingevoerd worden omdat er misschien mensen zijn met dezelfde naam. Er staan lotnummers voor wat de naam uniek kan maken
De disabled cursor was een bug vrees ik en is eruit gehaald.

Thanks voor de feedback en de bezoeken.

>> Er kan nooit achter elkaar dezelfde winnaar zijn. Uit tests is gebleken dat dit niet handig en gewenst is.

Hahaha ... en daar gaat het principe van de loting. Een loting is willekeurig en niet te voorspellen. Dat is nu juist de essentie ervan!

@Ozzie voordat je te hard gaat roepen.
http://www.naamloten.nl/

Ook hier geldt deze regel.

Maar bedankt voor je opbouwende kritiek.

Dus, omdat een ander het zo doet moet jij het ook zo doen? Bijzonder.
Toegegeven, rand() is niet lineair, maar wanneer je bijvoorbeeld iets gebruikt als lcg_rand() is dit geen probleem meer. Wanneer cryptografische random nodig zou zijn (eigenlijk wel logisch als het om loterijen gaat) kan het handiger zijn om de openssl functies te gebruiken, of bijvoorbeeld random_int().

Nee Ben dat is niet mijn uitgangspunt maar na een korte check bleek dat naamloten.nl dit ook doet. De hebben dezelfde gedachte gehad als ik waarschijnlijk.

Verder kan ik geen php voor de loterij, de reden waarom het zo snel is komt doordat het in JavaScript is geschreven. Openssl is geen optie helaas.

Waar baseer je dat op dat die functie van OpenSSL geen optie is?

Misschien omdat de website gebruik maakt van let's encrypt Arien?
Ik zou snappen dat je dan openSSL niet daarnaast laat werken.
Ben wel benieuwd of er een advies komt hoe je dat dan zou moeten doen.

Dat er geen php gebruikt kan worden snap ik eigenlijk niet helemaal.
Ik zou me kunnen voorstellen dat men een hele berg namen zou willen invoegen.
Misschien wel over dagen. Dan zou het makkelijker zijn als je de loterij aan een database koppelt waarin je de namen toe kan voegen. Misschien een API om andere websites te bedienen.
Je kan het voor zoveel meer dingen gebruiken, denk aan een ticket service bijvoorbeeld.
Dit kan echt wel een groot ding worden. Tis maar net hoever je wil gaan.
Hier zou ik wel brood in zien.

Overigens echt leuk gemaakt, simpel van opzet, makkelijk te gebruikt.
Een echte topper. :)