Bortkommenterat och bortglömt
Dec 18
De senaste veckorna har jag spenderat ihop med Fabian i ICC, ICC är ett team som har hand om integrationer mellan olika system så att de kan kommunicera på ett enkelt och smart sätt. Vad har vi fått göra här då, ja oftast har vi suttit med näsan i kod! Från html till java, från css:er till jsf och javascript. Vi har fått använda mycket av det vi har lärt oss hittills och fått lära oss mycket nytt! Speciellt har jag lärt mig att kod blir gammal fort. Jag tror aldrig jag har sett så mycket knepig, oflexibel och bortkommenterad kod som nu! Så varför händer detta? Mycket beror nog på att det ofta är många olika utvecklare som sitter i samma kod vid många olika tillfällen och ändrar, uppdaterar och lägger till nytt i koden. Oftast när man sitter och ska lägga till funktionalitet eller ska rätta en bugg som upptäckts så försöker man oftast sätta sig in i koden, snart inser man att det är för stort för att sätta sig in i. Så ger man upp och försöker istället hitta knasiga saker som kan ställa till problem (exempelvis ett semikolon på fel ställe…). Då händer det att man lägger utskrifter, kommenterar bort kod och så vidare, för att testa sina lösningar, samtidigt som man tänker ”det där ska jag fixa/ta bort/byta namn på sen när jag är klar!”. Låt mig säga en sak: det händer aldrig. Hur mycket man än vill så är det alltid någonting man glömmer bort att göra eller som man helt enkelt inte hinner för det finns viktigare saker att göra. På så vis skapar alla dessa människor som är inne i koden och ändrar, en kod i kaos som bara blir svårare och svårare att hantera.
Så snälla kodare där ute! Städa koden, tänk nu; inte sen. Var inte rädd för att ta bort kod som inte används eller kommentarer som inte varit aktuella på flera år. Det är dags att städa koden och underlätta för oss själva och varandra!
Isabel Siljelid
Mar 31, 2016 @ 06:04:51
Hej och tack för kommentaren! 🙂
Absolut ska vi vara glada för att koden förbättras!
Anledningen till svengelskan är så enkel att vi enligt våra riktlinjer ska skriva vår kod och alla kommentarer på svenska, med undantag för vissa allmänt accepterade nyckelord, exempelvis get, set, update. I vissa fall (som i detta) så är koden äldre och riktlinjerna fanns nog inte vid tillfället, så alla kodade lite som de själva ville.
DevMan
Mar 30, 2016 @ 20:57:07
Jag håller fullständigt med om att gammal bortremmad kod ska bort, men vi får väl ändå vara glada över att det är så pass risig kod som kommenterats bort?
Svengelskan i metoderna handleKvittoRequest(), signBean.isKvittoRequest() och dess like signBean.setKvittoRequest() skär genom hjärtat.
Konsekvensen verkar vara obefintlig, då felmeddelandet lite längre ner är formulerat på engelska, medan vissa andra kommentarer är på svenska.
Å andra sidan ser det illa ut längre upp också, i renderMode.updateBehorighet(). Huvaligen.
Helena
Jan 14, 2016 @ 13:27:22
Utan att vara utvecklare själv så är jag säker på att din analys är riktig! Din förhoppning , NU istället för SEN, är bra, håll fast vid den 😉