[Krolik]

#programowanie #pracbaza #programista15k

Wykop: Kryzys idzie, tysiące CV na jedno miejsce, juniorzy po bootcampach siedzą na ulicy i błagają o jedzenie, AI zabierze wszyskim pracę. Rzeczywistość w pracy: HR przysłało kandydata na interview na seniora, już po wstępnej selekcji (CV, rozmowa z julką z HR). Kod kandydata:

// reset hasfoobar to false
hasFooBar = false;

(nazwa zmiennej zmieniona, bo nie była istotna, ale reszta kodu wklejona dosłownie)

[tos-1_buratino]

@Krolik:
Miał ResetToFalseFactory napisać z interfejsami do tego?

[lycaon_pictus]

@tos-1_buratino: Pewnie tak, ale OP nie przedstawił żadnych wymagań do zadania, więc nie jesteśmy pewni co do tego.

[Kresse]

@tos-1_buratino: Chyba chodzi o bezsensowny komentarz ¯\(ツ)/¯

[Krolik]

@lycaon_pictus: Nie trzeba żadnych wymagań, żeby widzieć, że komentarz bezsensowny.

[lycaon_pictus]

@Krolik:

Nie trzeba żadnych wymagań, żeby widzieć, że komentarz bezsensowny.

Myślałem, że komentarz to treść zadania. Nie załapałem wpisu.

Co do meritum - pomyśl o tym co się musiało dziać w wcześniejszej firmie skoro kolo zostawia takie komentarze.

[tos-1_buratino]

@Kresse: @Krolik
Myślałem, że komentarz jest od firmy opa jako treść zadania.

[Krolik]

Miał ResetToFalseFactory napisać z interfejsami do tego?

@tos-1_buratino: To już by się pogrążył totalnie. Ale i tak się pogrążył bo tego spaghetti się nie da czytać. Ludzie chyba nie wiedzą że jest coś takiego jak dzielenie kodu na funkcje. :D

[nad__czlowiek]

@Krolik: jakby mi ktoś kazał zmienić na false to też bym dał komentarz żeby potem pamiętać o tych debilnych wymaganiach ( ͡°( ͡° ͜ʖ( ͡° ͜ʖ ͡°)ʖ ͡°) ͡°)

[Argony]

AI dodaje takie głupie komentarze

[NieBendePrasowac]

Nie trzeba żadnych wymagań, żeby widzieć, że komentarz bezsensowny.

@Krolik: widzisz, po pierwsze sam bardzo słabo przedstawiłeś we wpisie o co chodzi, po drugie - kiedyś na rekrutacji czepiali się mnie jak nie dałem komentarza, w moim odczuciu całkowicie zbędnego. Może koleś nauczony doświadczeniem komentuje wszystko, byle się ktoś nie przyczepił.

[Krolik]

@NieBendePrasowac: Doceniam starania, ale jednak nie. Parę linijek wyżej ma zainicjowana zmienna count. Bez komentarza i nie wiadomo co ona liczy. Trzeba się domyślać z kontekstu kiedy jest zwiększana (a jest zwiększana w kilku miejscach w takiej wielkiej pętli na 30 linii z 5 zagnieżdżeniami, nie jest to trywialny licznik petli).

Uwielbiam takich programistów - komentują rzeczy które widać w kodzie, a nie komentują tego co faktycznie nie jest oczywiste.

[przemyslaw-maczka]

@Krolik: Dla nich jest oczywiste... przez jakieś pół roku :D A potem niech się ktoś inny martwi.

[Krolik]

@przemyslaw-maczka: ja dostrzegam korelację - kod w którym jest mało komentarzy lub bezsensowne na ogół ma inne problemy np. bugi. Czyli często wcale nie jest oczywisty nawet dla twórcy.

[mleczko-kokosowe]

@Krolik: Nie wiem co ten kod ma robić i co to w ogóle jest wraz z wymaganiami.
Na rozmowach często prowadzący jest zakuty swoimi dogmatami używanymi w firmie, więc jak się nie domyślisz jaki code style czy praktyki lubi to twoje inne podejście zostanie uznane za błąd.
Najbardziej to widać jak przesiąka się technologią czy frameworkiem i nie widzi świata poza nim. Jak ktoś ogarnia dostosuje się do zasad od razu,

[RapIArbuzy]

ITT podkuce atakuja krolika, ktory zjada 99 procent tagu technicznie. Zabawny widok

[tellet]

@Krolik: Bramka numer 3 - naklepał rozwiązanie w Chat-gpt. On tam daje takie bzdurne komentarze return true if true itd.

[przemyslaw-maczka]

@Krolik: Od czego jest TDD? :D

[Krolik]

@tellet: też tak myślałem, ale wtedy oznaczałoby że ten gość musi być równocześnie bardzo przebiegły. I trochę mi się to nie klei. Chodzi o mi o to, że styl kodu zdecydowanie nie przypomina wcale stylu ChatGPT. Nawet inaczej klamry i spacje stawia niż chat, więc musiałby mu jeszcze dobrze zapromptować aby zmienić domyślny styl chata.

[enten]

@Krolik: Ale o co chodzi? Bez kontekstu to sobie możesz cokolwiek wkleić i powiedzieć że głupie bo nie wiadomo po co to jest.
Kod to zwykłe ustawienie flagi, czasami się tak robi jak jest potrzeba (iterujesz po czymś i chcesz mieć flagi że coś się w trakcie iteracji wydarzyło np). Tak więc nie rozumiem o co ból dupy. Wklej lepiej całość funkcji.

[parkor]

@Krolik czyli Twoim zdaniem, jedna ze składowych dobrego kodu to bardzo liczne komentarze?