Skip to content

Latest commit

 

History

History
40 lines (30 loc) · 1.91 KB

P46.org

File metadata and controls

40 lines (30 loc) · 1.91 KB

P46: Gör en informell kodgranskning under fas 2

Gör en informell kodgranskning på ett program utvecklat av ett annat kodpar i samma grupp (det får inte finnas några överlappningar i medlemmar mellan granskande par och par som blir granskat). Syftet med en kodgranskning är att hitta defekter och förbättra kodens kvalitet. Följ ett kodgranskningsprotokoll och ta anteckningar.

Ge synpunkter på koden genom GitHubs issue tracker som finns för varje repo, eller genom att kommentera enskilda rader. Se till att använda en särskild version av programmet så att ni kan hitta tillbaka till just den vid redovisning, och för att undvika att programmet ändras samtidigt som ni gör granskningen.

Ute i arbetslivet gör man detta av flera anledningar:

  • Kvalitetssäkring
  • Utvecklare blir modigare när man delar ansvar för korrekthet
  • När man vet att någon måste läsa koden skriver man ofta bättre kod
  • Man sprider kunskap om vad som händer i projektet till fler

Det är en viktig färdighet att kunna kritisera någons kod utan att kritisera personen (och likaså att ta kritik!). Vidare blir man också ansvarig för koden som man OK:ar vilket gör att man måste vara noggrann och inte luras att tro att målet är att säga “vilken bra kod” till de man granskar.

Redovisning

Du redovisar detta under en labb genom att visa upp ditt kodgranskningsprotokoll och issue log. Ha koden tillgänglig för frågor från examinatorn. Minns att det är du som skall driva redovisningarna, inte examinatorn.


Report a bug on this achievement? Please place an issue on GitHub.