Почему не до конца отрабатывается условие в моём коде на JavaScript?

Question

[Speakea1y1289]

Добрый вечер. Имеется следующий код:

Почему, если в поле "Внутреняя площадь дома, м2" вбивать значение больше 300 м2, то уведомление сверху не вылезает? На данный момент сообщение вылезает, если ввести значение меньше 90 или 120 (в зависимости от указанного этажа)

Answer 1

[hzzzzl]

смотри, что происходит если value > 300

if (value>300) { 
// я здесь value > 300
notify("Дома рассчитываются индивидуально!<br>"); value=300; 
} else { notify(""); }

if (value<120) { 
  notify("Дома рассчитываются индивидуально!<br>"); value=120; 
} else { 
// а теперь я здесь value < 120 = false
// блок обнуляется
   notify(""); 
}

Comments

[Speakea1y1289]

А что тут не так?

[hzzzzl]

Speakea1y1289, ну сначала отрабатывает notify('с текстом'), а потом в следующем блоке notify(''), просто оба ифа выполняются и поэтому кажется что текст "не выводится"

[Speakea1y1289]

hzzzzl, Ну так подожди, если value пусть 150, это же не >300 и поэтому по логике должен отрабатываться код в else, то бишь notify("");

[hzzzzl]

Speakea1y1289,
https://jsfiddle.net/1akb7pgm/1/

[Speakea1y1289]

hzzzzl, Ты гений)) Всё заработало, спасибо большое. Слушай, а что скажешь на счёт всего этого кода? Может его можно улучшить, оптимизировать, сделать его более правильным?

[hzzzzl]

Speakea1y1289, событие onchange поменять на oninput, чтобы после каждого нажатия на кнопку всё пересчитывалось и проверялось.. ну и вообще конструкция из четырех if .. else if .. мне самому не особо нравится, но сейчас как-то голова не варит это переделывать по красивому :D

[hzzzzl]

хотя не, если поставить oninput то уже не работает, потому что value меняется сразу на 90

[Speakea1y1289]

hzzzzl, Эмм... Если менять на oninput, то ты ничего тупо не сможешь ввести, постоянно будет ошибка

[Speakea1y1289]

hzzzzl, А можешь сделать зависимость также от количества этажей. Если данные в поле "Внутреняя площадь дома, м2" уже забиты, при этом кол-во этажей = 10 к примеру, то при изменении кол-ва этажей на 2 автоматически сразу менялся диапазон в площади дома

[Rustam Bainazarov]

Speakea1y1289, он может, поэтому лучше сразу спрашивайте номер карты hzzzzl.

[Speakea1y1289]

Рустам Байназаров, зачем лезете в чужой разговор? Это его дело, отвечать мне или послать на 3 буквы, но никак не ваше!

[Rustam Bainazarov]

Speakea1y1289,

[Speakea1y1289]

Рустам Байназаров, много оригинала не требуется в данной ситуации...

[Rustam Bainazarov]

Speakea1y1289, откуда вам знать, возможно, что я хотел вам написать решение вашей задачи, но теперь передумал :))

[Speakea1y1289]

Рустам Байназаров, пока этого я этого желания у вас не увидел, но как появится, то обязательно пишите :) Да и к чему эти все лишние разговоры...

[hzzzzl]

интересно тут у вас ;Д
короче самое изматывающее что я сделал это скопировал всё в codepen, jsfiddle бесит просто как-то

https://codepen.io/anon/pen/EzpJXQ?editors=1010

[Speakea1y1289]

hzzzzl, почему codepen.io, когда я зашёл, он постоянно перезагружается?))))

[Speakea1y1289]

hzzzzl, и это с любым решением

[hzzzzl]

Speakea1y1289, в смысле вообще всегда кодепен, или только по этой ссылке?
как бы должно работать, но в любом случае - https://jsfiddle.net/9fa2ecqo/

[Speakea1y1289]

hzzzzl, А если сделать без дефолтных значений? Там я заметил, что возникает проблема, если по умолчанию поля не заполнены и вводим допустим цифру 2 в кол-ве этажей, то в площади дома подставляется значения из мин. порога = 120 и при этом показывается уведомление. Вот как его не показывать в такой ситуации?

[Speakea1y1289]

hzzzzl, Блин, и можно ещё сделать, чтобы, к примеру, при заполнение только кол-ва этажей, не ставилось значение из диапазона, а чтобы:
поле с площадью дома зависело от кол-ва этажей при этом только, если поле с площадью заполнено каким-то значением.