Качество кода: читаемость, скорость и линтер

Вера Ланская · лид QA, #qa-antrakt

Посмотрела твой MR. Логика верная, но функцию на пятнадцать строк я читала двадцать минут, а у Глеба таких файлов сорок. И ещё: тест схемы фестивального поля идёт пять секунд, хотя проверяет одну вещь. До «Ночи продаж» хочу две вещи: линтер и форматтер в репозитории с этой недели и быстрый free_seats.

Код тестов читают чаще, чем пишут. Его открывают, когда тест упал в три часа ночи, когда меняется продукт, когда приходит новичок. Поэтому в этом уроке три темы в таком порядке: сначала понятность, потом скорость там, где она правда нужна, и в конце инструменты, которые следят за кодом, пока ты занят тестами.

Кухня, где всё на местах

На профессиональной кухне у каждого ножа, сковороды и банки со специями своё место, одно на всю смену. Повар, который пришёл сегодня впервые, находит соль за секунду, не спрашивая. Если каждый раскладывает по-своему, к вечеру соль ищут вдесятером.

Стайлгайд это правило «где что лежит». Линтер это шеф, который обходит кухню и говорит: «нож не на месте, а вот тут молоко скисло». Форматтер это уборщик, который молча раскладывает всё по местам.