Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- // вирівняв блоки завдяки індентації, код став читабільніший. Додав фігурні дужки там, де була навіть одна команда, знов
- // таки щоб було краще видно блок, а по-друге доволі часто доводится додати команду, і тоді, якщо не помітити, що дужок
- // нема, сложніше зрозуміти де баг. краще відраз брати в дужки навіть одну команду. До того ж ви самі це зробили у одному
- // випадку, та не зробили в іншому. Краще дотримуватися одного стилю.Все в ім'я простоти читання (як слід і написання) коду.
- let floor = prompt("Який поверх?")
- if (floor < 1 || floor > 9) {
- console.log("ok")
- }
- else if (floor >= 1 && floor <= 3 || floor >= 7 && floor <= 9) {
- console.log("ok")
- }
- else {
- let password = prompt("Гасло?!") // ви приймаєте строку від промпту, спробуйте у наступному рядку порівняти з ===
- if (password == 1337) { // тут ви порівнюєте строку від промпту та число, тому не суворе порявняння працює, суворе ні.
- // можна або +промпт і тоді буде намагатися перетворити строку в число, або пароль взяти у лапки, і буде строка.
- console.log("пройшов перевірку")
- } else {
- console.log("спробуй ще")
- }
- }
- // алгоритм норм, є прохання, у коді, що на рев'ю даєте не змінювати умови та відповіді, не ускладнювати завдання, не робити
- // зайвого. Мета прохання - економія часу вашого і рев'ювера. Та як показує досвід на підготовчому не має сенсу робити
- // зайві штуки, або покращувати існуючи. Значно корисніше поглибити знання та досвід кодування саме за темою. Можливо опти-
- // мізувати, або написати питання. Щодо оптимізування, то поміркуйте, чи може спростити код введення змінної
- // let answer = 'ok', а також не кваптеся виводи її, зробіть це лише раз, а до цього просто змінюйте, коли потрібно.
- // Оптимізований код за посиланням https://pastebin.com/f5Ui0JV2 , але подивиться його після своєї оптимізації
- // Гарного коду.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement