[Reevo]

Dostałem Code Review, robione przez zewnętrzną firmę, z wyszczególnionymi ~20 błędami. Moim zdaniem tylko 2 były sensowne, a reszta wręcz szkodliwa.

Próbowałem się do gościa dobić i przegadać dlaczego uważa tak, a nie inaczej, ale nie miałem okazji. Raz udał mi się wysłać mu wiadomość na discordzie, ale odpowiedział tylko że 'nie pamięta już szczegółów'.

Po 6 miesiącach nadal mnie to swędzi i czuję krytykę z tyłu głowy jak pracuję. Chyba z 50 razy już analizowałem tą listę wzdłuż i wszerz, kupiłem 3 książki na ten temat, a nadal nie jestem pewien który z nas jest upośledzony

Wiem gdzie typ pracuję, myślałem żeby wysłać mu maila z zaproszeniem do przegadania tego, ale nie chcę wyjść na świra.

Kilka dni temu wdrażałem dwa nowe pokemony do projektu (2 regularów) i dałem im im listę tych uwag w formie 'rekrutacji', żeby zobaczyć co by zrobili na moim miejscu.

Odpowiedzi były raczej w moją stronę, ale to jeszcze świeżaki, więc tym bardziej nie wiem czy to dobrze.

Panie doktorze, ego mnie boli (╥﹏╥)

#programowanie #programista15k

[WypadlemZKajaka]

@Reevo: Nie bierz tego tak do siebie

[L24D]

@Reevo: Jeżeli nic nie dociera do gościa to zwyczajnie bym zaimplementował to pod jego uwagi, jakby się z-----o ponownie, albo byłby problem przy rozwijaniu aplikacji to miałbym podkładkę, że pisałem i nie uzyskałem odpowiedzi.

[uzytkownik_wykop_pl]

@Reevo: trzymaj sie miras, olej te code review

[keton22]

z wyszczególnionymi ~20 błędami

@Reevo: zapłacili mu to znalazł. "Wszystko jest ok ale jak czegoś nie wpiszę to stracę reputację na mieście" tak mi kiedyś powiedziano. Zrozum chłopa, on też kredyt, żona i dzieci.

[Z.....a]

nie chcę wyjść na świra.

@Reevo: za pozno, trzeba było nie pisać tego wpisu ( ͡° ͜ʖ ͡°)

[Reevo]

@Reevo: to zasadz się z 2 świeżakami i mordę obij ( ͡° ͜ʖ ͡°)

@lexico: Świeżaki dobre goście. Po tygodniu zrobiliśmy układ że co tydzień ktoś inny niańczy PMa i ogarnia pożary. Oficjalnie żeby zachować corssdyscyplinarność zespołu. W praktyce, jak pozostali mogą pracować w spokoju, to robimy 2 tygodniowy sprint w 3-4 dni, a potem czas na CSa... znaczy refactoring, estymacji i prace nad

[andrew1985]

"z wyszczególnionymi ~20 błędami" - to, że ktoś cos skomentował to jeszcze nie oznacza, że to jest kompletnie źle, to błąd itp

[Mathouse88]

a nadal nie jestem pewien który z nas jest upośledzony

@Reevo: Ty bo myślisz o czymś sprzed pół roku ( ͡° ͜ʖ ͡°)

[kaczoor]

@Reevo No ale poprawiłeś te błędy i zmergowales? Bo jak tak, to przyklepales rację tego gościa i historia skończona ( ͡º ͜ʖ͡º)

[lajdak]

@Reevo: jakbyś w przyszłości chciał mieć różne "skórki" to moze mógłbyś sobie tak zmienić a tak to takie obiektowe p---------e XD

[a5f5c1]

@Reevo: miej to gdzieś, tyle w temacie ( ͡° ͜ʖ ͡°)

[g.....k]

@Reevo: bo to był słaby review na zasadzie "bo tak" bez żadnych argumentów. Na tym przykładzie AppColors, pobawię się w adwokata diabła:

Gdy PM będzie chciał aby aplikacja była dostępna w dwóch wersjach light i dark to dzięki wstrzykiwaniu AppColors będziesz mógł sterować kolorami całej aplikacji jedynie podmieniając beana podczas uruchomienia.

[krzych0]

@Reevo: znajdź kogoś najlepiej bardziej doświadczonego od ciebie i przegadaj z nim to. Jeżeli czujesz pod skóra ze krytyka była niezasadna jest duża szansa ze faktycznie tak jest ( ͡° ͜ʖ ͡°)

Nie szukaj tylko potwierdzenia swoich domysłów a raczej szukaj kogoś kto spojrzy na ten problem obiektywnie.

[Krolik]

@gos4k: YAGNI, nie powinno się robić rzeczy na zapas jeśli nie ma ich w wymaganiach, bo inaczej z kodu robi się FizzBuzz Enterprise.

[zwier]

@Reevo: nie myslales moze ze jestes po prostu cienki albo masz wybujałe ego? Troche krytyki i musiales taki wpis wyskrobac

[g.....k]

@Krolik: OCP , kod powinien być zamknięty na edycję ale otwarty na rozszerzalność. Żeby teraz dodać ten feature to będziesz musiał przeorać kod. Możemy się tak przerzucać mądrymi zasadami ale to nie ma sensu :P zrobiłbym tak jak Ty ale jest argument za podejściem z interfejsem, zależy w którą stronę aplikacja ma się rozwijać

[ArcadiusK]

@Reevo: zawsze możesz wrzucić na wykop i poddać pod dyskusję te wątpliwe przykłady. Albo na stackoverflow.