Code review og pair programming

Programmering B · HTX · B-niveau · Software-udvikling

💻 Code review og pair programming

Code review = systematisk gennemgang af kode for at finde fejl, forbedre kvalitet og videndele.

Hvad man kigger efter:

1. Korrekthed: virker logikken? Edge cases håndteret?

2. Læsbarhed: forklarende variabelnavne? Nødvendige kommentarer?

3. DRY (Don't Repeat Yourself): duplikeret kode?

4. Sikkerhed: SQL injection? Ukrypteret data?

5. Performance: unødvendige loops? Ineffektive datastrukturer?

6. Tests: dækker tests det nye?

Konstruktiv feedback-stil:

• Specifik: "Linje 45: denne liste-søgning O(n) kan erstattes med dict-lookup O(1)"
• Spørgsmål frem for påbud: "Hvad sker der her hvis listen er tom?"
• Anerkend det gode: "Flot brug af comprehension på linje 23"

Pull Request-beskeder: forklar HVAD og HVORFOR. "Ret bug" er dårlig. "Ret NaN-fejl i BMI-beregning ved højde = 0" er god.

Pair programming (XP-praksis):

• Driver: skriver kode
• Navigator: reviewer og tænker fremad
• Skift roller hvert 25. minut

Fordele: færre fejl, videndeling, onboarding, bedre design.

Ulemper: kan føles ineffektivt, kræver god kommunikation.

Læringsmål

• Gennemføre konstruktive code reviews med specifikke kommentarer
• Bruge pull requests som del af team-workflow
• Anvende pair programming-teknikken
• Identificere og forklare DRY-violations

Sådan kan du arbejde med emnet

• Beskriv hvad man typisk kigger efter, når man laver code review på en andens kode
• Forklar forskellen på rollerne "driver" og "navigator" i pair programming
• Diskutér en fordel og en mulig udfordring ved at programmere i par

Træningsforslag

• Review en klassekammerats kode og skriv mindst 5 konkrete forbedringer
• Lav et projekt med pull requests og code review-krav
• Prøv pair programming på en kompleks algoritme-opgave