Worauf achtest du, wenn du einen Code-Review durchführst?

• Übereinstimmung mit dem Code-Stil des Projekts
• Lesbarkeit und Verständlichkeit des Codes
• Keine Code-Duplikate (DRY)
• Richtige Verwendung von Architekturmustern und Designprinzipien (SOLID)
• Fehlerbehandlung und Grenzfälle
• Keine Speicherlecks (starke Referenzzyklen, unmanaged Objekte)
• Verwendung geeigneter Datenstrukturen und Algorithmen zur Lösung der Aufgabe
• Vorhandensein und Aktualität von Kommentaren
• Vorhandensein und Abdeckung von Unit-Tests
• Leistungsoptimierung (falls zutreffend)
• Atomare und gut lesbare Commits
• Übereinstimmung der Implementierung mit der Aufgabe und den Anforderungen

// Beispiel zur Überprüfung des Fehlens von Speicherlecks
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Verwendung von weak, um starke Zyklen zu vermeiden
}

// Beispiel zur Überprüfung von DRY
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Gemeinsame Logik
    return a + b
}

func processNumbers(_ x: Int, _ y: Int) {
    let result = computeSum(x, y)
    // Zusätzliche Verarbeitung
    print(result)
}

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Wiederverwendung von computeSum
    // Zusätzliche Verarbeitung
    print(result * 2)
}

Aspekt	Was ich überprüfe
Lesbarkeit	Variablennamen, Funktionen, Klassen; Einrückungen
Korrektheit	Logik, Fehlerbehandlung, Grenzfälle
Leistung	Schleifen, komplexe Berechnungen, Arbeit mit Sammlungen
Tests	Vorhandensein und Qualität der Tests
Architektur/Design	Einhaltung von Mustern und Prinzipien