Pomówiony o mobbing za... code review

O IT mówi się różne rzeczy. Głównie w kontekście bardzo wysokich zarobków. Mało kto wie jak wygląda praca w takim środowisku i z jakimi problemami trzeba się mierzyć. Po co robić CR jak kod działa? A że wygląda ch***wo, to już inna sprawa ¯\_(ツ)_/¯

[budyn]

Az mi się przypomniała moja historia.
https://www.wykop.pl/wpis/41202677/programowanie-programista15k-zalesie-procbaza-prob/

Mam taką jedną programistkę, która od 3 miesięcy walczy nad jednym (dosyć skomplikowanym) featurem.
Najgorsze już za nią. Daje PR kod do code-review.
Patrzę na szybko, kilkanaście komentarzy z zakomentowanym kodem, debug statements itd.

[IvanBarazniew]

@budyn: Ja kojarzę inną tragikomedię. Duży projekt, dość skomplikowany, kluczowa ma być wydajność. Kod związany z obsługą warstwy danych pisany przez hinduskiego speca w USA - tajne przez poufne - nikt nie może zahaczyć zanim nie będzie gotowe. Po 8 miesiącach (sic!) kod trafia do review w polskim oddziale. Po dwóch godzinach (pierwsza iteracja) 60 krytycznych komentarzy. Tydzień później zamknięcie polskiego oddziału bo się nie potrafi dogadać z ekspertem z

[budyn]

@IvanBarazniew: To nie jest w ogóle śmieszne.
Od tego miesiąca, "na próbę" (na szczęście), dokoptowali nam team hindusów.
Jest źle. Mam nadzieję, że do końca roku już ich nie będzie, bo się nie zapowiada na dodatni "zwrot" z tej "inwestycji"

[Helix]

Coś w tym jest ( ͡° ͜ʖ ͡°)

[a.....e]

@Helix: "I will fight for you" ale tylko przez 1 rok, bo potem odejdę do innej firmy, gdzie nie będą mi żałować podwyżki x2

[eloar]

@aczutuse: e tam. miałem ze 2x tak, że taki junior potem się kontaktuje z pytaniem, czy bym nie chciał też jakiej fajnej podwyżki, bo jego nowa firma szuka ludzi i dobrze płaci i mają w owocowe czwartki nie tylko jabłka, ale nawet mandarynki, także poza sezonem.

[testuser]

Pamiętam swoje początki jako junior od automatów - zaprzęgnięcie Selenium od 0, POP, te sprawy. Natrudziłem się, tyle dobrze, że było sporo dokumentacji wewnętrznej, bo mieliśmy to na poziomie firmy ogarnięte. PR - jeden z seniorów wbija spotkanko pt. "code-review" na 2h - oho, "będzie zabawnie" myślę. Spotkanie w salce, mój kod na TV, przechodzimy linijka po linijce. Konkluzje:
- było dobrze, nawet bardzo dobrze, jak na juniora, który dostał apkę i

[thund3rb0lt]

Dobre code-review to takie

@testuser: w ktorym oceniasz kod a nie tego kto go napisal. Niestety wiekszosc ludzi nie potrafi rozmawiac o nim nie uzywajac fraz w stylu "musisz to zrobic tak...", albo "nie mozesz robic tak i tak...". Bledem jest pozwalanie robienia code-review osobom, ktore nie wiedza jak to robic.

[5da4266d3de6dbaf425a2d4fc16225d0]

@thund3rb0lt: Wydaje mi się, że częściej to autor kodu ma problem żeby odciąć się od niego "personalnie" i jakakolwiek sugestia jest odbierana jako "ktoś mi mówi że słabo programuje" ( ͡° ͜ʖ ͡°)

[kidziel]

Pamiętam jak przyszedłem do roboty jako junior i dostałem 200+ komentów na 20 linijek kodu. Piękne czasy.

[lovelypl]

@paliwoda: ponad 200, nieuku.

[unintelligible]

>200

@paliwoda: o nie wierzę, jaki błąd u paliwody :D przy okazji warto wspomnieć, że w kontekście programowania "200+" jest w 100% poprawne, bo jest to branżowy zapis

[Bartholomew]

@kwanty: Tak uważają ci, którzy z projektowaniem grafiki nie mają bezpośredniego styku. AI może pomóc grafikom a nie ludziom, którzy potrzebują strony internetowej czy wizytówki do druku.

[kwanty]

Problem polega na tym ze miedzy "zrobic ulotke" a zrobic "zrobic dobra, przejzysta ulotke ktore nie wyglada jak informacyjna i wizualna sraka" to jest roznica. Pierwsza ci AI zrobi - do drugiej jest juz potrzebny czlowiek ktory to ogarnia :P

@10minuteman: Ty jeszcze nie jesteś świadomy, że AI zrobi tą ulotkę lepiej niż człowiek bo model przyswoił (nauczył się) wygląd wszystkich ulotek jakie wszyscy graficy zrobił do tej pory, rozróżnia style,

[odomdaphne5113]

Pracowałem z laską co każdą uwagę pod przełożonych traktowała jak mobbing.

[TheDzions]

@odomdaphne5113: bo nie wolno kobiecie w IT zwracać uwagi, ona jest tylko po to żeby była

[masiakla]

@TheDzions: to nie jest tylko problem it, płeć też nie ma z tym wiele wspólnego, chociaż częściej to można spotkać z kobiecej strony niestety. Przez ostatnie kilka lat jest wzrost delikutaśntch osób, które myślą, że w pracy będzie jak na fb, same like'i.

[sukienka]

@DooBeeDooBee: bardzo sensowny wpis.

[mala_rybka]

@DooBeeDooBee: to zadziała tylko o ile sprawdzasz pracę kogoś kto faktycznie chce swoją pracę zrobić dobrze. Używałam Twojej metody (nie przy CR ale blisko). No i co z tego wyszło?

Dlaczego zdecydowałeś się użyć X a nie Y? Co sądzisz o tym, aby (...).

Odpowiedzi "nie wiem" albo "bo tak" to norma. Fochy że mam powiedzieć krok po kroku co zrobić a nie oczekiwać myślenia też się

[Pan_dziwny]

To działa w 2 strony.

Kiedyś miałem zmianę. 2 miesiące walki i ustaleń z architektami i ludźmi od kilku komponentów ( temat trudny, wiele zmian procesach biznesowych - mieliśmy nad czym siedzieć i ustalać). Wszystko po ustaleniach zaimplementowane. CR i co? Cyrk.

Koleś zaczął od sproszenia chyba z 10 osób. 9 z nich nawet nie wiedziało co recenzuje, ale musieli się wypowiedzieć. Komponent dostali w spadku po innym zespole napisany w dość

[Stivo75]

@Pan_dziwny: ale wiesz że negocjowanie nazw metod to nie pierdoła?
Bo jak nazwiesz metodę która pobiera dane? no tak doGetData a co ktoś zwróci uwagę że powinno być skąd? Powinno w nazwie być w czym ma zwrócić? A może json? A może to i tamto?
Pewnie mieli metody w stylu doGetDataFromApiByIdReturnedInPrettyJson - no i to jest też zgodne ze standardami ( ͡° ͜ʖ ͡°)

[y.....l]

Historia znana mi z doświadczenia.

Kiedyś byłem sobie takim junior klepaczem C++ i robiłem akurat w działce, która przypadała panu wielkiemu PRINCIPAL inżynmierowi z ogromnym doświadczeniem i mega wiedzą. Niestety każda moja zmiana musiała przejść przez jego ręce.
Komentarze pod kodem w stylu:

- back to primary school

[Laktobakteria]

@yggdrasil: często jest tak, że taka wielka wiedza - to nic innego, że to jest wiedza produktu jak takiego, a nie umiejętności programować. Jeżeli jest takim wybitnym fachowcem, to dlaczego nie zgarnia kupy hajsu w FAANG?

[y.....l]

@Laktobakteria: Nie rozumiem pytania. :P Co to znaczy posadzenie na PIP-a?

[kill15you]

Jak tacy ludzie się uczą czegokolwiek skoro nie są gotowi na podjęcie dyskusji... patrzysz potem na kod typa 15 lat doświadczenia i się zastanawiasz co się w ogóle dzieje... jednocześnie gość pewnie zarabia więcej z racji tego, że ma 15 lat expa a ty 5 xD

[tellet]

"bo SAP tak rekomenduje"

@sosnajabadaba: ale w każdej większej firmie nie możesz zrobić czegoś, czego nie zaleca dostawca oprogramowania/framerworka/czegokolwiek, bo w razie wyjebki tego rozwiązania, idziesz do gazu bo zrobiłeś coś niezgodnego z zaleceniami.
Jak jeszcze dochodzi jakaś gwarancja czy inne wsparcie na jakiś program, to już w ogóle bez manuala nie zaczynasz pracy.