Complete all tasks #168
Complete all tasks #168
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Sandro,
Tutaj również ok 👍
Zwróciłem uwagę na 2 rzeczy, które warto poprawić ;)
| } else { | ||
| console.log(`${result.name} is less than or equal to 20`) | ||
| } | ||
| } |
| const results = x * i; | ||
| console.log(`${x} x ${i} = ${results}`); | ||
| } | ||
| } |
| i++; | ||
| } | ||
| console.log(`${sequence} = ${result}`); | ||
| } |
| console.log(`Podany argument ${sum} jest nieparzysty`); | ||
| } | ||
| } | ||
| const completeInfo = showInfo(sum, isSumEven); |
04/app.js
Outdated
|
|
||
|
|
||
| function arithmeticMean(arr) { | ||
| return arr.reduce((acc, curr) => acc + curr, 0) / arr.length; |
There was a problem hiding this comment.
Warto też sprawdzić czy tablica nie jest pusta, aby nie dzielić przez 0 :)
05/app.js
Outdated
| if (!grades || grades.length === 0) return 0; | ||
| const sum = grades.reduce((acc, val) => acc + val, 0); | ||
| return sum /grades.length; |
There was a problem hiding this comment.
Zwróć uwagę, że te 3 linie są praktycznie identyczne do tych z 29-31. W takiej sytuacji warto sprobować napisać funkcję/metodę, którą wykorzystasz w obu miejscach.
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Sandro,
Super, o to chodziło! 👍
Zostawiłem też istotny komentarz do przeanalizowania ;)
04/app.js
Outdated
| if( arr.length !== 0) { | ||
| return arr.reduce((acc, curr) => acc + curr, 0) / arr.length; | ||
| } else { | ||
| return 0; | ||
| } |
There was a problem hiding this comment.
Można też tak (chyba trochę czytelniej):
if(arr.length === 0) {
return 0;
}
return arr.reduce((acc, curr) => acc + curr, 0) / arr.length;
Tak, łatwiej złapać kontekst po rzuceniu okiem (taka niesubiektywna opinia).
Moja wersja: Jeśli brak elementów to zwróć 0, w przeciwnym razie oblicz sumę
Twoja wersja: Jeśli NIE ma braku elementów to oblicz sumę, w przeciwnym razie zwróć 0
| if (!grades || grades.length === 0) return 0; | ||
| const sum = grades.reduce((acc, val) => acc + val, 0); | ||
| return sum /grades.length; | ||
| return this.calculateAverage(this.grades[subject]); |
There was a problem hiding this comment.
Super! Zobacz jak bardzo zwiększyła się czytelność!
2 participants
No description provided.