Как мне заставить скрипт проверки формы работать с массивом? - PullRequest
0 голосов
/ 24 сентября 2011

Я пытаюсь реализовать скрипт проверки формы JavaScript из Javascript-Coder.com, который можно найти здесь .

У меня он работает для элементов формы, но яинтересно, как заставить его работать с массивом.В частности, у меня есть форма на веб-странице здесь , к которой пользователь может добавлять строки.Тогда у меня есть следующая форма:

<form method="post" name="booking" id="booking" action="bookingengine.php">
    <fieldset>
        <h2>Waged/Organisation Rate</h2>
       <p>
            <input type="text" name="name[]" id="name">
            <input type="text" name="email[]" id="email">
            <input type="text" name="organisation[]" id="organisation">
            <input type="text" name="position[]" id="position">
        </p>
        <p><span class="add">Add person</span></p>
    </fieldset>

    <fieldset>
        <h2>Unwaged Rate</h2>
        <p>
            <input type="text" name="name2[]" id="name2">
            <input type="text" name="email2[]" id="email2">
        </p>
        <p><span class="add">Add person</span></p>
    </fieldset>

    <p><input type="submit" name="submit" id="submit" value="Submit and proceed to payment page" class="submit-button" /></p>

</form>

В настоящее время скрипт проверки формы выглядит следующим образом:

<script  language="JavaScript" type="text/javascript">
 var frmvalidator = new Validator("booking");

 frmvalidator.addValidation("email[]","req","Please enter a valid email address");
 frmvalidator.addValidation("email[]","email","Please enter a valid email address");

</script>

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

ДОПОЛНИТЕЛЬНАЯ ИНФОРМАЦИЯ

По совету Melsi скрипт для генерации формы и обработки валидации был полностью переписан.Ответ Мелси, представленный ниже, включает в себя следующие функции, которые я запросил (большинство из которых были также в исходном скрипте):

  1. Форма теперь пуста при загрузке страницы, и все строки (текстовые поля)динамически добавляется пользователем с помощью кнопок.
  2. Значения по умолчанию для каждого текстового поля показывают, когда новая строка добавляется с уникальным цветом.
  3. Когда пользователь нажимает в каждом текстовом поле цвет дляизменения текста и фона.
  4. Кнопка «Удалить» добавлена ​​в конец каждой строки, чтобы можно было удалять строки.

ВАЛИДАЦИЯ НЕОБХОДИМА

Для каждой строки необходимы следующие проверки:

  1. Имя : не «Имя» (сообщение: «Пожалуйста, введите ваше имя»), не более 100 символов(Сообщение: «Ваше имя должно содержать не более 100 символов»)
  2. E-mail : действительный адрес электронной почты (Сообщение: «Пожалуйста, введите действительный адрес электронной почты»), не более 100 символов (Сообщение: «Ваш адрес электронной почты должен быть менее 100 чараcters ")
  3. Позиция : не" Позиция "(сообщение:" Пожалуйста, введите вашу позицию или N / A, если не применимо "), не более 100 символов (Сообщение:" Ваша позиция должнабыть не более 100 символов ")
  4. Организация : не" Организация "(сообщение:" Пожалуйста, введите вашу организацию или N / A, если не применимо "), не более 100 символов (Сообщение:«Ваша организация должна содержать не более 100 символов»)

Тогда мне потребуется подтверждение для отправки формы, которая проверяет, что строка была добавлена ​​в форму, с сообщением «Пожалуйста, добавьте хотя быодин человек в вашем бронировании "

Пример проверки:

//validate-name
box=document.getElementById(caller).name.charAt(0);
if(box=='n'){
if((document.getElementById(caller).value)=='Name')
    { 
        alert('Please enter your name')
        document.getElementById('message').innerHTML="<span>Please enter your name</span>";
        //put focus back again if you like
        document.getElementById(caller).focus();
        return;
    }   
}           
//if code comes here = validation success   
  document.getElementById(caller).style.backgroundColor = '#F5FEC1'; 
  document.getElementById('message').innerHTML="<span style="+dbq+"background-color: #A3BB00"+ dbq+">Thanks!</span>"; 
 }

Ответы [ 2 ]

1 голос
/ 24 сентября 2011

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

========== отредактированная часть, здесь был какой-то код, который был заменен лучшей версией =====

Я написал этот код в спешке, пример цвета применяется, новая строкаНапример, удаление также добавлено, пустое поле в фокусе также применяется и все остальные вопросы.

<html>
<head>
<script type="text/javascript">

/**
A dynamic name is assigned to a new field created.
*/ 
var id=0;
var dbq="\"";

/************************* addRow function end ******************************************/
 function addRow(count)
 { 
 /**
 Decide what fieldset is going to host the row
 */
    var myFieldset='fieldset2';
    var section='2';
    if(count==4){
        myFieldset='fieldset1';
        var organisationID = id++;
        var positionID = id++;
       var section=''
        }  
 /**
    Create ids
 */
    divID = id++;
    nameID = id++;
    emailID = id++;  
 /**
    The row will be hosted in a div
 */
    var myDiv = document.createElement("div"); 
         myDiv.setAttribute("id", divID);  
 /**
    Create the text boxes
 */
    myDivInnerHTML=
    '<input type=text name=name'+section+'[]'+' value=Name id='+nameID+
    ' onFocus='+dbq+'emptyTheBox('+nameID+');'+dbq+ 
    ' onkeyup='+dbq+'changeInputColor('+nameID+');'+dbq+ 
    ' onBlur='+dbq+'fieldValidator('+nameID+');'+dbq+
    ' style='+dbq+'color:#66634F;'+dbq+' >'+   

    '<input type=text name=email'+section+'[]'+' value=Email id='+emailID+
    ' onFocus='+dbq+'emptyTheBox('+emailID+');'+dbq+
    ' onkeyup='+dbq+'changeInputColor('+emailID+');'+dbq+    
    ' onBlur='+dbq+'fieldValidator('+emailID+');'+dbq+  
    ' style='+dbq+'color:#66634F;'+dbq+'>' ;
 /**
    Decide if we need 4 or 2 boxes
 */
    if(count==4)
    myDivInnerHTML=myDivInnerHTML+
    '<input type=text name=organisation'+section+'[]'+' value=Organisation id='+organisationID+
    ' onFocus='+dbq+'emptyTheBox('+organisationID+');'+dbq+
    ' onkeyup='+dbq+'changeInputColor('+organisationID+');'+dbq+ 
    ' onBlur='+dbq+'fieldValidator('+organisationID+');'+dbq+  
    ' style='+dbq+'color:#66634F;'+dbq+' >'+   

    '<input type=text name=position'+section+'[]'+' value=Position id='+positionID+
    ' onFocus='+dbq+'emptyTheBox('+positionID+');'+dbq+
    ' onkeyup='+dbq+'changeInputColor('+positionID+');'+dbq+ 
    ' onBlur='+dbq+'fieldValidator('+positionID+');'+dbq+ 
    ' style='+dbq+'color:#66634F'+dbq+'>' ;
 /**
    Create a button to remove the row too.
 */
    myDivInnerHTML=myDivInnerHTML+
    '<input type=button class="remove" value="Remove"  onClick='+dbq+'removeDiv('+divID+','+ myFieldset +');'+dbq+' >';   
 /**
    Add the div-row to the fieldset
 */
    myDiv.innerHTML = myDivInnerHTML; 
    document.getElementById(myFieldset).appendChild(myDiv);      
 }
/************************* addRow function end ******************************************/

 /**
    Change the color of the text being entered
 */
function changeInputColor(caller){   
    document.getElementById(caller).style.color = 'black';
}

 /**
    Remove a row on demand
 */
function removeDiv(divID, myFieldset){ 
    myFieldset.removeChild(document.getElementById(divID)); 
}

 /**
    Empty the box on initial click
 */
function emptyTheBox(caller)
{
    var val=document.getElementById(caller).value;
    if(val=='Name' || val=='Email' || val=='Organisation' || val=='Position'  ) 
        document.getElementById(caller).value='';  
 }
/**
    Display a message
 */
 function echo(message)
 {
        document.getElementById('message').innerHTML="<h3>"+message+"</h3>";   
 }


/**********************Validates a single field, return false on fail************************/
function fieldValidator(caller)
 {  
    var error='';
    /** 
        Identify the field (if it is email, name etc) by getting the first character
        which is always the same,also get its value and full name
    */  
    var currentFieldCategory = document.getElementById(caller).name.charAt(0);
    var currentFieldValue = document.getElementById(caller).value; 
    var currentField = document.getElementById(caller);

    /** 
        Check for empty value
    */
     if(currentFieldValue == '')
     {
        echo('Please fill the data!');currentField.focus(); 
        return 'Please fill the data!';
     }  
    /** 
        Check if default value left behind  
    */  
    if(currentFieldValue.toLowerCase()=="name" || currentFieldValue.toLowerCase()
    =="email" || currentFieldValue.toLowerCase()=="organisation" ||
    currentFieldValue.toLowerCase()=="position" )  
    { 
        echo('Please check you entry, default data left behind!');currentField.focus();   
        return 'Please check you entry, default data left behind!';
    }    
    /** 
        Validate the NAME field
    */   
    if(currentFieldCategory=='n')
    {
        if(currentFieldValue.match(/^[ |'|-]/)||!(/^[a-zA-Z- ']*$/.test(currentFieldValue))
            || currentFieldValue.length<4 || currentFieldValue.length>70)  
        {
            echo('Non valid name found!');currentField.focus();   
            return 'Non valid name found!';
        }//inner if
    }//outer if
    /** 
        Validate a non empty EMAIL name field
    */  
    if(currentFieldCategory=='e')
    {
        var atpos=currentFieldValue.indexOf("@");
        var dotpos=currentFieldValue.lastIndexOf(".");
        if (atpos<1 || dotpos<atpos+2 || dotpos+2>=currentFieldValue.length) 
        {
            echo('Non valid email found!');currentField.focus();   
            return 'Non valid email found!';
        }//inner if 
    }//outer if
    /** 
        Validate a non empty ORGANIZATION name field
    */  
    if(currentFieldCategory=='o')
    {
        if(currentFieldValue.length<2 || currentFieldValue.length>50)
        {
            echo('Use at least 2 letters and less than 50 for your organisation.'); 
            currentField.focus();           
            return 'Use at least 2 letters and less than 50 for your organisation.';        
        }//inner if  
    }//outer if 
    /** 
        Validate a non empty POSITON name field
    */  
    if(currentFieldCategory=='p')
    {
        if(currentFieldValue.length<7 || currentFieldValue.length>40)
        {
            echo('Use at least 7 letters and less than 40 to describe your position.');
            currentField.focus(); 
            return 'Use at least 7 letters and less than 40 to describe your position.';            
        }//inner if 
    }//outer if     
    /** 
        Now on success do the rest
    */  
    document.getElementById(caller).style.backgroundColor = '#FF9900';  
    document.getElementById('message').innerHTML="";
    return true;
 }
 /*****************fieldValidator ***function ends*****************************************/



/*******************************************************************************************/
function finalValidator()
{
    /**
        Get the form object
    */  
    var myForm=document.getElementById('booking').elements;
    /**
        Check if the form has no rows, for now 3 indicates no rows,
        BE CAREFULL it might change if more buttons added,
        just alert the length to see.
    */  
    if(myForm.length==3)
        return false; 
    /**
        Iterate through the form for all fields
    */  
    for(var i = 0; i < myForm.length; i++)
        {   
            //If it is a text field validate it
            if(myForm[i].type=='text')
            {
                var validation = fieldValidator(myForm[i].id);
                if(validation!==true)
                {
                    echo (validation);
                    return false;//prevent submit
                }//validation if
            }//field type if 
        }//for loop   
}//function
/*******************************************************************************************/


</script> 

</head>
<body bgcolor=gray>

<div id="add-buttons"><span class="add" onclick="addRow(4);"><input type="button" class="button" value="Add Waged Person"></span><span class="add" onclick="addRow(2);"><input type="button" class="button" value="Add Unwaged Person"></span></div>


<div id="message"  ></div>
<div id="form-wrap">
<form method="post" name="booking" id="booking" action="bookingengine.php">
    <fieldset id="fieldset1">
        <div class="subtitle">Waged/Organisation Rate</div>
    </fieldset> 

    <fieldset  id="fieldset2">
        <div class="subtitle">Unwaged Rate</div>
    </fieldset>

   <!-- return finalValidator will allow submit only if fields are validated-->
    <p><input type="submit" name="submit" id="submit"  onClick="return finalValidator();" 
     value="Submit booking" class="submit-button" /></p>
</form> 


</body>
</html>

Проверка добавлена.Массив и цветные вещи немного тривиальны, и вы должны показать, какие усилия вы приложили здесь сами. Интересно участвовать, только когда вы видите желание кого-то.

0 голосов
/ 01 октября 2011

Я решил добавить другой ответ, потому что есть некоторые другие дополнительные вещи, которые я хочу упомянуть об отладке.

Что касается ошибки, о которой вы упоминаете, вызванной этой строкой:

<div id="booking" style="font-size:16px;margin-bottom:10px;padding:5px">Please add the number of people 

Почему это произошло?

Если вы присмотритесь, то вам будет присвоен идентификатор бронирования, но у вас также есть HTMLФорма назначается с тем же идентификатором, это создает конфликт, так как JavaScript не знает, какой из них вы имеете в виду.


Советы

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

- Вы упомянули конкретную функцию, создающую проблему (и вы были правы), затем вам нужно было зайти внутрь функции и поставить одно предупреждение («Привет, что угодно!»);в строке за один раз, начиная с самого верха, как только предупреждение не появится, вы узнаете, в какой строке возникла проблема.Затем вы предупреждаете переменные в этой строке, чтобы увидеть, имеют ли они значение, которое они должны иметь ... скорее всего, значение не будет иметь значения, которое должно иметь, и ваша проблема уже стала очень маленькой.

Это пока, позаботьтесь об идентификаторах, они должны быть уникальными!Если вы видите мой пример, я назначаю им увеличивающийся индекс в конце

id="someElement"+(id++);

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

...