Błędy code review ⌨️ hello roman #146

Code review to jeden z najważniejszych procesów developmentu – nie tylko pozwala nam uniknąć poważnych błędów, ale przede wszystkim nauczyć się tony rzeczy, dzięki współpracy z innymi programistami! W tym odcinku wymieniam najczęstsze błędy popełniane przez programistów podczas code review.

⭐️ Wesprzyj kanał za 34,99 zł na miesiąc i uzyskaj dostęp do wszystkich moich kursów na kanale:

📌 Sprawdź też:

W moich filmach znajdziecie wiele praktycznych porad, jak zacząć swoją przygodę z programowaniem, na co uważać, jak motywować się do pracy, aby nie stracić zapału. Wspólnie odkrywać będziemy możliwości jakie kryją JavaScript, HTML i CSS – pomogę Wam także zrozumieć, jak skutecznie wykorzystywać potencjał popularnych frameworków, takich jak Vue lub React.

[DziedziuchProgramuje]

Dodam jeszcze od siebie, że inni ludzie zdecydowanie lepiej przyjmują nasze uwagi, jeśli ubierzemy je w formie pytań. Na przykład „A co sądzisz o tym, że można to zrobić tak....?” albo „czy nie uważasz że ten sposób jest czytelniejszy?” zamiast „popraw to ” albo „zrób to tak”, bo nie jesteśmy sędzią najwyższym który ma zawsze racje. Wyrażamy swoje zdanie, z którym ktoś może się zgodzić, ale nie musi 🙂 ogólnie bardzo fajny odcinek 😉

[malipek]

Kolejny props za security: code review testów.

[MrLucki]

Jednym z dosc ciekawych bledow podczas robienia codereview jest gdy ktos zamiast skupic sie na Twoim kodzie i starac sie go polepszyc swoim feedbackiem, ciagle na sile stara sie "wcisnac" swoje alternatywne rozwiazanie, ktore ani nie jest lepsze ani gorsze. Po prostu jest inne. Kiedys mielismy takiego kolege na projekcie co chyba kazdy CR zaczynal od zastanowienia sie jak przepisac ten kod na swoje. A pozniej sie obrazal, ze ludzie nie chca uwzgledniac jego pomyslow. Coz problemem bylo tu chyba wielkie ego kolegi. Ja jesli robie CR to zawsze staram nie hejtowac rozwiazan ktore sa dobre lub optymalne, a moze ja zrobilbym inaczej. Zawsze wazne jest moim zdaniem aby uwzgledniac AUTONOMIE innego programisty. A nie na zasadzie "Moja racja jest wieksza niz twoja" 😀 Pozdro

[Adam-odiz]

U nas z przymrużeniem oka - uwagi bez emotek, zawsze z kropką na końcu, pozytywy typu "podoba mi się to rozwiązanie ale w jquery napisałbym to lepiej" albo "kup mi obiad za approve" - oczywiście wszystko w granicach rozsądku i pod warunkiem że klient nie ma dostępu do repo, no i team nie może mieć kija w dupie :D

[bachi]

Odczuwacie może spadek wydajności po update do Big Sur? Można coś na to poradzić?

[heisenbergnet]

Nie jestem programistą, ale coś tam wiem, jak się kod pisze. Gdy słucham czasem Romana, to mam wrażenie, że niedługo prościej będzie zrozumieć kod niż programistów :p Oczywiście bez urazy, chodzi mi tylko o niektóre terminy, których używasz, a ich nie rozumiem. Smutno mi było jak nie zrozumiałem sucharka ;(

[koksikskkj]

Hello Roman! A może Adam? Ja mam pomysł na odcinek a mianowicie, czy używanie frameworków jest lepsze niż pisanie pure js? Sam od jakiegoś czasu nauczyłem się JS'a i rozglądałem się za frameworkami i czy one naprawdę są lepsze niż pure JS czy to tylko rozwiązanie bardziej biznesowe? Może dość ogólny pomysł, ale jak masz jakeś pomysły zestawów dla juniora czy coś to możesz to poruszyć :D Innych też zachęcam do komentarzu

[poelover]

Romek a jak się ma być programista do być graczem, da się pogodzić np granie w jakieś mmo wymagające czasu z pracą jako programista? Ja nawet chętnie odcinek o tym zobaczę jeśli w coś grasz to w co 😂 😂

[kliMaLd]

Code review - czyli przeglad kodu w zamysle: czy i gdzie i jakie sa w nim bledy, nie ma to nic wspolnego z "recenzja".

[TheLoniard]

15:09 czoło do memów pt: "code after CR"