Hur man bäst utvärderar dragförfrågningar (utan styrande lösningar) I teknik arbetar vi alltid mot att skriva bättre, starkare, mer underhållbar kod. Kodrecensioner är en viktig del av den processen. Ändå är det en färdighet som vi sällan fokuserar på att förbättra. Att bli en kvalitet granskare kan göra dig till en riktig tillgång för ditt team, men det är inte lätt. Många av oss närmar sig kod recensioner från fel vinkel eller med missledda syften. I den kan omvandla ditt lags dynamik och sätta dig på en väg till framgång. Rätt saker Rätt sätt The Purpose of a PR Review Syftet med en PR-översikt Som ingenjörer är vårt primära jobb att skriva kod. För att lösa problem. För att hålla världen snurrar en bit eller byte i taget. Men vad är vår roll när man granskar kod? Den frågan kan vara skrämmande, särskilt första gången du ombeds granska någon annans kod. So, let’s start with what you’re not expected to do: Du förväntas inte lösa problemet för författaren. Du förväntas inte skriva om deras kod för att matcha din egen vision. So, what are you responsible for? Ett lags framgång beror ofta på kvaliteten på koden som gör det till produktion och hur snabbt kritiska problem upptäcks och löses. Problemet som uppkommit är löst, lösningen är ren och testbar, och Koden är effektiv och komplett. Ofta kommer lösningen att se väldigt annorlunda ut än hur du skulle ha löst det, och det är bra.. Det är inte relevant för granskningen. En annan stor fördel med recensioner är kunskapsöverföring. Alla löser problem annorlunda, och granskning av kod är en möjlighet för teamet att lära av varandra. Så låt oss dyka in lite mer och prata om varför du gör detta. Is This Done? Är detta gjort? Vi har alla varit där: du börjar lösa ett problem, upptäcker en cool tangentiell idé, och plötsligt bygger du något helt annorlunda. du kommer till slutet och du håller upp din idé för alla att se, bara för att inse att du inte faktiskt löste problemet du satte upp för att lösa. du byggde något coolt och fantastiskt, men det löser inte det avsedda syftet. PR-granskningen kan hjälpa oss att undvika dessa små felaktigheter från att göra det till produktion och orsaka kundkänslor.Granskaren bör fokusera på problemet vid hand. Ha en liten bekantskap med biljetterna i samband med denna PR, ha läst README eller kommentarer på PR, och ha en solid förståelse för problemet som löses.Vi vill vara säkra på att ingenjören förstod problemet också och tog itu med rätt problem.Fel händer, och det är okej.Vi behöver bara rätta dem när det är möjligt. Housekeeping Hushållning De flesta av oss kommer att skriva kod på ett iterativt sätt, vilket innebär att vi löser problemet instinktivt men sedan skriva om koden flera gånger för att göra det så rent, tydligt och effektivt som möjligt. Men ibland gör vi inte lika många iterationer som vi kan behöva komma till den verkliga bottenlinjen. Här är den svåra delen: ditt jobb är inte att döma om lösningen är den absolut renaste möjliga, men om den är tillräckligt ren för att vara testbar och underhållbar. En översyn bör fokusera på om denna kod är i ett tillstånd där det kommer att vara lätt att testa och underhålla; det är det. Läs koden och förutse framtida buggar eller förändringar som kan uppstå, och hur svårt kommer denna kod att debugga eller ändra? Om det verkar förvirrande eller komplicerat, bör vi förmodligen förenkla det. Du är inte här för att lösa problemet för författaren, men för att se till att de löste problemet själva. tips och tricks hjälper oss att lära och växa, så ditt råd är välkommet, men gör det inte till ett blockerande problem. Conclusion Slutsats Code recensioner handlar inte om att bevisa att du är den smartaste ingenjören i rummet eller tvinga varje lösning att matcha din stil. Om du närmar dig recensioner genom att fokusera på Koden uppnår i stället för När du kommer dit hjälper du ditt team att flytta snabbare, bygga förtroende och främja samarbete. Vad Hur I nästa del kommer vi att gräva in i De principer och metoder som gör en granskning effektiv, respektfull och produktiv eftersom även om du känner till syftet, krävs det verklig skicklighet för att leverera feedback som höjer både koden och kodaren. Hur Ursprungligen publicerad : https://substack.com/@halexmorph
