Går vidare med E1, E2 och F1 - F4
E1: Kräver mer än ett steg för att bygga
Din arbetsplats ska vara så automatiserad som möjligt. Faktiskt tycker jag inte att denna punkt går tillräkligt långt. Kompilering och bygge borde ske i bakgrunden och aldrig behöva aktiveras manuellt, utan alltid vara klart. Men självklart ska man inte behöva köra olika skript, flytta filer manuellt för att kunna bygga applikationen. Jag skulle också vilja tillägga att allt ska gå snabbt. Att vänta i 2-3 minuter flera gånger i timmen för att kolla att något fungerar är en enorm tidstjuv.
E2: Kräver mer än ett steg för att köra tester
Inget konstigt och i samma anda som E1. Din arbetsplats skall vara så automatiserad som möjligt. Kommandot för att köra alla tester ska alltid ligga framme för enkel åtkomst och inte kräva att man följer en wizard etc. Snabbt, enkelt och självklart.
F1: För många argument
Metoder som inte tar några argument är bäst. En, två och undantagsvis tre variabler kan vara acceptabelt. Fler variabler i en metod bör utlösa en flod av förslag på förändringar som kan få koden att lösa sin uppgift på ett bättre sätt.
F2: "Output" argument
När man använder outputargument tvingar man läsaren av koden att tänka baklänges. Många klarar att tänka baklänges men för nästan alla kräver det en extra insats. Det är bättre att returnera ett objekt som innehåller de variabler som ska vara tillgänglia efter metodanropet. Detta gäller även för att modifiera collections där det är tydligare om man returnerar den modifierade "samlingen" än lämnar läsaren av koden med frågan om något hände med objektet som stoppades in i metoden.
F3: Boolska flaggor som arguement
En boolean som argument visar ofta att metoden har två uppgifter. Om argumentet är sant gör si och om det är falskt gör så. Bättre då att skriva två olika metoder eller kanske till och med tre metoder. Tre metoder förklaras med att den boolska flaggan bara påverkar en del av metoden och då kan man skriva den gemensamma delen som en metod och sedan två metoder för den delen som styrs av flaggan.
F4: Oanvända funktioner
Funktioner som inte används raderar vi självklart och tveklöst. Vi har versionshanteringssystem för att hitta kod som en gång i tiden funnits.
lördag 31 januari 2009
Prenumerera på:
Kommentarer till inlägget (Atom)
Inga kommentarer:
Skicka en kommentar