Moduldiskusjon:WikidataAlder

Bruk av variabelnavn

Aldri eksponer forkortede variabelnavn på utsiden av en funksjon, spesielt ikke i invoke-kallet. I disse tilfellene propagerer egenskaper fra koden og ut i omkringliggende systemer, noe som er sett på som bad style. — Jeblad 27. sep. 2015 kl. 16:13 (CEST)

Det er også en del navn som bruker «jumblenamestyle». Bruk CamelCase eller snake_style. — Jeblad 27. sep. 2015 kl. 16:40 (CEST)

separasjon av logikk og presentasjon, testtilfeller

Logikk og presentasjon bør separares, nå skjer alt i samme funksjonene. Det gjør testing vanskeligere, men ikke kritisk vanskelig i det her tilfellet. Testtilfeller mangler, det ville gitt en -2 fra meg. — Jeblad 27. sep. 2015 kl. 16:47 (CEST)

Jeg kan ikke huske å ha bedt om code review og jeg registrer at mitt forsøk på å bli kjent med lua en sen kveld har blitt belønnet med strykkarakter, motivasjonen for å gå igjennom kommentarene blir deretter. Profoss (diskusjon) 27. sep. 2015 kl. 20:23 (CEST)