Hejto.pl
Dodaj post

Wpisz coś do wyszukania (minimum 2 znaki)

Wpis użytkownika ZohanTSW w Hydepark

Fanatyk

w Hydepark

97piorunów

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.

Komentarze (20)

Sum1piorunów

Looks LGTM to me

Sum1piorunów

@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

Fanatyk0piorunów

@Bylina_Rdestu nie no 300 linii to elegancki kod, 3000 to LGTM

Kosmonauta0piorunów

Teraz mozna ai uzyc do code review 😉

Tytan3piorunów

@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ć :grinning:

Fanatyk1piorunów

@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

Tytan1piorunów

@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

Fanatyk1piorunów

@b0lec zdecydowanie, i to powinno wyłapać wszystkie pierdoły jako pierwsza faza review

Zawodowiec0piorunów

Ja jestem chętny, ale reszta zespołu już mniej. Najzabawniej jest jak kłócą się z dokumentacją xd

GURU1piorunów

LGTM

Kosmonauta1piorunów

@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 ) jest coraz mniej.

Kosmonauta1piorunów

@ZohanTSW Raczej, że nie ufamy, że jako ludzie jesteśmy nieomylni. Co do reszty, jak najbardziej.

Fanatyk3piorunów

@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

GURU6piorunów

@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ć 😆

GURU1piorunów

@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

Fanatyk1piorunów

@damw ale jak, że jedna iteracja? Znęcasz się nad całością, robią poprawki i potem znęcasz się nad poprawkami!

Lider1piorunów

@ZohanTSW teraz już jest nawet gorzej, bo się często reviewuje kod bota z pajacem po środku w formie głuchego telefonu.

Gruba ryba7piorunów

A tu się przypierdolę, nieprawda xD

Fanatyk6piorunów

@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