À quoi fais-tu attention lorsque tu fais une revue de code?

• Conformité au style de code du projet
• Facilité de lecture et de compréhension du code
• Absence de duplication de code (DRY)
• Utilisation correcte des patrons architecturaux et principes de conception (SOLID)
• Gestion des erreurs et des cas limites
• Absence de fuites de mémoire (cycles de références fortes, objets non gérés)
• Utilisation de structures de données et d'algorithmes appropriés pour résoudre la tâche
• Présence de commentaires et leur actualité
• Présence et couverture des tests unitaires
• Optimisation des performances (si applicable)
• Atomicité et lisibilité des commits
• Conformité de la mise en œuvre avec la tâche et les exigences

// Exemple de vérification de l'absence de fuites de mémoire
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Utilisation de weak pour éviter les cycles forts
}

// Exemple de vérification du DRY
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Logique commune
    return a + b
}

func processNumbers(_ x: Int, _ y: Int) {
    let result = computeSum(x, y)
    // Traitement supplémentaire
    print(result)
}

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Réutilisation de computeSum
    // Traitement supplémentaire
    print(result * 2)
}

Aspect	Ce que je vérifie
Lisibilité	Noms de variables, fonctions, classes; indentations
Correctitude	Logique, gestion des erreurs, cas limites
Performance	Boucles, calculs complexes, travail avec collections
Tests	Présence et qualité des tests
Architecture/Design	Conformité aux patterns et principes