[xxx2013]

Mirki zrobi mi ktos male code review? https://jsfiddle.net/qo38n8rd/31/

You have an array of objects in JavaScript. Each one contains a name (a string) and age (a number). Write a function which returns the objects ordered by age.

Jest to pytanie rekrutacyjne na front-enda. Pracy na razie nie mam zamiaru zmieniac, ale z nudow czasami mozna sie pobawic ( ͡° ͜ʖ ͡°)

Co byscie tu poprawili, a co ulepszyli?

#js #javascript #programowanie

[MacDada]

@xxx2013: Napisz testy

[xxx2013]

@MacDada: Dobra rada. I zarazem trafiles w moj najslabszy punkt ( ͡° ͜ʖ ͡°)

[m.....o]

@xxx2013: kto w dzisiejszych czasach używa for, zrób to ładnie na fluent api

[Afrael]

https://jsfiddle.net/qo38n8rd/31/

@xxx2013: w zasadzie ciężko się do czegoś doczepić. Wszystko jest poprawnie, ale powiedz mi, jakie to jest stanowisko? Bo wygląda to na dosyć proste zadania.

[m.....o]

@xxx2013: i używaj pan lambd

[xxx2013]

@Afrael: Front end. Moim skromnym zdaniem chodzi tu bardziej o sprawdzenie stylu pisania, czy kod jest czytelny i latwy do zrozumienia. Z doswiadczenia wiem, ze sa osoby, ktore potrafia tak pisac kod, ze czlowiek siedzi i rozkminia o co chodzi.

[M.....r]

@xxx2013:
- Używasz składni JSa z lat 90. Podszkól się z ES6 (chodzi mi szczególnie o pętle for, vary, brak fluent api).
- Funkcja sortByAge() tworzy tablicę obiektów, a powinna sortować po wieku według nazwy.
- Funkcja createArrayOfObjects() powinna przyjmować jako parametr liczbę obiektów do utworzenia. Ewentualnie wydziel tę liczbę do consta zadeklarowanego gdzieś u góry. Oprócz tego nazwa kompletnie nic nie mówi. Array obiektów czego?
- Masa liczb w kodzie bez żadnego komentarza (poczytaj o antywzorcach, "magicznej

[kao3991]

@xxx2013: createArrayOfObjects ta nazwa jest słaba, może createArrayOfDevelopers? tworzysz tablicę developerów, a nie jakichś bliżej nieokreslonych obiektów.
Math.floor(Math.random() * (max - min + 1)) + min - to bym z createArrayOfObjects wyrzucił do osobnej funkcji generateRandomAge.
całe przygotowanie pojedynczego developera też wyodręniłbym do funkcji, żeby jedna funkcja nie robiła jednocześnie dwóch rzeczy (developera i listy developerów)
funkcja sortByAge nie powinna zajmować się stworzeniem kolekcji, tylko posortowaniem jej. Kolekcję

[xxx2013]

Widzę, że dopiero uczysz się programować.

@MKu8ar: Tak, swiezak ze mnie ( ͡° ͜ʖ ͡°)

Masa liczb

[xxx2013]

Math.floor(Math.random() * (max - min + 1)) + min - to bym z createArrayOfObjects wyrzucił do osobnej funkcji generateRandomAge.

całe przygotowanie pojedynczego developera też wyodręniłbym do funkcji, żeby jedna funkcja nie robiła jednocześnie dwóch rzeczy (developera i listy developerów)

@kao3991: Przyszlo mi to do glowy.

var arrayOfDevelopers = []; trochę chyba za długie

[xxx2013]

brak fluent api

@MKu8ar: Masz to na mysli? https://www.tutorialspoint.com/entity_framework/entity_framework_fluent_api.htm

[m.....o]

@xxx2013: nie chodzi o to żeby nazwy były długie, im krótsza tym lepsza, ale musi dobrze opisywać zmienną, zamiast 'arrayOfDevelopers' napisałbym 'developers', po pierwsze nieistotne jest to, że to jest array, a po drugie krócej

[r.....e]

var arrayOfDevelopers = []; trochę chyba za długie te nazwy

@xxx2013: wystarczyło developers

[kao3991]

@xxx2013: może kwestia podejścia, żaden ze mnie guru, ale ja wyznaję raczej zasadę że nazwę należy skrócić kiedy tylko można bez straty czytelności - po prostu łatwiej się czyta.

[chladek317]

ja trochę z innej beczki. Zna ktoś jakąś stronę z podobnymi zadaniami?( ͡° ͜ʖ ͡°)

[M.....r]

@xxx2013: masz tu na szybko napisany kod. Postaraj się iść w tym kierunku:

https://jsfiddle.net/gg3t1t2r/

[xxx2013]

@MKu8ar: Dzieki za nakierowanie

[M.....r]

@xxx2013: z książek, które ładnie wyjaśniają podstawy ES, to polecam "JavaScript. Programowanie obiektowe" Helionu. Trochę nieaktualna, ale warto od niej zacząć przygodę z frontem.

[cevilo]

@rubytree: tak, to array w nazwie nie jest potrzebne.
Poza tym createArrayOfObjects to zła nazwa dla funkcji, szczególnie, że zwracamy arrayOfDevelopers
Powinno być developers i createDevelopers
Tutaj jest zbyt mało czytelnie z jakiego zakresu wiek może być wylosowany - Math.floor(Math.random() * (max - min + 1)) + min

[vytah]

Podszkól się z ES6

@MKu8ar: I kompiluj JS do JS, bo tak ( ͡° ͜ʖ ͡°)