Waar let je op bij het uitvoeren van een code review?

• Overeenstemming met de code-stijl van het project
• Leesbaarheid en begrijpelijkheid van de code
• Afwezigheid van code duplicatie (DRY)
• Correct gebruik van architecturale patronen en ontwerpprincipes (SOLID)
• Foutafhandeling en randgevallen
• Afwezigheid van geheugenlekken (sterke referentiecycli, unmanaged objecten)
• Gebruik van geschikte datastructuren en algoritmen voor het oplossen van de taak
• Aanwezigheid van commentaar en hun actualiteit
• Aanwezigheid en dekking van unittests
• Prestatieoptimalisatie (indien van toepassing)
• Atomairiteit en leesbaarheid van commits
• Conformiteit van de implementatie met de taak en vereisten

// Voorbeeld van controle op afwezigheid van geheugenlekken
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Gebruik van weak om sterke cycli te voorkomen
}

// Voorbeeld van controle op DRY
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Gemeenschappelijke logica
    return a + b
}

func processNumbers(_ x: Int, _ y: Int) {
    let result = computeSum(x, y)
    // Extra verwerking
    print(result)
}

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Hergebruik van computeSum
    // Extra verwerking
    print(result * 2)
}

Aspect	Wat ik controleer
Leesbaarheid	Variabelenamen, functies, klassen; inspringingen
Correctheid	Logica, foutafhandeling, randgevallen
Prestaties	Lussen, complexe berekeningen, werken met collecties
Testen	Aanwezigheid en kwaliteit van tests
Architectuur/Design	Conformiteit aan patronen en principes