Ludzie generalnie lubią się przypierdalać. Chodzą i szukają problemu u innych, tam gdzie go nie ma, byle pokazać swoją mądrość i skorygować drugiego człowieka.
Chyba że chodzi o code review, to wtedy nie ma chętnych.
#przemyslenia #it #programowanie #heheszki #programista15k
A tu się przypierdolę, nieprawda xD
@serel no bo w zespole znajdzie się taki ktoś jak ja, co chętnie się poznęca, ale kiedy ja już proszę kogoś o review to nie ma mądrych
@ZohanTSW teraz już jest nawet gorzej, bo się często reviewuje kod bota z pajacem po środku w formie głuchego telefonu.
@ZohanTSW ja tam lubiłem w poprzedniej robocie robić. Siadałem, łapałem szybko jakiś poważny fuckup, dawałem blocka na PR i komentarz: to wy⁎⁎⁎ie ten i ten microservice przy takich danych. A jak poprawił i znalazłem znów chwilę czasu, to szukałem czegoś kolejnego xD
Te bardziej kumate devy od czasu do czasu dobrego browara przynosili i im robiłem całe review naraz, więc tylko raz musieli poprawiać
@damw ale jak, że jedna iteracja? Znęcasz się nad całością, robią poprawki i potem znęcasz się nad poprawkami!
@ZohanTSW ale ja nie powiedziałem, że nie znęcałem się nad kolejną iteracją. Ja znęcałem się na "raty" nad pierwszą wersją, a potem na raty nad kolejnymi. A osoby które podrzucały browarki miały po prostu mniej tych iteracji do robienia xD
@ZohanTSW CR zakłada dobre intencje i brak własnego interesu. Zakłada też, że ludzie z zespołu się lubią i nie przypierdalaja się o małe rzeczy i dążą do ulepszenia kodu.
Coraz tego mniejbtych dobrych intencji, a kodu takiego co można go pooglądać w takim zamkniętym pomieszczeniu ( porno room programistyczny z pierwiastkiem z Quake #kmwtw ) jest coraz mniej.
@dziad_saksonski CR zakłada że sobie nie ufamy wzajemnie, a nawet sobie samemu, dlatego chcemy żeby ktoś inny zerknął na ten kod i sprawdził czy to wszystko się trzyma kupy, czy nie ma jakichś błędów i czy można coś zrobić lepiej. Możemy się lubić, ale nie można dopuścić do sytuacji że "tego lubię, to mu odpuszczę". Oczywiście nie powinniśmy przeciągać w drugą stronę, że ktoś się uprze na jakąś pierdołę i nie przepuści tego. Jeśli w ogóle takie sytuacje się zdarzają, to coś poważnie nie gra. Każdy powinien wypychać taki kod, w którym można się skupić na logice, a nie przypierdolkach typu dodaj pustą linię, a tę zmienną camelCase'm
@ZohanTSW Raczej, że nie ufamy, że jako ludzie jesteśmy nieomylni. Co do reszty, jak najbardziej.
Ja jestem chętny, ale reszta zespołu już mniej. Najzabawniej jest jak kłócą się z dokumentacją xd
Teraz mozna ai uzyc do code review 😉
@b0lec zdecydowanie, i to powinno wyłapać wszystkie pierdoły jako pierwsza faza review
@ZohanTSW i ktos uzył AI do pisania tego kodu, juz chyba mało co się pisze samemu, albo piszesz sam pozniej po ai poprawiasz xd
@konrad1 w C++ jeszcze chyba sporo, bo szczerze mówiąc AI jeszcze ssie pałę w tym xd no chyba że do testów, to jest nawet ok, chociaż i tak potrafi wygenerować test zawierający assert(true) jako jedyny check
@ZohanTSW okk w Java wygląda to dość spoko zazwyczaj:P i tak dziwne czasy ogólnie, nie mówię że złe ale ludzie którzy nauczą się pisać tylko kod od AI, nie wiedząc samemu czy on jest dobry? seniorów mało, oby zdążyli wszystkich wyszkolić
@ZohanTSW to zależy, jak jest zmienionych 10-15 linijek to wtedy elegancko przynajmniej 7,8 komentarzy. Jak linii jest 300 to LGTM i elo
@Bylina_Rdestu nie no 300 linii to elegancki kod, 3000 to LGTM
Looks LGTM to me
@ZohanTSW lgtm
Zaloguj się aby komentować
