tasks js-basics-arrays finished#138
tasks js-basics-arrays finished#138code-mike-code wants to merge 5 commits intodevmentor-pl:masterfrom
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Michale,
Ostatecznie zadanka wyglądają bardzo dobrze (mimo problemów - one są częścią nauki) 👍
Zostawiłem też parę komentarzy odpowiadających i uzupełniających :)
| @@ -0,0 +1,3 @@ | |||
| const users = ['Jan Kowalski', 'Kasia Kowalska', 'Anna Janowska', 'Michał Janowski', 'Piotr Piotrowski']; | |||
|
|
|||
| console.log(users[0], users[2], users[4], users.length); No newline at end of file | |||
| }); | ||
|
|
||
| // wyswietlanie ostatniego elementu tablicy | ||
| console.log(randomArray[randomArray.length -1]); |
| // przy 'n=0' miałem pustą tablicę [] | ||
| // prz 'n=1' miałem [1] | ||
| // przy 'n=100' miałem liczby nieparzyste o 1 do 99 |
There was a problem hiding this comment.
Jeśli sprawdzisz tylko 1 przypadek to nie masz pewności czy zadanie działa czy nie np. zawsze dodajesz do tablicy liczbę 1, jeśli sprawdzisz czy zadanie działa dla 1 to tak będzie, ale jak już sprawdzisz przy n=2 to tak nie będzie.
Jeśli sprawdzisz najmniejszą, największą (lub tutaj dużą) i coś po środku to jest większe prawdopodobieństwo że działa dla wszystkich liczb :)
| // prz 'n=1' miałem [1] | ||
| // przy 'n=100' miałem liczby nieparzyste o 1 do 99 | ||
|
|
||
| //plus przy .push wstawiłem nawiasy kwadratowe za pierwszym razem i nie ukrywam, że było to dla mnie problemem bo nie wiedziałem gdzie jest bład, a vsc nic nie podświetlał No newline at end of file |
There was a problem hiding this comment.
Chyba jednak musiał coś podkreślać na czerwono - u mnie tak jest, sprawdziłem :P
| return currentYear - year; | ||
| }); | ||
|
|
||
| console.log(yearsPassed); No newline at end of file |
| // jesli dobrze to rozumiem to .reduce() dodaje do siebie wszystkie elementy tablicy i zwraca je jako jedną wartość?? | ||
|
|
||
| // czy raczej dodaje wszystkie elementy tablicy do pierwszego elementu (accumulator)?? No newline at end of file |
There was a problem hiding this comment.
Oba są prawdziwe :P
reduce działa tak, że to co w return wpada do pierwszego parametru i operacja jest wykonywana aż do przejścia po każdym elemencie tablicy, który jest w drugim parametrze. Warto zapisać trochę inaczej reduce tj. dodać 0 jako pierwszy element w sum - wtedy pewnie będzie to czytelniejsze jak wyświetlimy zawartość parametrów tj.
.reduce(function(sum, number) {
console.log(sum, number);
return sum + number;
}, 0);
2 participants
Zadanie 05 z .reduce() było chyba najtrudniejsze — tu dużo pomogła mi dokumentacja na mdm, aczkolwiek nadal nie do końca rozumiem zasadę działania tej metody
trochę problemów sprawiły mi tez złe nawiasy, czy średnik w złym miejscu, teraz zaczyna do mnie docierać jak problematyczne mogą być takie malutkie błędy