Błędy code review ⌨️ hello roman

Поделиться
HTML-код
  • Опубликовано: 19 янв 2025

Комментарии • 41

  • @DziedziuchProgramuje
    @DziedziuchProgramuje 4 года назад +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  4 года назад +3

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

  • @MrLucki001
    @MrLucki001 4 года назад +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

  • @malipek
    @malipek 4 года назад +3

    Kolejny props za security: code review testów.

  • @Adam-od4iz
    @Adam-od4iz 4 года назад +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 4 года назад +2

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

  • @bachi4855
    @bachi4855 4 года назад +1

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

  • @bartek2886
    @bartek2886 4 года назад +7

    Będzie coś o testach aplikacji?

    • @helloroman
      @helloroman  4 года назад +2

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

  • @adamsiekierski3133
    @adamsiekierski3133 4 года назад +7

    LGTM 🔥

  • @heisenbergnet5476
    @heisenbergnet5476 4 года назад +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  4 года назад +1

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

    • @heisenbergnet5476
      @heisenbergnet5476 4 года назад +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  4 года назад

      @@heisenbergnet5476 Już rozumiem 👍 dzięki

  • @aleksanderszut2098
    @aleksanderszut2098 4 года назад

    16:00 jak taki bot się nazywa ?

    • @helloroman
      @helloroman  4 года назад +2

      autorski projekt :) sami sobie napisaliśmy

    • @skumpuntele8941
      @skumpuntele8941 4 года назад +1

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

  • @poelover2024
    @poelover2024 4 года назад

    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  4 года назад +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 4 года назад

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

    • @helloroman
      @helloroman  4 года назад

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

    • @poelover2024
      @poelover2024 4 года назад

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

  • @koksikskkj7937
    @koksikskkj7937 4 года назад +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 4 года назад +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 4 года назад

      @@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 4 года назад +1

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

    • @barszczCzerwony666
      @barszczCzerwony666 4 года назад

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

    • @krzysztof8429
      @krzysztof8429 4 года назад +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.

  • @tomaszgasior772
    @tomaszgasior772 4 года назад

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

    • @helloroman
      @helloroman  4 года назад +1

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

  • @kliMaLd
    @kliMaLd 4 года назад

    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  4 года назад

      Krzywdzące uproszczenie :)

    • @kliMaLd
      @kliMaLd 4 года назад

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

  • @TheLoniard
    @TheLoniard 4 года назад

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

  • @MoDrazzz
    @MoDrazzz 4 года назад +3

    LGTM :D

  • @bartek2886
    @bartek2886 4 года назад +1

    LGTM 😜🤗

  • @MrLucki001
    @MrLucki001 4 года назад

    Pierwszy!!!

  • @Bitekpierwszy
    @Bitekpierwszy 4 года назад

    Moja dziewczyna ogarnęła :D

  • @FaktyNaSzybko
    @FaktyNaSzybko 4 года назад +1

    pierszy

  • @dziurappa
    @dziurappa 4 года назад

    .