La ce acorzi atenție când faci o revizuire de cod?

• Conformitatea cu stilul de cod al proiectului
• Ușurința de citire și înțelegere a codului
• Absența duplicării codului (DRY)
• Utilizarea corectă a pattern-urilor arhitecturale și a principiilor de proiectare (SOLID)
• Tratarea erorilor și a cazurilor limită
• Absența scurgerilor de memorie (circuite de referință puternice, obiecte unmanaged)
• Utilizarea structurilor de date și algoritmilor potriviți pentru rezolvarea sarcinii
• Prezența comentariilor și actualitatea acestora
• Existența și acoperirea testelor unitare
• Optimizarea performanței (dacă este cazul)
• Atomicitatea și lizibilitatea commit-urilor
• Conformitatea implementării cu sarcina și cerințele

// Exemplu de verificare a absenței scurgerilor de memorie
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Utilizarea lui weak pentru evitarea ciclurilor puternice
}

// Exemplu de verificare DRY
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Logică comună
    return a + b
}

func processNumbers(_ x: Int, _ y: Int) {
    let result = computeSum(x, y)
    // Procesare suplimentară
    print(result)
}

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Reutilizarea lui computeSum
    // Procesare suplimentară
    print(result * 2)
}

Aspect	Ce verific eu
Lizibilitate	Numele variabilelor, funcțiilor, claselor; indentare
Corectitudine	Logică, tratarea erorilor, cazuri limită
Performanță	Buclă, calcule complexe, lucru cu colecții
Testare	Prezența și calitatea testelor
Arhitectură/Design	Conformitatea cu pattern-urile și principiile