Како најбоље прегледати захтеве за повлачење (без решења за управљање) У технологији, увек радимо на писању бољег, јачег, одрживијег кода. Прегледи кода су суштински део тог процеса. Ипак, то је вјештина на коју се ретко фокусирамо на побољшање. Превише често, прегледи постају нешто што се пробијамо, успоравајући тимове и повређујући укупни квалитет. Постати квалитетни рецензент може вас учинити правом имовином за свој тим, али то није лако.Многи од нас приступају прегледима кода из погрешног угла или са погрешним сврхама. У о Можете трансформисати динамику вашег тима и поставити вас на пут ка успеху. Prave stvari Pravim putem The Purpose of a PR Review Svrha PR pregleda Као инжењери, наш примарни посао је да напишемо код. Да решимо проблеме. Да задржимо свет да се окреће један бит или бајт у исто време. Али која је наша улога када прегледамо код? So, let’s start with what you’re not expected to do: Не очекујете да ћете решити проблем за аутора. Не очекујете да поново напишете њихов код како бисте одговарали вашој визији. So, what are you responsible for? Успех тима често зависи од квалитета кода који га доводи у производњу и колико брзо се критична питања откривају и решавају. Постављени проблем је решен, решење је чисто и тестирано, и Код је ефикасан и комплетан. Често, решење ће изгледати веома другачије од тога како бисте га решили, и то је у реду.. То није релевантно за преглед. Још једна велика предност прегледа је пренос знања.Свако решава проблеме другачије, а преглед кода је прилика да тим научи једни од других.Приступ прегледи као прилика да проширите своју перспективу, а не диктирају решења. Дакле, хајде да потопимо мало више и разговарамо о томе зашто то радите. Is This Done? Da li je ovo učinjeno? Сви смо били тамо: почињете да решавате проблем, откријете кул тангенцијалну идеју, и одједном градите нешто потпуно другачије. Дођете до краја и узбуђено држите своју идеју за све да видите, само да схватите да нисте заправо решили проблем који сте поставили да решите. Преглед ПР-а може нам помоћи да избегнемо ове мале погрешне смернице од увођења у производњу и узроковања невоља купаца. Прегледник треба да се фокусира на проблем на руци. Имајте мало упознавања са картама повезаним са овим ПР-ом, прочитали сте РЕАДМЕ или коментаре о ПР-у, и имати солидно разумевање проблема који се решава. Желимо да будемо сигурни да је инжењер разумео проблем и решио прави проблем. Постоје грешке, и то је у реду. Housekeeping Домаћинство Већина нас ће писати код на итеративан начин, што значи да ми решавамо проблем инстинктивно, али затим преписујемо код неколико пута како бисмо га учинили што чистијим, јаснијим и ефикаснијим. Али понекад, не правимо баш толико итерација као што бисмо можда морали да дођемо до правог дна. Овде је тежак део: ваш посао није да процените да ли је решење апсолутно најчистије могуће, већ да ли је довољно чисто да се тестира и одржава. Преглед треба да се фокусира на то да ли је овај код у стању у којем ће бити лако тестирати и одржавати; то је то. Прочитајте код и замислите будуће грешке или промене које би се могле појавити, и колико ће бити тешко овај код да се дебугира или промени? Ако изгледа збуњујуће или компликовано, вероватно бисмо га требали поједноставити. Niste ovde da biste rešili problem za autora, već da biste se uverili da su sami rešili problem. saveti i trikovi pomažu nam da učimo i rastemo, pa je vaš savet dobrodošao, ali ne pretvarajte ga u problem blokiranja. Conclusion Закључак Прегледи кода нису о доказивању да сте најпаметнији инжењер у соби или присиљавању сваког решења да одговара вашем стилу. Ако приступите прегледима фокусирајући се на Код се постиже уместо Kada stignete tamo, pomoći ćete svom timu da se kreće brže, izgraditi poverenje i unaprediti saradnju. Šta Kako U sledećem delu ćemo ući u : принципи и праксе који чине преглед ефикасним, поштованим и продуктивним. јер чак и ако знате сврху, потребна је права вештина да бисте доставили повратне информације које подижу и код и кодер. Kako Оригинални пост: https://substack.com/@halexmorph
