GoDasturchi
Giving Feedback That's Specific and Kind

Giving Feedback That's Specific and Kind

Aniq va mehribon fikr bildirish

Ikki xil murabbiyni tasavvur qiling. Birinchisi sportchisiga "sen yomon o'ynading" deydi — bu haqiqat bo'lishi mumkin, lekin sportchi ENDI NIMA qilishni bilmaydi. Ikkinchisi esa: "sening chap oyog'ing burilishda bir zarb kech qo'yilyapti — shuning uchun to'p sendan uzoqlashyapti, keyingi safar bir zarb OLDINROQ qo'y" deydi. Ikkalasi ham "muammo bor" deydi, lekin faqat ikkinchisi sportchiga NIMA o'zgartirishni aniq ko'rsatadi.

Oldingi beshta darsda siz turli xil real xatolarni (nil pointer, yutilgan xato, race condition, resurs sizishi, off-by-one) topishni mashq qildingiz. Lekin xatoni TOPISH — ishning yarmi. Ikkinchi yarmi — buni yozgan odamga qanday AYTISH, shunda u nafaqat tuzatadi, balki KEYINGI safar shu xatoni o'zi oldindan ko'radi.

Yomon fikrYaxshi fikr
"Bu noto'g'ri.""Bu yerda u.Profile nil bo'lishi mumkin (yuqoridagi struct izohiga qarang) — u.Profile.AvatarURLga murojaat qilishdan oldin nil tekshiruvi qo'shsak, panic'ning oldi olinadi."
"Bu funksiya yomon yozilgan.""Bu funksiya to'rtta ish qilyapti (tekshirish, saqlash, xabar yuborish, jurnal yuritish) — ularni alohida funksiyalarga bo'lsak, har birini alohida sinash osonlashadi."
"Nega bunday qilding?""Men bu yerda X yondashuvini kutgandim, chunki Y sabab bilan — sen Z yo'lni tanlagansan, buning sababini tushuntirib bera olasanmi?"

"Yaxshi fikr" ustunidagi har bir misolda uchta narsa bor: (1) ANIQ qaysi qator yoki qism haqida gap ketayotgani, (2) NEGA bu muammo ekanligining tushuntirilishi, va (3) KO'PINCHA, qanday tuzatish mumkinligi haqida taklif. Bu — Reading a Stack Trace darsida ko'rgan "aniq joy va aniq sabab" g'oyasining, endi INSONGA aytilgan fikrga qo'llanilishi.

"Mehribonlik" — muammoni yashirish yoki yumshatish DEGANI EMAS. Bu — muammoni ODAMGA emas, KODGA qaratib aytish: "sen xato qilding" o'rniga "bu yerda nil tekshiruvi yetishmayapti" deyish. Kod muallifi himoya pozitsiyasiga o'tmasdan, tinch holda muammoni ko'rib, tuzatishga tayyor bo'ladi — bu esa oxir-oqibat TEZROQ, sifatliroq tuzatishga olib keladi.

Foydali odat: fikr bildirishdan oldin o'zingizga savol bering — "agar men bu fikrni OVOZ CHIQARIB, muallifning YUZIGA qarab aytsam, xijolat bo'larmidim?" Agar javob "ha" bo'lsa, fikrni qayta yozib, KONKRET va KOD haqida qiling, shaxsga qaratilgan tondan qochib.

Key Takeaway

Key Takeaway:

Yaxshi review fikri uchta narsani o'z ichiga oladi: aniq qaysi joy haqida gap ketyapti, nega bu muammo ekanligi, va qanday tuzatish mumkinligi. Muammoni odamga emas, kodga qarating — bu muallifni himoyaga o'tishdan, tezroq tuzatishga yo'naltiradi.

NEXT UP

Blocking Comments vs. Nitpicks

Giving Feedback That's Specific and Kind

Aniq va mehribon fikr bildirish

Ikki xil murabbiyni tasavvur qiling. Birinchisi sportchisiga "sen yomon o'ynading" deydi — bu haqiqat bo'lishi mumkin, lekin sportchi ENDI NIMA qilishni bilmaydi. Ikkinchisi esa: "sening chap oyog'ing burilishda bir zarb kech qo'yilyapti — shuning uchun to'p sendan uzoqlashyapti, keyingi safar bir zarb OLDINROQ qo'y" deydi. Ikkalasi ham "muammo bor" deydi, lekin faqat ikkinchisi sportchiga NIMA o'zgartirishni aniq ko'rsatadi.

Oldingi beshta darsda siz turli xil real xatolarni (nil pointer, yutilgan xato, race condition, resurs sizishi, off-by-one) topishni mashq qildingiz. Lekin xatoni TOPISH — ishning yarmi. Ikkinchi yarmi — buni yozgan odamga qanday AYTISH, shunda u nafaqat tuzatadi, balki KEYINGI safar shu xatoni o'zi oldindan ko'radi.

Yomon fikrYaxshi fikr
"Bu noto'g'ri.""Bu yerda u.Profile nil bo'lishi mumkin (yuqoridagi struct izohiga qarang) — u.Profile.AvatarURLga murojaat qilishdan oldin nil tekshiruvi qo'shsak, panic'ning oldi olinadi."
"Bu funksiya yomon yozilgan.""Bu funksiya to'rtta ish qilyapti (tekshirish, saqlash, xabar yuborish, jurnal yuritish) — ularni alohida funksiyalarga bo'lsak, har birini alohida sinash osonlashadi."
"Nega bunday qilding?""Men bu yerda X yondashuvini kutgandim, chunki Y sabab bilan — sen Z yo'lni tanlagansan, buning sababini tushuntirib bera olasanmi?"

"Yaxshi fikr" ustunidagi har bir misolda uchta narsa bor: (1) ANIQ qaysi qator yoki qism haqida gap ketayotgani, (2) NEGA bu muammo ekanligining tushuntirilishi, va (3) KO'PINCHA, qanday tuzatish mumkinligi haqida taklif. Bu — Reading a Stack Trace darsida ko'rgan "aniq joy va aniq sabab" g'oyasining, endi INSONGA aytilgan fikrga qo'llanilishi.

"Mehribonlik" — muammoni yashirish yoki yumshatish DEGANI EMAS. Bu — muammoni ODAMGA emas, KODGA qaratib aytish: "sen xato qilding" o'rniga "bu yerda nil tekshiruvi yetishmayapti" deyish. Kod muallifi himoya pozitsiyasiga o'tmasdan, tinch holda muammoni ko'rib, tuzatishga tayyor bo'ladi — bu esa oxir-oqibat TEZROQ, sifatliroq tuzatishga olib keladi.

Foydali odat: fikr bildirishdan oldin o'zingizga savol bering — "agar men bu fikrni OVOZ CHIQARIB, muallifning YUZIGA qarab aytsam, xijolat bo'larmidim?" Agar javob "ha" bo'lsa, fikrni qayta yozib, KONKRET va KOD haqida qiling, shaxsga qaratilgan tondan qochib.

Key Takeaway

Key Takeaway:

Yaxshi review fikri uchta narsani o'z ichiga oladi: aniq qaysi joy haqida gap ketyapti, nega bu muammo ekanligi, va qanday tuzatish mumkinligi. Muammoni odamga emas, kodga qarating — bu muallifni himoyaga o'tishdan, tezroq tuzatishga yo'naltiradi.

NEXT UP

Blocking Comments vs. Nitpicks