На что именно ругается браузер в коде?

Question

[Антон]

<input type="radio" data-js-action="switch-type" 
class="radio-d" onchange="fun3()" value="Сферический" 
name="DESIGN" id="dvd2"/>

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

<script>

function fun1() {
var chbox;
chbox=document.getElementById('tab5');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun2() {
var chbox;
chbox=document.getElementById('dvd1');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun3() {
var chbox;
chbox=document.getElementById('dvd2');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun4() {
var chbox;
chbox=document.getElementById('dvd3');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
function fun5() {
var chbox;
chbox=document.getElementById('dvd4');
  if (chbox.checked) {
    document.querySelector(".next-slide-text").click();
  }
}
    </script>

Comments

[Максим Федоров]

ругается браузер

ваш код не похож на текст ругани :):):)

[Антон]

Максим Федоров, (index):135 Uncaught ReferenceError: fun3 is not defined
at HTMLInputElement.onchange ((index):135)

[Алексей]

и как он ругается? или там нецензурно, потому не публикуется?

[Антон]

(index):135 Uncaught ReferenceError: fun3 is not defined
at HTMLInputElement.onchange ((index):135)

[Алексей]

anton99zel, ну а сама функция fun3() где? в этом же файле или в подключаемом js-файле?

[Алексей]

anton99zel, а можно второй посмотреть как оформлен второй <input type="radio">.
Просто так обычно не делают, потому и не совсем понятно где может быть ошибка.

[Антон]

Алексей,

<input type="radio"  data-js-action="switch-type"     class="radio-d" onchange="fun2()" value="Асферический" name="DESIGN" id="dvd1"/><label for="dvd1"><img src="/upload/d1.png" class="notChecked"><img src="/upload/d12.png" class="ifChecked"></label>
<input type="radio"   data-js-action="switch-type"   class="radio-d" onchange="fun3()" value="Сферический" name="DESIGN" id="dvd2"/><label for="dvd2"><img src="/upload/d2.png" class="notChecked"><img src="/upload/d22.png" class="ifChecked"></label>
<input type="radio"   data-js-action="switch-type"    class="radio-d" onchange="fun4()" value="Офисные" name="DESIGN" id="dvd3"/><label for="dvd3"><img src="/upload/d3.png" class="notChecked"><img src="/upload/d32.png" class="ifChecked"></label>
<input type="radio"  data-js-action="switch-type"    class="radio-d" onchange="fun5()" value="Прогрессивные" name="DESIGN" id="dvd4"/><label for="dvd4"><img src="/upload/d4.png" class="notChecked"><img src="/upload/d42.png" class="ifChecked"></label>

[Антон]

интересно, почему под админом все ок, а под неавториз.пользователем ошибки

[Алексей]

anton99zel,
В общем суть в чем. Событие onchange обычно вешается на select или на text, textareа - это понятно, было одно значение, стало другое значение.
Но если мы говорим о radio, то это группа полей объединенных одним именем name="DESIGN".
Потому событие onchange может некорретно работать в некоторых браузерах именно потому, что допустим у вас был выделен первый элемент, и вы нажали на второй. Как должно произойти событие onchange ? у первого и второго одновременно? то есть у первого снялось выделение, у второго появилось.
Попробуйте в ваших функциях поставить просто

function fun1() {
alert('fun1');
}
function fun2() {
alert('fun2');
}
function fun3() {
alert('fun3');
}

Посмотрите что будет происходить.
Но говорю, что обычно делают по другому - вешают общий обработчик событий на группу name="DESIGN"

[Антон]

Алексей, подскажи пожалуйста, а мог быть глюк в этом:
я почистил код от комментов и все заработало, вот код был

<script>
        $(function () {
            $('#form_id').submit(function(e){
                e.preventDefault();//отменяем действие по умолчанию(отправка формы)
                var data = $(this).serialize();// получаем данные из формы
                $. ajax ({
                    url: 'ajax .php',//адрес обработчика
                    type: 'post',//метод передачи данных
                    data:data,
                    success:function(res){
                        $('#result').html(res);//выводим ответ с сервера в контейнер с id='result'
                    }
                })
            })
        })
    </script>

я сделал так

<script type="text/javascript">
        $(function () {
            $('#form_id').submit(function(e){
                e.preventDefault();
                var data = $(this).serialize();
                $. ajax ({
                    url: 'ajax.php',
                    type: 'post',
                    data: data,
                    success:function(res){
                        $('#result').html(res);
                    }
                })
            })
        })
    </script>

и еще вопрос - есть разница между тегами
<script> и <script type="text/javascript">
?

[Алексей]

anton99zel,
Использование <script type="text/javascript"> требовалось в html4. Сейчас когда все браузеры поддерживают html5, писать type не нужно. А вы используете именно html5, так как есть атрибуты data-js-action="switch-type"
По идее разницы не должно быть, но возможно, некоторые браузеры находили конфликт когда смешиваются определения для html4 и html5.
Вообще надо смотреть при загрузке страницы в консоль, возможно там возникают ошибки уже на этапе загрузке и инициализации.

Answer 1

[FlamyXD]

По коду что вы дали: потому что нет HTML элемента с классом .next-slide-text, у которого был бы обработчик события click.