Ошибка JavaScript при проверке наличия двух числовых значений

Question

Надеюсь, я получу этот формат. Я знаю, что это вопрос новичков и, вероятно, довольно очевидный, но я смущен тем, как проверить эти поля. У меня есть два поля ввода на файл JSP:

<input id="CMDScheduleNumber" type="number" class="textsmall" maxlength="5" 
     onKeyPress="return numbersonly(this, event)"/> 

<input id="CMDContractYear" type="number" class="textsmall" maxlength="4" 
     onKeyPress="return numbersonly(this, event)"/> 

У меня есть функция в сценарии под названием «searchEFT», который проверяет, если либо номер графика или год контракта заполняется, то оба они должны быть заполнены.

 <script type="text/javascript"> 
      //function for onchange  
      $(document).ready(function() { 
       $("#searchEFT").click(function() { 

        var Cmd_Sched_Number = document.getElementById("CMDScheduleNumber"); 
        var Cmd_Contract_Year = document.getElementById("CMDContractYear"); 
        var Cmd_Status = document.getElementById("CMDSchedStatus"); 
        var Cmd_Creation_Date = ocument.getElementById("CMDCreationDate"); 

        if (Cmd_Sched_Number == "") { 
         If(Cmd_Contract_Year !== "") 
         alert("Schedule Number and EFT Contract Year must be both populated"); 
         return; 
        } 
        else if (Cmd_Sched_Number == "") { 
         alert("Schedule Number and EFT Contract year must be both populated"); 
         return; 
        } 

Когда я попытался сделать отладчик, если поле Cmd_Sched_Number значение отображается как «», но valueasnumber показан как «NaN». Поэтому, когда я делаю чек, я должен проверить, что это «" или проверить его как числовое с isNaN и/или IsNull?

Спасибо за помощь

Answer 1

var Cmd_Sched_Number = document.getElementById("CMDScheduleNumber"); Получает элемент.

Использование .value получить value от Element

Что-то вроде:

var Cmd_Sched_Number = document.getElementById("CMDScheduleNumber").value; 

Кроме того, поскольку у вас есть JQuery уже, рассмотреть возможность использования его. как:

var Cmd_Sched_Number = $("CMDScheduleNumber").val(); 

Answer 2

Пользовательские валидация кода действительно беспорядок. Сколько условий вы можете проверить? Существует множество библиотек с открытым исходным кодом, и они отлично справляются с этой задачей.

Я бы порекомендовал вас использовать validate.js. Его очень простой и простой в использовании. Он устанавливает правила в полях и проверяет их.

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

Answer 3

Как правильно указал Арагорн, убедитесь, что вы получаете значения, а не объекты JQuery или элементы DOM.

function isPopulated(val) { 
    return !(val === '' || isNaN(val)); 
} 

//and then in your click event handler...: 

if((isPopulated(Cmd_Sched_Number) || isPopulated(Cmd_Contract_Year)) && !(isPopulated(Cmd_Sched_Number) && isPopulated(Cmd_Contract_Year))) { 
    //Handle the case where one is populated and the other isn't, assuming you want to treat any non-numbers as not populated. 
} 

Это если вы хотите общий блок для любого сценария одного заселенных и другого нет, она будет оцениваться как XOR.

Причина, по которой моя функция isPopulated проверяет как пустую строку, так и isNaN - это isNaN ('') будет оцениваться как false.

Если вам не важно, действительно ли введенное значение является числовым или нет, тогда вы, возможно, захотите проверить значение.length> 0, например.