Błędy code review ⌨️ hello roman

แชร์
ฝัง
  • เผยแพร่เมื่อ 14 พ.ย. 2024

ความคิดเห็น •

  • @DziedziuchProgramuje
    @DziedziuchProgramuje 3 ปีที่แล้ว +33

    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
      @helloroman  3 ปีที่แล้ว +3

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

  • @malipek
    @malipek 3 ปีที่แล้ว +3

    Kolejny props za security: code review testów.

  • @MrLucki001
    @MrLucki001 3 ปีที่แล้ว +9

    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
    @Adam-od4iz 3 ปีที่แล้ว +2

    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
      @michalbacinski6700 3 ปีที่แล้ว +2

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

  • @adamsiekierski3133
    @adamsiekierski3133 3 ปีที่แล้ว +7

    LGTM 🔥

  • @bachi4855
    @bachi4855 3 ปีที่แล้ว +1

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

  • @bartek2886
    @bartek2886 3 ปีที่แล้ว +7

    Będzie coś o testach aplikacji?

    • @helloroman
      @helloroman  3 ปีที่แล้ว +2

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

  • @heisenbergnet5476
    @heisenbergnet5476 3 ปีที่แล้ว +1

    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
      @helloroman  3 ปีที่แล้ว +1

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

    • @heisenbergnet5476
      @heisenbergnet5476 3 ปีที่แล้ว +1

      @@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
      @helloroman  3 ปีที่แล้ว

      @@heisenbergnet5476 Już rozumiem 👍 dzięki

  • @MoDrazzz
    @MoDrazzz 3 ปีที่แล้ว +3

    LGTM :D

  • @koksikskkj7937
    @koksikskkj7937 3 ปีที่แล้ว +1

    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
      @rghbv 3 ปีที่แล้ว +2

      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
      @koksikskkj7937 3 ปีที่แล้ว

      @@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
      @rghbv 3 ปีที่แล้ว +1

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

    • @barszczCzerwony666
      @barszczCzerwony666 3 ปีที่แล้ว

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

    • @krzysztof8429
      @krzysztof8429 3 ปีที่แล้ว +1

      @@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.

  • @poelover2024
    @poelover2024 3 ปีที่แล้ว

    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
      @helloroman  3 ปีที่แล้ว +1

      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
      @poelover2024 3 ปีที่แล้ว

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

    • @helloroman
      @helloroman  3 ปีที่แล้ว

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

    • @poelover2024
      @poelover2024 3 ปีที่แล้ว

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

  • @kliMaLd
    @kliMaLd 3 ปีที่แล้ว

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

    • @helloroman
      @helloroman  3 ปีที่แล้ว

      Krzywdzące uproszczenie :)

    • @kliMaLd
      @kliMaLd 3 ปีที่แล้ว

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

  • @bartek2886
    @bartek2886 3 ปีที่แล้ว +1

    LGTM 😜🤗

  • @aleksanderszut2098
    @aleksanderszut2098 3 ปีที่แล้ว

    16:00 jak taki bot się nazywa ?

    • @helloroman
      @helloroman  3 ปีที่แล้ว +2

      autorski projekt :) sami sobie napisaliśmy

    • @skumpuntele8941
      @skumpuntele8941 3 ปีที่แล้ว +1

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

  • @TheLoniard
    @TheLoniard 3 ปีที่แล้ว

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

  • @tomaszgasior772
    @tomaszgasior772 3 ปีที่แล้ว

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

    • @helloroman
      @helloroman  3 ปีที่แล้ว +1

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

  • @Bitekpierwszy
    @Bitekpierwszy 3 ปีที่แล้ว

    Moja dziewczyna ogarnęła :D

  • @FaktyNaSzybko
    @FaktyNaSzybko 3 ปีที่แล้ว +1

    pierszy

  • @MrLucki001
    @MrLucki001 3 ปีที่แล้ว

    Pierwszy!!!

  • @dziurappa
    @dziurappa 3 ปีที่แล้ว

    .