JS Basics: complete tasks #170
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Natalio,
Całość prezentuje się ok! 👍
Zostawiłem dwa istotne komentarze :)
|
|
||
| count(numberA, b); | ||
| count(10, 10); | ||
| count(10, -10); |
02/app.js
Outdated
| } | ||
| console.log(`${str} = ${poweredResult}`); | ||
| } | ||
| calculatePower(); |
There was a problem hiding this comment.
Ogólna uwaga do rozwiązania. Lepiej mieć funkcję z parametrami i w ten sposób przekazywać dane.
W drugim rozwiązaniu masz zaszyte w funkcji wiele czynności tj. liczenie potęgi, odbieranie danych od użytkownika. To powoduje, że to rozwiązanie nie jest elastyczne. Lepiej mieć osobną funkcję do liczenia potęgi, osobną do odbierania danych od użytkownika i potem połączenie obu tj.
const a= getUserInput();
const n = getUserInput();
const result = calculatePower(a, n);
There was a problem hiding this comment.
@devmentor-pl Faktycznie, w tym zadaniu nie byłam pewna czy mamy od użytkownika pobierać obie liczby do potęgowania, czy tylko jedną (czyli tą, którą chcemy spotęgować). I faktycznie, nie pomyślałam, żeby te dane pobierać osobno. Zaraz to poprawię.
There was a problem hiding this comment.
@devmentor-pl poprawiłam, miałam też tam błąd, że prompt zwracał stringa zamiast liczby.
| /* wykorzystanie funkcji */ | ||
| const sum = getSum(a, b, c); | ||
| const even = isEven(sum); | ||
| showInfo(sum, even); |
| return finalAvg; | ||
| } | ||
| const finalResult = calculateFinalAverage(finalArr); | ||
| console.log(finalResult); |
05/app.js
Outdated
| const arrLength = allGrades.length; | ||
| const sumOfAllGrades = allGrades.reduce(function (acc, currentVal) { | ||
| return acc + currentVal; | ||
| }, 0); | ||
| const avg = Math.ceil((sumOfAllGrades / arrLength) * 100) / 100; | ||
| return avg; |
There was a problem hiding this comment.
Zaznaczona cześć jest dość mocno podobna do tej z linii 65-70. Może warto napisać metodę/funkcję, którą wykorzystasz w obu miejscach?
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Natalio,
Super, o to chodziło! 👍
|
|
||
| const userDataNumber = getUserInput(); | ||
| const userDataPower = getUserInput(); | ||
|
|
| const avgOfOne = Math.ceil((sum / numOfElements) * 10) / 10; | ||
| return avgOfOne; | ||
| const avg = this.calculateAvg(subjectArray); | ||
| return avg; |
2 participants
No description provided.