На какво обръщаш внимание, когато правиш преглед на код?

• Съответствие с стила на кодиране на проекта
• Лесна за четене и разбиране код
• Липса на дублиране на код (DRY)
• Правилна употреба на архитектурни шаблони и принципи на проектиране (SOLID)
• Обработка на грешки и гранични случаи
• Липса на течове на памет (силни цикли на препратки, unmanaged обекти)
• Използване на подходящи структури от данни и алгоритми за решаване на задачата
• Наличие на коментари и тяхната актуалност
• Наличие и покритие на юнит тестове
• Оптимизация на производителността (ако е приложимо)
• Атомарност и четливост на комитите
• Съответствие на реализацията с поставената задача и изискванията

// Пример за проверка за липса на течове на памет
class Parent {
    var child: Child?
}

class Child {
    weak var parent: Parent? // Използване на weak за избягване на силни цикли
}

// Пример за проверка на DRY
func computeSum(_ a: Int, _ b: Int) -> Int {
    // Общата логика
    return a + b
}

func processNumbers(_ x: Int, _ y: Int) {
    let result = computeSum(x, y)
    // Допълнителна обработка
    print(result)
}

func handleValues(_ p: Int, _ q: Int) {
    let result = computeSum(p, q) // Повторно използване на computeSum
    // Допълнителна обработка
    print(result * 2)
}

Аспект	Какво проверявам
Читаемост	Имена на променливи, функции, класове; отстъпи
Коректност	Логика, обработка на грешки, гранични случаи
Производителност	Цикли, сложни изчисления, работа с колекции
Тестове	Наличие и качество на тестовете
Архитектура/Дизайн	Съответствие с шаблони и принципи