Krytyka kodu początkującego a odpowiedź autora

Odpowiedz Nowy wątek
2019-08-08 14:12
0

Hej, na jednej z grup fb'owych ktoś poprosił o ocenę projektów javowych na gicie (nie podam linka, nie wiem czy autor by sobie tego tutaj życzył).
W skrócie:

  • 4 projekty (każdy to java fx)
  • 3 projekty 5 klas, 1 projekt 9 klas
  • pod każdym widnieje info, że jest w trakcie kodowania, więc dlatego tak mało funkcjonalności
  • po kilkadziesiąt linii w plikach, tylko w jednym projekcie użyta baza sql (jakość masakra), w reszcie baza jako plik txt/xml

W przerwie od kodowania, w której spędzam czas na yt/fb/innych głupotach postanowiłem zajrzeć w kod na kilka minut i coś napisać.

Moje komentarze:

Szczerze to nic tam nie masz. Napisałeś "mam za to kilka własnych projektów hobbystycznych" - gdzie?
Przecież pod każdym widnieje "under deployment" czyli zaden nie jest skończony (nie jest nawet na przyzwoitym poziomie co do wielkości).
Każdy z nich ma max 10 klas javowych (połowa to settery i gettery).
Brak jakiejkolwiek bardziej skomplikowanej logiki.

  1. Brak loggerów - nie używaj sys outów
  2. Zapoznaj się z mavenem/gradlem do budowania projektu
  3. Podepnij baze danych do apki - plik txt jako baza to szczyt twoich możliwości?
  4. Wywal projekt ToDoList - robią to praktycznie wszyscy na starcie, a Ty chcesz się przecież czymś wyróżnić.

Zapewne znalazło by się więcej rzeczy ale to tak na szybko.

Reasumując:

  • na juniora zbyt wielkich szans na obecnym rynku raczej nie masz
  • za małe projekty, zrób jeden większy, dodaj jakąś logikę bardziej skomplikowaną niż wrzucanie / odczytywanie danych z bazy.

Bym zapomniał, z samą JavaFX daleko nie zajdziesz.

Ale nie poddawaj się :)

oraz

Tak widziałem, że jeden projekt ma bazę sqlową, ale nad formą zapytań mógłbyś popracować.

Txt czy xml, nie ma znaczenia. Używaj baz danych sql/nosql.

aaa: 5 klas javowych
bbb: 2 + 1 + 2 (2 ostatnie to settery i gettery)
ccc: 5 kls javowych
ddd: 9 klas

(nazwy projektów ukryte)

Z tą 10 to przesadziłem, jak większość to 5.

Chciałeś krytykę to ją dostałeś, a Ty się obrażasz.

Co miałem CI napisać, że jest idealnie i już jutro masz iść na rozmowę na juniora?
Zderzył byś się ze ścianą.
Kolejna sprawa, jak już masz same apki desktopowe, do których linków nie stworzysz, dodaj jakieś screeny, opis. Pokaż co to robi. Nikt nie będzie pobierać kodu i go odpalać.

oraz

To będzie mój ostatni komentarz, bo widzę, że nic do Ciebie nie dociera.

To, że poświęciłem kilka minut na kod obcej mi osoby to jest brak szacunku? Skąd taki wniosek w ogóle? Bo śmiałem napisać, że praktycznie nic nie masz w tych projektach?
Przyjmij to na klatę i zrób jeden duży porządny projekt, jeśli chcesz dostać się na juniora.

Kolega, dał inne wskazówki, ja inne.
Mam Ci pisać jak zianstalować mavena i jak go używać?
Co więcej dodać o nieużywaniu sys outów tylko loggerów?
Info o bazach - nic więcej tutaj nie trzeba.
Lepiej jeden duży projekt, niż 10 projektów na 5 klas - to też podtrzymuję.

No ale tak, lepiej się obrazić, bo ktoś nie powiedział, że projekt jest perfekcyjny.

Zmień podejście, pozdrawiam.

W odpowiedzi na to dostałem komentarz autora, mówiący, że nie mam do niego szacunku, ze się na nim wyżywam bo jest młodszy (stażem), że mam problem z umiejętnościami miękkimi. Że jak mogłem poświęcić tylko 3 minuty na analizę (toż to brak szacunku!!!).

Jak uważacie, czy początkujący programista ma rację?
Czy moje komentarze można uznać za atak / wyżywanie?

edytowany 2x, ostatnio: kixe52, 2019-08-08 14:14

Pozostało 580 znaków

2019-08-08 14:15
2

Uważam, że to zwykły pajac i nie rozumiem czemu marnujesz na niego dalej czas :D

Chyba dlatego, że było to na grupie publicznej więc chciałem zadbać jakoś o swój wizerunek i odpowiedzieć w miare normalnie, bez emocji. - kixe52 2019-08-08 14:16
@kixe52 ludzie na grupach publicznych mają bardziej kruche ego (bo widać ich imię i nazwisko), więc bardziej zaciekle się bronią. Jednak zasłonięcie się za nickiem pozwala zwykle na bardziej kulturalne rozmowy. - LukeJL 2019-08-08 15:16

Pozostało 580 znaków

2019-08-08 14:15
4

Jeśli cenisz sobie zdrowie psychiczne to nie siedź w wannabe junior grupach na fb ;)


Spring? Ja tam wole mieć kontrole nad kodem ᕙ(ꔢ)ᕗ
Haste - mała biblioteka do testów z czasem.

Pozostało 580 znaków

2019-08-08 14:31
5

Tak, Twoje komentarze moga byc odebrane zbyt personalnie.

ale wezmy to na czynniki pierwsze

Szczerze to nic tam nie masz

skoro tam napisal cos to znaczy ze cos tam ma. Jak sie zaczynalo programowanie to te 2-3 klasy to bylo sporo zamiast tego mozesz napisac

Mysle, ze Twoj kod z biznesowego punktu widzenia nie ma zadnej wartosci bo (tutaj jakies wyjasnienie). A Ty jak uwazasz?

dzieki takiemu prostemu zabiegowi mowisz dokladnie to samo, ale niedosc, ze w lagodniejszy sposob to konczac pytaniem zmuszasz autora do przemyslenia i odpowiedzi

Zamiast

Napisałeś "mam za to kilka własnych projektów hobbystycznych" - gdzie?

mozna

Nie widze kilku projektow, pod tym adresem widze tylko (tutaj co widzisz). Moglbys mi wskazac gdzie mam patrzec?

Przecież pod każdym widnieje "under deployment" czyli zaden nie jest skończony (nie jest nawet na przyzwoitym poziomie co do wielkości).

To akuratnie nie ma znaczenia. Review code mozna zawsze zrobic

Każdy z nich ma max 10 klas javowych (połowa to settery i gettery).
Odnosi sie do pierwszego komentarza. Moglby byc wrecz jako argument ze nie nadaje sie do "prawdziwego swiata programowania"

Brak jakiejkolwiek bardziej skomplikowanej logiki.

Niektore bibloteki (szczegolnie w npm!) nie maja skomplikowanej logiki. Pisze juz pare lat i rzadko kiedy pisze cos bardziej skomplikowanego

Zamiast

Brak loggerów - nie używaj sys outów

mogles

Brak loggerów. (tutaj link do artykulu wyjasniajacy dlaczego tak jest) - nie używaj sys outów (tutaj link dlaczego tak, albo sam wyjanij)

Ogolnie rozkazujesz, ale nie mowisz dlaczego i po co, wiec omine reszte

na juniora zbyt wielkich szans na obecnym rynku raczej nie masz

Pamietaj ze to Twoja opinia. Pracowalem w kilku firmach, niektorzy na prawde nie powinni pracowac jako programisci a byli seniorami

za małe projekty, zrób jeden większy, dodaj jakąś logikę bardziej skomplikowaną niż wrzucanie / odczytywanie danych z bazy.

To tez nie do konca, lepiej maly projekt ktory bedzie uzyteczny niz duzy byle by byl

Poki co starczy, reszta do przepychanki, ktore nie maja nic wspolnego z code review

Zeby nie bylo. Mi np odpowiada sposob jak ktos bezposrednio i dosadnie komentuje kod, ale to jest rzecz bardzo indywidualna, dlatego napisalem, ze jak najbardziej Twoj code review moze byc odebrany jako pasywno-agresywny

edytowany 2x, ostatnio: fasadin, 2019-08-08 14:33
Dzięki o to mi chodziło! Chciałem zobaczyć punkt widzenia osoby trzeciej. - kixe52 2019-08-08 14:34
jest dosc duzo artykulow jak pisac bezosobowe review-code Jak nie bedizesz mogl znalezc daj znac to poszukam w wolnej chwili - fasadin 2019-08-08 14:42
Ogólnie niektórzy ludzie mają problem z przyjmowaniem ostrzejszej krytyki i od razu zaczynają mieć pretensje. Dla mnie np każda krytyka jest cenna i nawet wole ostrą/dostadną niż głaskanie po główce - baant 2019-08-08 14:44
sam napisales slowo kluczowe dla mnie dla tej osoby nie. Mozna krytyke napisac tak, ze bedzie dla kazdego ok. - fasadin 2019-08-08 14:47

Pozostało 580 znaków

2019-08-08 14:47
0

Odniosę się jeszcze do słów @fasadin o tym, że mimo małej ilości klas code review i tak można zrobić. No dobrze, mamy projekt 5 klas (java fx), jeden to typowy main odpalania okienka, drugi to controller na kilka linii. Zostaja 3 klasy, jedna to typowa encja z getterami/setterami. Mamy 2 pliki po kilkadziesiąt linii, w których praktycznie nie ma do czego się odnieść bo nic tam nie ma.

Czy słowa "Przecież pod każdym widnieje "under deployment" czyli zaden nie jest skończony (nie jest nawet na przyzwoitym poziomie co do wielkości)." są aż tak ostre w takiej sytuacji? Stwierdzam fakt - "nic tu nie ma". Co się dało to wypunktowałem.

Pozostało 580 znaków

2019-08-08 14:52
1

Chyba malo code-review robiles skoro mowisz ze ma tam co oceniac

Sa testy? Nie ma? Juz masz uzasadniony komentarz

Mamy 2 pliki po kilkadziesiąt linii, w których praktycznie nie ma do czego się odnieść bo nic tam nie ma.

skoro maja kilkadziesiat linii to uwierz mi, ze znalazlbym cos. Chociazby jakiegos ifa ktorego da sie pozbyc albo dodanie klasy by bylo bardziej SOLID

Nie sa ostre te slowa ale np to

Chciałeś krytykę to ją dostałeś, a Ty się obrażasz.

jest komentarz pasywno agresyny i mnie nie przekonasz ze jest inaczej bo nie masz racji i sie nie znasz. Chciales krytyke to masz

w Powyzszym zdaniu tez stwierdzilem fakt, zrobilem to w niezbyt przyjemny sposob, ale przeciez to fakt zgadza sie?

edytowany 3x, ostatnio: fasadin, 2019-08-08 14:52
Różnica taka, że ja zakładając ten temat oczekuję krytyki i ja przyjmuję. Ale nie atakuję Ciebie gdy nie podoba mi się Twoje zdanie. Temat założony właśnie z tego powodu - zacząłem się zastanawiać czy naprawdę dałem powód autorowi aby mnie atakował? Dziękuję za komentarze ;) - kixe52 2019-08-08 14:55
jest jeszcze jedna roznica. To ze ja napisalem pasywno-agresyny post (jedno zdanie), nadal nie daje Ci prawa mnie atakowania. Tak samo druga osoba nie powinna Cie atakowac za to w jaki sposob robisz code-review. Mogla podac Ci przyklad tak jak ja ze woli taki code-review - fasadin 2019-08-08 14:57
w praktyce, ludzie nabieraja piany zbyt szybko i nawet proste "spadaj" potrafi wywolac fale negatywnej reakcji - fasadin 2019-08-08 14:59

Pozostało 580 znaków

2019-08-08 15:43
0

Nie wchodząc w to, czy merytorycznie masz rację, czy nie - moim zdaniem komentarze są stonowane i całkowicie OK. Nieraz na 4P padają znacznie mocniejsze słowa. Nie napisałeś niczego, co jest obraźliwe albo niestosowne. Poza tym autor sam prosił o ocenę, więc musiał się liczyć z tym, że usłyszy kilka niemiłych słów.

Z drugiej strony nie znamy wersji kolesia, nie wrzuciłeś tutaj jego odpowiedzi, więc ciężko jednoznacznie ocenić jaki był przebieg całej rozmowy oraz do czego się ustosunkowujesz, niemniej na chwile obecną, moim zdaniem, nie masz sobie nic do zarzucenia (pomijam warstwę merytoryczną, bo szczerze mówiąc parę uwag @fasadin jest nawet sensownych).


That game of life is hard to play
I'm gonna lose it anyway
The losing card I'll someday lay
So this is all I have to say
Nie chciałem podawać linka do posta ani kopiować jego słów - próbowałem je streścić. Według autora nie mam do niego szacunku, cytując: "Człowieku, "tak na szybko" to możesz sobie zjeść hotdoga. Okazujesz mi totalny rak szacunku takim podejściem" - to odnosi się do poświęcenia 3 minut na analizę kodu (3 minuty to też jego słowa). Potem był atak w stylu, że współczuje mi i moim współpracownikom, bo nie mam umiejętności miękkich itp. Stwierdził, że "moje pseudowskazówki służą temu zebym, się poczuł lepiej i że wyżywam się na młodszym koledze". - kixe52 2019-08-08 15:49
Na koniec się tłumaczył, że jest świadomy "małego rozmiaru projektów" i że wie co to maven (ale czemu tego nie użył, to już nie wyjaśnił). - kixe52 2019-08-08 15:50

Pozostało 580 znaków

2019-08-08 15:57
1

Jak dla mnie te komentarze wyglądają ok. Jest tu inny problem. Ludzie zbyt personalnie traktują swój kod i potem przekładają krytykę kodu na krytykę osobistą. Ostatnio w firmie mieliśmy nad tym dyskusję, bo ktoś podlinkował artykuł w stylu "jak robić code review, żebym nie czuł się urażony", gdzie było pełno głupich (moim zdaniem) rad, które tylko zaciemniały recenzję kodu. Ostatecznie uznaliśmy między sobą, że lepiej jest pisać wprost, wytykać wszystko, co nam się wydaje kiepskie (nawet po 10 razy, jak jest powielane) itd. No i jednocześnie podczas recenzji warto zaznaczyć jakieś fajne rozwiązanie, jeśli byśmy na nie nie wpadli albo są bardzo eleganckie czy czegoś nas uczą. Daje to większą motywację do dalszego pisania kodu na poziomie.

edytowany 1x, ostatnio: Michał Sikora, 2019-08-08 15:58

Pozostało 580 znaków

2019-08-08 16:38
1

Jak to dobrze, że nie piszę w Javie, skoro tam liczba klas jest wyznacznikiem jakości projektu :D

Hej, na jednej z grup fb'owych

Ja na Javowe nie wchodzę, tylko na frontendowe grupy na FB - ale o ile jeszcze kilka lat temu był tam jakiś poziom (nawet duży, bo dużo dobrych programistów się tam udzielało), to teraz jest tam takie szambo internetu. Ludzie nie umieją już dyskutować, tylko się przekrzykują, która technologia lepsza, a jak ktoś się nie zgadza albo ma jakiś zarzut do danej technologii, to go wyśmiewają albo piszą z nutą wyższości, że "u mnie działa, widocznie nie umiesz z tego korzystać".

Z kolei na grupach dla juniorów widziałem podobny temat - czyli, że ludzie wrzucają portfolio, niby "do oceny", a w rezultacie jak ktoś im napisze coś, z czym się nie zgadzają, to się obrażają. Albo nawet ktoś trzeci pisze coś w stylu "ktoś się chciał pochwalić portfoliem, a ty go krytykujesz, bez przesady, jak tak można" (co mi trochę przypomina "modern educayshun" - taka poprawność polityczna, gdzie nie można komuś zwrócić uwagi na błędy merytoryczne, żeby zaraz ktoś nie wyskoczył oburzony).

Albo np. ktoś wstawia portfolio i mówi np. co można, a co nie można krytykować - np. powiedzcie, czy dobrze napisałem kod HTML - i wtedy jak zwrócisz uwagę na inne rzeczy (że np. font jest nieczytelny albo strona się wolno ładuje itp.) nie można, bo ktoś się obrazi i będą gadać, że nikt cię nie pytał o zdanie. Że on nie przyjmie takiej "niezamawianej" krytyki.

W d... się poprzewracało od dobrobytu.


((0b10*0b11*(0b10**0b101-0b10)**0b10+0b110)**0b10+(100-1)**0b10+0x10-1).toString(0b10**0b101+0b100);
edytowany 5x, ostatnio: LukeJL, 2019-08-08 16:41

Pozostało 580 znaków

2019-08-08 18:06
7

Żal patrzeć jak rynek robi z ludzi męskie ”waginy”. Ktoś szczerze i w prosty sposób (ale nie wulgarny czy napastliwy) napisze że coś jest kiepskie to się wagina obraża, strzela focha jak pokwitująca nastolatka, a z krytykującego robi się buca i chama bez „wykształconych umiejętności miękkich” (i innego bullshitu). Niezły mindfuck, dość groteskowy.

Jak coś jest do złe to jest do złe i trzeba to napisać wprost, podając jednocześnie wzorce do naśladowania, coby trzymać poziom merytoryczny. Natomiast hodowanie tego typu płaczliwych osobników poprzez staranne dobieranie słów i owijanie wszystkiego w bawełnę robi krzywdę nie tylko osobie krytykowanej bezpośrednio, ale też wszystkim innym czytelnikom danego tematu. Już dziś widać ”zwaginienie” społeczeństwa głównie młodych ludzi (nie tylko jeśli chodzi o przyszłych deweloperów) – strach pomyśleć co będzie w przyszłości.

Nikt nigdy nie nabierze dystansu do swojej pracy, jeśli nie zostanie choć te kilka razy dosadnie uświadomiony o posiadanych brakach. Podkreślam dosadnie, nie wulgarnie czy w sposób poniżający (choć i taka forma krytyki ma swoje plusy, jednak mało kto potrafi je dostrzec).


edytowany 3x, ostatnio: furious programming, 2019-08-08 18:08

Pozostało 580 znaków

2019-08-11 17:18
3

To nie męska wagina, tylko zwykła chodząca ciapa. ;) ;) ;)


"HUMAN BEINGS MAKE LIFE SO INTERESTING. DO YOU KNOW, THAT IN A UNIVERSE SO FULL OF WONDERS, THEY HAVE MANAGED TO INVENT BOREDOM."
A tyle się gimnastykowałem, aby nie być wulgarnym… :D - furious programming 2019-08-11 19:55
Ale przecież ciapa nijak wulgarna nie jest. Toć to określenie niemalże pieszczotliwe :D - Klojtex 2019-08-11 20:13
Dopóki nie zrobisz literówki, wszystko jest w porządku. ;) - somekind 2019-08-11 23:00
przecież ciapa nie oznacza wcale narządu płciowego https://sjp.pwn.pl/sjp/ciapa;2448974.html i jest mniej wulgarne niż wagina w tym kontekście (które w tym kontekście jest określeniem seksistowskim, że niby kobiety = obrażanie - co jest śmieszne, bo nawet idąc w obserwacje na temat tego, w jaki sposób się zachowują zwykle obie płcie, to z moich obserwacji to faceci czesto się obrażają i wyjeżdżają z obroną ego, a kobiety są zwykle w stanie przyjąć pokornie krytykę dotyczącą swoich umiejętności). - LukeJL 2019-08-14 10:43
A kto twierdzi, że ciapa określa narząd płciowy? Przecież właśnie o to chodzi, że nie określa! - somekind 2019-08-14 23:34
no właśnie, a już ktoś zasugerował tutaj w postach, że ciapa to wulgarne słowo niby. - LukeJL 2019-08-15 01:15

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0