моя функция JavaScript не отображает все сообщения проверки - PullRequest
Новая
       10

моя функция JavaScript не отображает все сообщения проверки

1 голос
/ 17 ноября 2011

Так что, если обе проверки работают по отдельности (если я просто включил проверку переключателей в функцию без проверки вращателя, тогда проверка переключателей работает и то же самое наоборот), как получится, когда я добавлю обе проверки в функцию, верхняя проверка, котораяработает ли радио-валидация, но нижняя валидация, которая является валидацией счетчика, не отображает сообщение, когда предполагается? 

Code is below:

        <head>
                <meta http-equiv="Content-Type" content="text/html;charset=utf-8" />
                <title>Create a Session</title>
                        <script type="text/javascript">

    function validation() {
                    var errMsgO = document.getElementById("radioAlert");
                    var btnRadioO = document.getElementsByName("sessionNo");               
                    var isbtnRadioChecked = false;
                    var errQuesMsgO = document.getElementById("numberAlert");
                    var questionNumberO = document.getElementsByName("txtQuestion");

                    for(i=0; i < btnRadioO.length; i++){
                        if(btnRadioO[i].checked){
                            isbtnRadioChecked = true;
                        }
                    }

                    if(!isbtnRadioChecked) {
                       errMsgO.innerHTML = "Please Select the Number of Sessions you Require";
                    } else {
                        errMsgO.innerHTML = "";
                    }
                    return false; 



        if(questionNumberO[0].value == 0){
            errQuesMsgO.innerHTML = "Please Set the Number of Questions";
        } else {
            errQuesMsgO.innerHTML = "";
        }
           return false; 
                }    
</script>
        </head>
        <body>
        <form action="create_session.php" method="post" name="sessionform">
    <table>
              <tr>
              <th>2: Number of Sessions :</th>
              <td class="sessionNo"><input type="radio" name="sessionNo" value="1" />1</td>
              <td class="sessionNo"><input type="radio" name="sessionNo" value="2" />2</td>
              <td class="sessionNo"><input type="radio" name="sessionNo" value="3" />3</td>
              <td class="sessionNo"><input type="radio" name="sessionNo" value="4" />4</td>
              <td class="sessionNo"><input type="radio" name="sessionNo" value="5" />5</td>
              </tr>
              </table>
              <div id="radioAlert"></div>
        <table>
                    <tr>
                    <th>Number of Questions:</th>
                        <td class="spinner"><textarea class="spinnerQuestion" name="txtQuestion" id="txtQuestion" cols="2" rows="1"></textarea></td>
                        <td><button class="scrollBtn" id="btnQuestionUp"><img src="Images/black_uppointing_triangle.png" alt="Increase" /></button>
                        <button class="scrollBtn" id="btnQuestionDown"><img src="Images/black_downpointing_triangle.png" alt="Decrease" /></button></td>
                        </tr>
                        </table>
                        <div id="numberAlert"></div>
                        <p><input class="questionBtn" type="button" value="Prepare Questions" name="prequestion" onClick="validation()" /></p>      <!-- Prepare Questions here-->

                </form>
        </body>

1 Ответ

2 голосов
/ 17 ноября 2011

Единственная ошибка, которую я мог заметить, это строка if(questionNumberO == 0){ (, кроме того, вы не закрыли свой элемент <script> с помощью </script>).

Правильный путьis: 

if(questionNumberO[0].value == 0){

Почему:

getElementsByName() вернет коллекцию элементов с указанным атрибутом name,Если вы знаете, что есть только один, вы можете запросить первый ([0]).

Свойство value хранит текущее значение элемента ввода.См. HTMLInputElement в MDN .

Обновление: Я создал работающую демонстрацию jsFiddle для вас.

...