TDP005 Projekt: Objektorienterade System
Kodgranskning
Som ett moment i kursen ingår kodgranskning, och att kunna organisera ett kodgranskningsmöte. Ni ska i grupper om 2-3 projektgrupper ha ett möte där ni går igenom varandras kod. Detta möte ska ske en bit in i projektet när ni påbörjat er kodning, men helst innan all kod är klar. Det kan lämpligtvis ske vid det för ändamålet schemalagda tillfället (se "Schema"), då Jonas finns tillgänglig för handledning. Men det går bra för er inom kodgranskningsgrupperna att komma överens om ett annat tillfälle. Om grupperna är i ofas går det även bra att ha mer än ett kodgranskningsmöte, så att ni granskar olika gruppers kod vid olika tillfällen. Varje projektgrupp ska skriva en egen rapport om kodgranskningen.
- Deadline: Se "Schema"
- Fokus för granskningen:
- Ni skall granska de andra gruppernas kod utifrån bedömningskriterierna angivna under "Projektbeskrivning". Särskild fokus bör ligga på:
- Klassdesign (Code Complete, kapitel 6)
- Variabler, speciellt ur ett OO-perspektiv (Code Complete, kapitel 10-13)
- Satser (Code Complete, kapitel 14-19) (Dessa kategorier finns närmare beskrivna i Code Complete, en bok ni kan finna i biblioteket om ni inte redan äger den.)
- Täck även andra aspekter som dyker upp.
- Glöm inte att ge både positiv feedback på sådant som är bra, som konstruktiv feedback på sådant som är problematiskt. Försök ge kritik som kan hjälpa den andra gruppen att förbättra sin kod!
- Ni skall granska de andra gruppernas kod utifrån bedömningskriterierna angivna under "Projektbeskrivning". Särskild fokus bör ligga på:
- Innehåll:
- Kort redogörelse för hur kodgranskningsmötet gick till, samt när det hölls.
- En beskrivning av er granskning av de andra gruppernas kod. Fokusera gärna på de punkterna ni anser är viktigast, och ge en mer översiktlig beskrivning av övrigt.
- Beskriv vilken feedback ni fått av de andra grupperna, och vilka konskvenser det fått för er fortsatta utveckling.
- Lagom omfattning på er rapport är kring 2-3 sidor. Tänk på att kvalitet är viktigare än kvantitet!
Kodgranskningsgrupper
Följande grupper ska granska varandras kod:
-
Erik Sneitz + Marcus Sneitz
Fredrik Kujansuu + Victor Ellingsen
Fredrik Öhrström + Jesper Bäck
-
Maija Vilkina + Olle Kvarnström
Leo Granström + Shpat Krasniqi
André Andersson + Theodor Bladh
-
Arvid Karlsson + Simon Gustafsson
Björn Kihlström + Daniel Roxbo
Hampus Andersson + Isak Jonason
-
David Svenson + Martin Samuelsson
Joakim Andersson + Marianne Nordlund
Elias Lund Lilja + Harald Vogel
-
Andreas Isacsson + Karl Knutsson
Erik Emanuelsson + Johanna Gripenstam
-
Christoffer Stridh + Tommy Lindell
Daniel Persson + Victor Dannetun
För de kodgranskningsgrupper som består av tre projektgrupper får ni själva välja hur ni vill lägga upp granskningen. Ni kan antingen göra så att alla granskar varandras kod, eller att ni "cirkelgranskar", så att varje grupp granskar och blir granskad av en annan grupp.
Sidansvarig: Jonas Lindgren
Senast uppdaterad: 2013-12-15