Aktywne Wpisy
![Van-der-Ledre](https://wykop.pl/cdn/c3397992/Van-der-Ledre_DVsiYimVww,q60.jpg)
Van-der-Ledre +64
![Van-der-Ledre - Czas spożyć trunek. Czysta, do odcięcia. Od grudnia raz w tygodniu. N...](https://wykop.pl/cdn/c3201142/9621b6a411b70b21a33c28d71fd6d7e3a671aa4278170d446c787353dd25aaaf,w150.jpg)
źródło: 1000019306
Pobierz![zbrodnia_i_kawa](https://wykop.pl/cdn/c3397992/zbrodnia_i_kawa_RRZk3e71gd,q60.jpg)
#carnivore #damskiprzegryw ale tez troche #wygryw bo alergii ani widu ani slychu i juz troche wierze ze to jednak dziala
![zbrodniaikawa - #carnivore #damskiprzegryw ale tez troche #wygryw bo alergii ani widu...](https://wykop.pl/cdn/c3201142/bffac6bec135427b01d22484d40b296713c94cf0949e83a001e99f3e6719d837,w150.jpg?author=zbrodnia_i_kawa&auth=1848403e49c6aaa9bd439658a05a2df1)
źródło: temp_file624677588370938020
PobierzSkopiuj link
Skopiuj linkźródło: 1000019306
Pobierzźródło: temp_file624677588370938020
PobierzWykop.pl
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
@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.
- 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
createArrayOfObjects
ta nazwa jest słaba, możecreateArrayOfDevelopers
? tworzysz tablicę developerów, a nie jakichś bliżej nieokreslonych obiektów.Math.floor(Math.random() * (max - min + 1)) + min
- to bym zcreateArrayOfObjects
wyrzucił do osobnej funkcjigenerateRandomAge
.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ę powinna@MKu8ar: Tak, swiezak ze mnie ( ͡° ͜ʖ ͡°)
Ogolnie ucza mnie, ze komentarze to zlo. I kod powinien byc tak napisany, ze komntarze powinny byc zbedne.
Dzieki, takich komentarzy oczekiwalem.
@kao3991: Przyszlo mi to do glowy.
Szczere pytanie, jest cos zlego w dlugich nazwach? Ucza mnie, ze to dobry nawyk, jezeli dlugie
@MKu8ar: Masz to na mysli? https://www.tutorialspoint.com/entity_framework/entity_framework_fluent_api.htm
@xxx2013: wystarczyło
developers
https://jsfiddle.net/gg3t1t2r/
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
@MKu8ar: I kompiluj JS do JS, bo tak ( ͡° ͜ʖ ͡°)