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 ¯\_(ツ)_/¯
shpyo z- #
- #
- #
- #
- #
- 420
Komentarze (420)
najlepsze
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.
Dalej nie patrzyłem i piszę
Po przeczytaniu z płaczem wyszła z biura.
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"
- było dobrze, nawet bardzo dobrze, jak na juniora, który dostał apkę i
@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.
@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
Programiści bywają przynajmniej pragmatyczni a te "artystyczne" cioty ciągle uważają, że każda uwaga w ich stronę to jak naszczanie na ich cenne płótna. Sami za to tylko czekają aby mieć okazję obrzygać kogoś (nie daj boże żeby pytał ich o zdanie na temat swojej pracy).
Śmieszny przykład: na FB jest taka jedna grupa dla grafików i tam mod rzuca co
@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,
Odpowiedzi "nie wiem" albo "bo tak" to norma. Fochy że mam powiedzieć krok po kroku co zrobić a nie oczekiwać myślenia też się zdarzają.
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ść
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 ( ͡° ͜ʖ ͡°)
@dybligliniaczek: Co do ch*ja XD Nie chciałbym pracować w takiej firmie, gdzie ktoś sobie decyduje, że akurat wystawi PR.
Testy też piszesz od widzimisie? No chyba tak skoro cała aplikacją klękła. Widzisz, doszukujesz się problemu u innych - a problem jest po Twojej stronie. Obrażasz się na CR, nie obsłużysz zmiany porządnie testami - a potem.. ja
Kiedyś byłem sobie takim junior klepaczem C++ i robiłem akurat w działce, która przypadała panu wielkiemu #!$%@? 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
- no
- wrong
itp.
No i jako, że byłem juniorkiem bez doświadczenia to do niego idę i pytam się o co chodzi, bo
Ex zależy w dużej mierze od lat przepracowanych - a to poprzez uczenie się na błędach/uczenie się w sytuacjach kryzysowych. Po dwóch latach nawet w najlepszym projekcie gówno wiesz i jesteś smarkiem.
@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.