Błędy code review ⌨️ hello roman

[@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 😉

[@helloroman]

Bardzo słuszna uwaga! Sam staram się tak pisać komentarze i faktycznie są wtedy dużo lepiej odbierane.

[@malipek]

Kolejny props za security: code review testów.

[@MrLucki001]

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-od4iz]

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

[@michalbacinski6700]

fajnie byłoby popisać takie komentarze :P ale w moim zespole to ten kij aż o sufit stuka

[@adamsiekierski3133]

LGTM 🔥

[@bachi4855]

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

[@bartek2886]

Będzie coś o testach aplikacji?

[@helloroman]

Będzie w kursie o React, który pojawia się dla wspierających kanał :)

[@heisenbergnet5476]

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 ;(

[@helloroman]

Nie wydaje mi sie, zebym tutaj jakichs zaawansowanych terminów używał 🤔 task? Chyba najczęściej się przewijał w tym odcinku.

[@heisenbergnet5476]

@@helloroman cytująć 'niedługo prościej będzie zrozumieć kod niż programistów' miałem na myśli, że technologie idą tak do przodu, że ciężko za tym nadążyć. Zależało mi w tym porównaniu na tym, aby pokazać, jak wielką widzą muszą posługiwać się programiści.

[@helloroman]

@@heisenbergnet5476 Już rozumiem 👍 dzięki

[@poelover2024]

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 😂 😂

[@helloroman]

Chyba musisz sobie lepiej poustawiać priorytety w życiu jeśli granie stawiasz na równi ze sposobem na życie (no chyba że mowimy o esporcie)

[@poelover2024]

@@helloroman nie no bez przesady, perły nad wieprze. Ale zdarza ci się grać?

[@helloroman]

No pewnie, w Ghost Recon Breakpoint cisnę namiętnie (ale mam regułę że maks 1h co dwa dni)

[@poelover2024]

@@helloroman w sumie ty jeszcze blogi robisz pewnie też czasu to kosztuje. Dzięki za odpowiedź 👍

[@aleksanderszut2098]

16:00 jak taki bot się nazywa ?

[@helloroman]

autorski projekt :) sami sobie napisaliśmy

[@skumpuntele8941]

ja w pracy uzywam tego slack.github.com/ i daje rade

[@koksikskkj7937]

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

[@rghbv]

już było na kanale o tym. "Kiedy zaczaąć się uczyć frameworków" czy jakoś tak się nazywa materiał. a frameworki są o tyle wygodne, że nie musisz pisać od zera wszystkiego. zamiast pisać kod do obsługi np. slidera na stronie korzystasz z już gotowego kodu. 1. nie poświęcasz czasu na wynajdywanie koła od nowa 2. najprawdopodobniej kod z frameworka będzie lepszy niż sam byś napisał(bo ktoś poświęcił na to znacznie więcej czasu niż Ty teraz masz) 3. jest zwyczajnie prościej. 4. Twój kod często dzięki temu jest o wiele czytelniejszy.

[@koksikskkj7937]

@@rghbv To jest prawda co napisałeś. Jestem też ciekaw co inni o tym uważają, jakie mają podejście co do tego. Właściwe to wszystko poruszyłeś co tu można powiedzieć. Dzięki wielkie za treściwą odpowiedź

[@rghbv]

@@koksikskkj7937 a miło mi :) polecam się na przyszłość.

[@barszczCzerwony666]

​@@koksikskkj7937 używając frameworków znacznie łatwiej jest wdrożyć nową osobę w projekt

[@krzysztof8429]

@@rghbv Dodałbym: 5. Zazwyczaj twoim celem nie jest wytworzenie samego kogo, aby po prostu był , masz jakieś zadanie które według zleceniodawcy ma przynieść mu zysk. Dzięki frameworkom ten zysk dostarczasz znacznie szybciej.

[@MoDrazzz]

LGTM :D

[@tomaszgasior772]

15:31 od tego przypadkiem nie są powiadomienia e-mail i webowe oraz strona github.com/pulls/review-requested ?

[@helloroman]

Są, ale jak się nie korzysta z maila w pracy tylko ze slacka, to lepiej mieć bota :)

[@bartek2886]

LGTM 😜🤗

[@kliMaLd]

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

[@helloroman]

Krzywdzące uproszczenie :)

[@kliMaLd]

@@helloroman ale prawdziwe :) chodzilo mi tu o to jak mowiles, ze ktos z zespolu mowil co to ma wspolnego z recenzja po prostu.

[@TheLoniard]

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

[@MrLucki001]

Pierwszy!!!

[@Bitekpierwszy]

Moja dziewczyna ogarnęła :D

[@FaktyNaSzybko]

pierszy

[@dziurappa]

.