[niedorzecznybubr]

Wrzucam jeszcze raz bo zjadłem słowo "nie"

#programista15k #programowanie #pracait #pracbaza #korposwiat

[W.....n]

@niedorzecznybubr to wy w ogóle czytacie kod?

[pyszniarz]

@niedorzecznybubr: jprld, za mało sobie liczę....

[magic_johnson]

@pyszniarz: więcej ci nie zapłacą za te wprawki dla juniorów

[pyszniarz]

@magic_johnson: co to są "wprawki dla junirów" xD

[mag_zbc]

@niedorzecznybubr: jeśli chodzi o code review, to jestem zagorzałym zwolennikiem filozofii "niech ktoś"

[Romska_Palo_Ul_Laputa]

@niedorzecznybubr: jak nie rozumiem, to sprawdzam o co chodzi w Internecie i się douczam albo pytam autora. Mogę się dowiedzieć czegoś ciekawego albo przyłapać lesera na ściąganiu z neta bez zrozumienia.

[niedorzecznybubr]

@Romska_Palo_Ul_Laputa: akurat bardziej miałem na myśli, że nie rozumiesz co to w sumie robi w sensie logiki

[RapIArbuzy]

@niedorzecznybubr: 80 procent nie, czasem nie chce mi się zagłębiać w kod i patrzę tylko na code quality

[W.....n]

@pyszniarz za dużo sobie liczysz, marnować czas na czytanie czyjegoś kodu to każdy by chciał, a robić nie ma komu, kod ma działać a nie wyglądać, a jak ktoś źle napisał to niech sam sobie poprawia

[pyszniarz]

@WpiszSwojLogin: ( ͡°( ͡° ͜ʖ( ͡° ͜ʖ ͡°)ʖ ͡°) ͡°)
To jak w tym kawale z robotnikiem na budowie z taczkami

[OlekAleksander]

akurat bardziej miałem na myśli, że nie rozumiesz co to w sumie robi w sensie logiki

@niedorzecznybubr: W takim sensie to nie zapoznaję się z całą logiką funkcjonalności, QA sprawdza, że to działa, ja patrze tylko na kod.

[niedorzecznybubr]

@OlekAleksander: a, no ja właśnie też tak podchodzę, tzn. ufam że albo autor wie co robi, albo się w--------i na testach xd. I tylko się czepiam rzeczy bardziej kosmetycznych. Chyba że zauważę że z logiką jest coś nie tak

[peoplearestrange]

@niedorzecznybubr: którego nie czytam* poprawilem za ciebie ( ͡° ͜ʖ ͡°)

[krunky]

@niedorzecznybubr: Jak dużo linijek to LGTM i approve ( ͡° ͜ʖ ͡°)

[Zlmmerman]

@niedorzecznybubr: Zawsze zaczynam od testów. U nas zawsze pisze się UT & SCT, a pokrycie musi mieć +90%. Jak zrobie CR testów to potem implementacje idzie łatwo ogarnąć. Problem jest jak ktoś n--------i kolejek w c++ przy asynchronicznym odbieraniu msg restowych itd i potem jej nie zwolni. Ciężko to wyłapać jak nie ma dobrych testów.

[zibizz1]

@niedorzecznybubr: Żeby zrozumieć to trzeba otworzyć Taska, przeczytać zrozumieć o co chodzi, a potem patrzeć na kod i patrzeć czy robi to co było trzeba zrobić. Jak robie review nowym albo tym co odstaja jakością pracy to tak robię, gdy natomiast oceniam kolehe z którym robimy już z 1-2 lata i wiem że robi dobrze to tylko przeglądam czy z grubsza zrobił podejście dobre bo może warto było dane zapisac