A cosa presti attenzione quando fai una revisione del codice?

• Conformità con lo stile di codice del progetto
• Facilità di lettura e comprensione del codice
• Assenza di duplicazione del codice (DRY)
• Uso corretto di pattern architetturali e principi di progettazione (SOLID)
• Gestione degli errori e casi limite
• Assenza di perdite di memoria (cicli di riferimenti forti, oggetti non gestiti)
• Uso di strutture dati e algoritmi appropriati per risolvere il compito
• Presenza di commenti e loro attualità
• Presenza e copertura di test unitari
• Ottimizzazione delle prestazioni (se applicabile)
• Atomicità e leggibilità dei commit
• Conformità dell'implementazione con il compito e i requisiti

// Esempio di verifica dell'assenza di perdite di memoria
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Uso di weak per evitare cicli forti
}

// Esempio di verifica DRY
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Logica comune
    return a + b
}

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

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Riutilizzo di computeSum
    // Elaborazione aggiuntiva
    print(result * 2)
}

Aspetto	Cosa verifico
Leggibilità	Nomi variabili, funzioni, classi; indentazioni
Correttezza	Logica, gestione errori, casi limite
Prestazioni	Cicli, calcoli complessi, lavoro con collezioni
Test	Presenza e qualità dei test
Architettura/Design	Conformità a pattern e principi