Millele pöörad tähelepanu koodi ülevaatamisel?

• Koodistiilide vastavus projektistandarditele
• Koodi lihtne lugemine ja mõistmine
• Koodi dubleerimise vältimine (DRY)
• Õige arhitektuurimustrite ja disainiprintsiipide kasutamine (SOLID)
• Vigade ja piirjuhtumite käsitlemine
• Mälulekkede puudumine (tugevad viitedsilmused, haldamata objektid)
• Õige andmestruktuuride ja algoritmide kasutamine ülesande lahendamiseks
• Kommentaaride olemasolu ja nende asjakohasus
• Ühikutestid ja nende katvus
• Töövõime optimeerimine (kui rakendatav)
• Atomaarne ja loetav commitide ajalugu
• Teostuse vastavus ülesandele ja nõuetele

// Näide mälulekkede kontrollimiseks
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Kasutades weak, et vältida tugevaid tsükleid
}

// Näide DRY põhimõtte kontrollimiseks
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Ühine loogika
    return a + b
}

func processNumbers(_ x: Int, _ y: Int) {
    let result = computeSum(x, y)
    // Täiendav töötlemine
    print(result)
}

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Korduv computeSum kasutus
    // Täiendav töötlemine
    print(result * 2)
}

Aspekt	Mida kontrollin
Loetavus	Muutujate, funktsioonide, klasside nimed; sisestused
Korrektsus	Logika, vigade käsitlemine, piirjuhtumid
Tõhusus	Tsüklid, keerulised arvutused, kogumite kasutamine
Testimine	Testide olemasolu ja kvaliteet
Arhitektuur/disain	Vastavus mustritele, põhimõtetele