Wat een aardige TweakerF.West98 schreef op vrijdag 09 september 2016 @ 21:38:
Members only:
Alleen zichtbaar voor ingelogde gebruikers. Inloggen
Probleem blijft altijd dat er meerdere wegen naar Rome leiden en je in software development al snel verzandt in een discussie over persoonlijke voorkeuren. Veel codereviews die men onderling doet gaan over guidelines, dan lopen collegae elkaar af te kraken dat het schijnbaar moeilijk is om je aan de standaarden te houden en je daarom slechte en onleesbare code maakt. Dat soort reviews gaan bij mij in de vuilnisbak, want die slaan nergens op. Zolang het maar consistent is, dan wen je echt wel aan de codeer standaard van een developer. Geloof me, ik heb genoeg code gezien die 100% voldeed aan de guidelines, maar echt technisch om te janken was en zwaar instabiel bleek te zijn.
Een review moet meer zien op het naleven van bepaalde principes en het maken van code die begrijpelijk is. Of een accolade op de goede regel staat boeit in principe geen moer voor de onderhoudbaarheid.
Zelf ben ik veel bezig met de kwaliteit van code, aangezien ik een team van 9 man aanstuur, met verschillende niveau's. Daar is het belangrijk dat ze een standaard kader hebben waarbinnen ze kunnen werken en dat ze kunnen vertrouwen dat de code om hen heen stabiel genoeg is (en ze dat ook als referentie mogen gebruiken). Daarnaast werken we ook samen met Fontys Hogescholen op een aantal projecten, waarbinnen wij zwaar inzetten op kwaliteit en good-code-craftmanship. Heb diverse interessante gesprekken gehad met mensen daar en het blijft lastig om in die 4 jaar mensen echt te leren programmeren op een goede manier. Je kan niet meteen aankomen met een HelloWorldFactoryBuilderProxyDecorator in het eerste jaar, maar je wil ze het ook niet onthouden tot hun afstuderen, dus moet je binnen je curriculum best rare sprongen maken. Daarnaast verwacht de business heel andere zaken dan een docent op een opleiding. Wat kan mij een strak technisch design nou boeien, dingen veranderen toch binnen no-time.
The #1 programmer excuse for legitimately slacking off: "My code's compiling"
Firesphere: Sommige mensen verdienen gewoon een High Five. In the Face. With a chair.