Где-то пропускаю ошибку, скрипт не дорабатывает

@Дима Артёмов · Регистрация: 16.10.2009

Author24 — интернет-сервис помощи студентам

Здравствуйте. Вот эта конструкция должна посчитать количество отмеченных чекбоксов (и считает), и если оно менее, допустим, 2, дать отлуп и не отправлять форму на сервер. То есть, вернуть false при сабмите формы. Так вот, не возвращает. Скажите, где я слажал?
Спасибо.

PHP/HTML

<html>
    <head>
        <meta charset='utf-8'>
        <script type = "text/javascript">
        window.onload = function()
    {
        document.getElementById("Myform").onsubmit = function() 
        {
       isCountCheck();
        }
        function isCountCheck() 
        {
        var i, len, inputs = document.f.getElementsByTagName("input");
        var count = 0;
            for (i = 0, len = inputs.length; i < len; i++) 
            {
                if (inputs[i].type === "checkbox" && inputs[i].checked)
                {
                count+=1;
                }
            }
            if (count<2)
            alert("Отлуп определенного содержания");
        return false
        }   
    }
        
        </script>
    </head>
    <form action="anketa.php" method="post" name="f" id="Myform">
    
    С++<input type="checkbox" name="checkbox[]" id="С" value="C++"><br>
    JavaScript<input type="checkbox" name="checkbox[]" id="JS" value="JavaScript"><br>
    Perl<input type="checkbox" name="checkbox[]" id="Perl" value="Perl"><br>
    <input type="checkbox" name="checkbox[]" id="PHP" value="PHP"><br><br>
    <input type="submit" name="submit" id="countChecked" value="Отправить"> 
    </form>
    <a href="script.php">очистить</a><br><br><br>
 
   </body>
</html>

@whiteapps · 20.10.2014, 09:41

Дима Артёмов,

Javascript

 document.getElementById("Myform").onsubmit = function() 
        {
       return isCountCheck();
        }

@Lazy_Den · 20.10.2014, 09:46

Дима Артёмов, для того, чтоб отменить событие submit формы, вместо "return false", используйте preventDefault():

Javascript

function init(){
    document.f.addEventListener('submit', function(e){
        var chbx = this.querySelectorAll('input[type="checkbox"]'),
            cnt = [].filter.call( chbx, function( el ) {
            return el.checked;
        }).length;
        if(cnt < 2) e.preventDefault(); // отменяем submit формы
    }, false);
}
window.addEventListener('load', init, false);

@whiteapps · 20.10.2014, 09:53

Lazy_Den,

1) preventDefault() - это метод из jquery, а ТС его не используется
2) preventDefault от return false отличается только тем что, return false - не дает событию всплывать.
preventDefault актуален допустим при клике на элементе, когда вы не хотите мешать отработке события на родительских элементах.
submit - это событие только формы, он не всплывает. так что для него безразлично что использовать preventDefault или return false

@Lazy_Den · 20.10.2014, 09:57

whiteapps, сделаем проще... Вот вариант с pareventDefault, а вот тот же, но с return false. Сразу смотрим, что jQuery не подключен и открываем для себя то, что с return false, оно таки не работает, как хотелось бы..

@whiteapps · 20.10.2014, 10:07

Не по теме:

Lazy_Den, пойду РТФМить :read:

@Lazy_Den · 20.10.2014, 10:19

Сообщение от whiteapps

пойду РТФМить

Тут не нужно особо углубляться в манулы

В случае с формой, событие происходит на кнопке (в объекте события заметим, что "explicitOriginalTarget" - явная первоначальная цель - именно кнопка). И вы сами правильно сказали, что preventDefault предотвращает всплытие, а всплытие события от кнопки, приходится как раз на форму. ))

Добавлено через 5 минут
P.S. По своему ответу на основной вопрос - что-то я с утра перемудрил. ))

Javascript

1	var cnt = this.querySelectorAll('input:checked').length; // сразу количество отмеченых

@Дима Артёмов · 20.10.2014, 11:41 **[ТС]**

Всем спасибо! Век живи- век учись.

@whiteapps 413 / 378 / 220 Регистрация: 18.07.2014 Сообщений: 1,259
	20.10.2014, 09:53	4
	Lazy_Den, 1) preventDefault() - это метод из jquery, а ТС его не используется 2) preventDefault от return false отличается только тем что, return false - не дает событию всплывать. preventDefault актуален допустим при клике на элементе, когда вы не хотите мешать отработке события на родительских элементах. submit - это событие только формы, он не всплывает. так что для него безразлично что использовать preventDefault или return false 0

@Lazy_Den 3325 / 2845 / 1423 Регистрация: 15.01.2014 Сообщений: 6,170
	20.10.2014, 09:57	5
	whiteapps, сделаем проще... Вот вариант с pareventDefault, а вот тот же, но с return false. Сразу смотрим, что jQuery не подключен и открываем для себя то, что с return false, оно таки не работает, как хотелось бы.. 1

@whiteapps
	20.10.2014, 10:07 #6
	Не по теме: Lazy_Den, пойду РТФМить :read: 0

@Дима Артёмов 2 / 2 / 1 Регистрация: 16.10.2009 Сообщений: 112
	20.10.2014, 11:41 [ТС]	8
	Всем спасибо! Век живи- век учись. 0