Code review — kod o‘zgarishini asosiy branchga qo‘shishdan oldin boshqa dasturchilar tomonidan tekshirish jarayoni. U xato topish bilan birga dizayn, tushunarlilik, test, xavfsizlik va jamoa standartlariga moslikni baholaydi. Review avtomatik test va statik tahlilning o‘rnini bosmaydi; mashina tekshiruvlari takrorlanadigan qoidalarni, reviewer esa kontekst va trade-offlarni ko‘radi.
O‘zgarishni tayyorlash
Kichik va bitta maqsadga qaratilgan pull requestni tushunish osonroq. Muammo, tanlangan yechim, muhim qaror, test usuli va deployment ta’siri tavsifda yoziladi. Katta refactoring bilan yangi funksiyani bir o‘zgarishda aralashtirish diffni kattalashtiradi va xato ehtimolini yashiradi.
Muallif reviewdan oldin o‘z diffini qayta o‘qiydi. Debug kodi, tasodifiy fayl, maxfiy qiymat va keraksiz format o‘zgarishi olib tashlanadi. CI build, lint va testlarni bajaradi. Generated file yoki dependency lockdagi katta o‘zgarish sababi alohida ko‘rsatiladi.
Reviewer tekshiruvi
Reviewer avval talab va umumiy dizaynni, keyin implementatsiya detallarini ko‘radi. Kod normal holat bilan birga xato, timeout, null qiymat, concurrency va ruxsat holatlarini qanday boshqarishi tekshiriladi. Ma’lumotlar bazasi o‘zgarishida migration, indeks, transaction va eski versiya bilan moslik ko‘riladi.
Security tekshiruvi input validation, authorization, secret, logdagi maxfiy ma’lumot va injection xavfini qamrab oladi. Performance bo‘yicha esa keraksiz query, cheksiz loop, katta xotira allocationi yoki bloklovchi I/O aniqlanishi mumkin. Har o‘zgarishda barcha ehtimoliy masalani chuqur tekshirish shart emas; riskga qarab e’tibor taqsimlanadi.
Izoh yozish
Review izohi kodga emas, shaxsga qaratiladi. Muammo, uning ta’siri va kerak bo‘lsa alternativ yo‘l tushuntiriladi. “Buni o‘zgartir” o‘rniga “bu funksiya transaction tashqarisida ishlagani sababli ikki yozuv orasida qisman holat qolishi mumkin” aniqroq.
Blocking comment merge oldidan tuzatish talab qiladi. Non-blocking suggestion yaxshilash g‘oyasi bo‘lib, joriy o‘zgarishni to‘xtatmasligi mumkin. Jamoa nit, question, suggestion kabi belgilar ma’nosini kelishib oladi. Uslub bo‘yicha bahs formatter yoki lint qoidasi bilan avtomatlashtiriladi.
Jarayon
Muallif izohga javob beradi, kodni o‘zgartiradi yoki qarorini dalil bilan tushuntiradi. Yangi commitdan keyin reviewer muhim qismni qayta tekshiradi. Approval kod mutlaqo xatosizligini kafolatlamaydi; u reviewer mavjud ma’lumot va tekshiruv doirasida o‘zgarishni qabul qilganini anglatadi.
Review uzoq kutsa yetkazib berish sekinlashadi. Jamoa kichik pull request, reviewer navbati va javob vaqti bo‘yicha ish tartibini belgilaydi. Murakkab arxitektura masalasi faqat diff izohida hal qilinmay, design reviewda oldinroq muhokama qilinishi mumkin.
Xavfga asoslangan review
Authentication, to‘lov, migration yoki concurrency o‘zgarishi oddiy matn tuzatishidan ko‘proq tekshiruv talab qiladi. CODEOWNERS ma’lum katalog uchun tegishli mutaxassis reviewini avtomatik so‘rashi mumkin. Juda ko‘p majburiy reviewer esa mas’uliyatni tarqatib, hamma boshqasi tekshiradi degan holat yaratishi mumkin.
Security-critical diff uchun threat model, dependency o‘zgarishi uchun supply chain va license tekshiruvi qo‘shiladi. Generated kod qo‘lda qatorma-qator o‘qilmasligi mumkin, lekin generator versiyasi, kirish sxemasi va hosil bo‘lgan API farqi tekshiriladi. Review checklist xotirani qo‘llab-quvvatlaydi, biroq kontekstual fikrlashni almashtirmaydi.
Bog‘liq tushunchalar
Pull request, diff, static analysis, CI, unit test, coding standard, design review