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)

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

Zawodowiec0piorunów

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

Osobistość1piorunó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.

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