#3 JS05(boolean): 7

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_04/JS.js#L5-L6: тут не нужно и + и parseInt. Они выполняют схожую задачу, и parseInt после + уже нечего делать. Проверка !parseInt даст true и запустит тело if в двух ситуациях:

  • NaN. Собственно для этого оно и сделано.
  • 0. А вот это значение вполне корректно. Так что нельзя будет проверить ноль на четность. Для проверки на NaN используйте isNaN.

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L92 - тут надо было написать || alert..... после prompt(.....), наподобие того, как это написано в следующем задании.

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L180 - можно это написать в виде rate = action ? 37.5 : 36.1, что компактнее и нагляднее.

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L210-L213 - условия можно было объединить через еще одно ||.

http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L5-L6: тут не нужно и `+` и `parseInt`. Они выполняют схожую задачу, и `parseInt` после `+` уже нечего делать. Проверка `!parseInt` даст `true` и запустит тело `if` в двух ситуациях: - `NaN`. Собственно для этого оно и сделано. - `0`. А вот это значение вполне корректно. Так что нельзя будет проверить ноль на четность. Для проверки на `NaN` используйте `isNaN`. http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L92 - тут надо было написать `|| alert.....` после `prompt(.....)`, наподобие того, как это написано в следующем задании. http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L180 - можно это написать в виде `rate = action ? 37.5 : 36.1`, что компактнее и нагляднее. http://gitlab.a-level.com.ua/ostapenkonataliia/HomeWork_JS/src/master/Js_04/JS.js#L210-L213 - условия можно было объединить через еще одно `||`.
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.