Ik moet hier een stuk code van een collega programmeur bijwerken. Collega in kwestie is er eentje van het 'als het maar werkt is het goed' soort: weinig tot geen attentie voor kleine details (zoals alles netjes uitlijnen in een user interface), foutboodschappen waar geen hond iets aan heeft (en die jammer genoeg meer zichtbaar zijn dan je zou willen), ... Enfin in ons plaatselijk dialect hebben we daar een leuk woord voor: ne kloskloot.
Nu dus ook weer hetzelfde treurige verhaal:
- globaal definieerde variabelen die toch bij elke functie binnen scope als argumenten worden doorgegeven (moest het nu nog by value zijn maar 't is begot by reference)
- het ene moment toont hij messageboxen met titel, dan weer zonder
- spelfouten in de berichten naar de gebruiker :s
- spelfouten in de namen van zijn variabelen. OK - Engels is niet iedereen z'n moedertaal en niet iedereen is er even vlot in maar hou het dan bij wat je wel kan. En schrijf woorden die je elke dag gebruikt in godsnaam zonder fout.
- logica waar welgeteld 2 seconden bij stilgestaan is. Op een gegeven moment moet een bestandsnaam in extensie en naam opgesplits worden. Eerst bepaalt hij dat het om een word bestand gaat door te test of alles links van .doc nog iets bevat (in het geval van een andere extensie wordt .doc niet gevonden en geeft die code een lege string terug). Een beetje knullig maar so far so good. Dan gaat hij de boel opsplitsen door alles behalve de laatste 4 karakters als naam te beschouwen en de laatste 4 karakters zijn dan de extensie. Hallo? Zijn we nog in 1990? Nog nooit van .docx gehoord zeker? Enfin het gevolg laat zich raded: elk .docx bestand ziet zijn naam gesplitst in naam. (met punt erachter) en docx. Na wat gefoefel met de inhoud van het bestand wordt het dan bewaard als naam + '.docx'. Jammer genoeg bevatte de naam al een puntje en krijg je onzin als "catalog_NL..docx". Om grijs haar van te krijgen...
Enfin de herwerkte code is een stuk efficienter, korter en is foutloos (zelfs die meldingen voor de gebruikers zijn nu in verstaanbaar Engels). All in a day's work zeker?
Geen opmerkingen:
Een reactie posten