[Trusiak]

Pierwszy dzień w nowej robocie jako junior js developer.
Senior zrobił mi code review na konwersacji z szefostwem.
Zapowiada się elegancka atmosfera pracy (⌐ ͡■ ͜ʖ ͡■)

#programowanie #javascript #frontend #heheszki

[Trusiak]

@mistrzus: ten scope ma znacznie więcej metod, które będą dopisywane. Do tego ciągle pobierasz dane bez debounce, to dopiero morderstwo dla zasobów.

[Landmark]

@Trusiak:

@mistrzus: nooo, czytelne fest (✌ ﾟ ∀ ﾟ)☞

[MarcinOrlowski]

@Phatee: ale wiesz że tu nie chodzi o if-a tylko o iife? xD

Dla tych nie w temacie "iife" -> Immediately Invoked Function Expression == a JavaScript function that runs as soon as it is defined.

@Passer93: ale to nie zmienia ze odzywka kolesia jest buracka i trzeba tępić takich pajaców, bo to akurat jest toxic.

[msichal]

@Trusiak: co to jest iife?

[Phatee]

@Passer93: @MarcinOrlowski: Człowiek najlepiej uczy się na głupotach które powiedział. W robocie unikam js jak mogę, ale nie mogę powiedzieć że żałuję że źle zrozumiałam o co chodziło bo teraz przynajmniej mnie nie przestraszą te dziwne zapisy.

Co do opa to fajna tam atmosfera musi być jak już w pierwszym dniu łamią czwartą ścianę i rozmawiają w sposób bezpardonowy. Albo jest tam zajebiście, albo zajebiście c-----o xD Nic

[p.....o]

@mistrzus:

ale to nie zmienia ze odzywka kolesia jest buracka i trzeba tępić takich pajaców, bo to akurat jest toxic.

Dokładnie. Inaczej nic się nie nauczy.

[Passer93]

@Phatee: Żadna to głupota, nikt nie wie wszystkiego. Teraz już wiesz o jedną rzecz więcej.

No atmosfera albo c-----a albo to jest bait xD Bo cięzko mi wyobrazić sobie firme gdzie tak wyglądają code review.

[Morf]

Pierwszy dzień w nowej robocie jako junior js developer.

@Trusiak: dlaczego pracujesz w firmie gdzie code review robi się na fejsbuku? XD

[motokate]

@Trusiak: Ale już do braku { } przy dolnym if-ie się nie przyczepił.


Albo - dlaczego onscroll, a nie addEventListener. Albo nazewnictwo - const selector to raczej powinno być const element.

[Rudy_]

@motokate a dlaczego AddEventListener??
Oho, klamerki bola

[motokate]

@Rudy_: Bo onscroll może mieć tylko jedną wartość na raz. Być może OP właśnie nadpisał jakiś inny handler, o którym nie wie.

[MarcinOrlowski]

@Trusiak: Ale już do braku { } przy dolnym if-ie się nie przyczepił.

@motokate: kiedyś bym bił kijem za brak {} przy blokach ale po obcowaniu z QMLem mi przeszło. Wszystko zależy od kontekstu się okazało. W QML im mniej tym lepiej i jak możesz zrobić onelinera zamiast 3 to lepiej jest, więc sugeruje "chill" w tych sprawach :)

[Flypho]

@Trusiak:
Cały JS wygląda dla mnie jak kulfon. ( ͡º ͜ʖ͡º)

[motokate]

@MarcinOrlowski: Jak już się chce oszczędzić na maksa znaki, to można użyć ternary operatora. ;) Za dużo siedziałam w legacy kodzie, gdzie autor w ogóle się nie przejmował nawiasami (albo było za mało, albo za dużo, rzadko kiedy tyle, ile potrzeba), żeby mi to nie podnosiło ciśnienia. Urywałabym nogi przy samej dupie i posypywała solą. ヽ( ͠°෴ °)ﾉ

[motokate]

@Trusiak: Jeszcze headerPos się niezbyt dobrze nazywa - pos sugeruje, że to mogą być obie współrzędne, nic w nazwie nie wskazuje na to, że jest jedynie pionowa.

[Nolimit]

@Trusiak: Brak klamerek boli motzno, ale ogółem odzywka bardzo słaba

[jjczopek]

@motokate: oj tak, pamiętam dawno temu jak sobie nie dałem klamer dla else i się zastanawiałem co tu sie od******a że się tylko jedna instrukcja wykonuje... A były ze 3. Od tego czasu zachęcam mocno do klamer, chyba ze ktoś bardzo nie chce to niech wyrażenie będzie w jednej linicje z tym ifem albo elsem i wtedy jest wszystko jasne.

[K.....a]

@mistrzus: o hui, takim kodem to zabijesz przeglądarke

[peasoup]

@Trusiak: Możesz dać ternary operator.

[beerman]

@Trusiak: współpracownicy pewnie mają tu konta