#5 JS11(functions): 6?

오픈
gitgod2 년 전을 오픈 · 0개의 코멘트
gitgod 코멘트됨, 2 년 전

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`
로그인하여 이 대화에 참여
레이블 없음
마일스톤 없음
담당자 없음
참여자 1명
로딩중...
취소
저장
아직 콘텐츠가 없습니다.