#5 JS11(functions): 6?

Open
opened 2 years ago by gitgod · 0 comments
gitgod commented 2 years ago

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L71 - функция не проверяет наличия fatherName, без него будет "Вася Пупкин undefined"

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L190-L213 - маловато будет, относительно кода предыдущего задания;

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L223 - это не набор параметров, это один параметр;

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L231 - такой return всегда возвращает undefined. Для false это еще полбеды, так как undefined и false оба являются ложными значениями в булевском контексте. Однако return console.log('true') тоже не возвращает true (ниже с этим уже ок)

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L224-L227 - вместо этой копипасты в previousValue можно занести -Infinity и сделать цикл с 0го индекса, а не с первого;

шото не вижу getSetForm

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L71 - функция не проверяет наличия `fatherName`, без него будет "Вася Пупкин undefined" http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L190-L213 - маловато будет, относительно кода предыдущего задания; http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L223 - это не набор параметров, это один параметр; http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L231 - такой `return` всегда возвращает `undefined`. Для `false` это еще полбеды, так как `undefined` и `false` оба являются ложными значениями в булевском контексте. Однако `return console.log('true') ` тоже не возвращает `true` (ниже с этим уже ок) http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_10/js.js#L224-L227 - вместо этой копипасты в `previousValue` можно занести `-Infinity` и сделать цикл с 0го индекса, а не с первого; шото не вижу `getSetForm`
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.