Opinion validación javascript

[helenp]

Iniciado por Pantaláimon

Cierto, los innerHTML cambian (que por cierto, si sólo quieres añadir texto usa mejor textContent ). Pero vaya, la idea es la misma, poner la información que cambia en un objeto o array y recorrer la estructura que se repite mediante un bucle:

Gracias, habia una pequeña errata y añadiendo otra validacion en este orden funciona:

Código Javascript:

<script type="text/javascript">
    function OnSubmitBooking()
{
    if(document.Booking.emailtrue.value != document.Booking.repeat_email.value)
 {
divresult = document.getElementById('repeat_emailbox');
divresult.className = 'validation'
divresult.innerHTML = 'Email and repeat email are not equal';
 document.Booking.repeat_email.focus();
        return false;
 }
        var info = {
            name        : ['namebox'        , 'Please fill in name' ],
            passport    : ['passportbox'    , 'Please fill in your identification' ],
            emailtrue   : ['emailtruebox'   , 'Please fill in email' ],
            repeat_email: ['repeat_emailbox', 'Please repeat email' ],
            telmobile   : ['telmobilebox'   , 'Please fill in mobile to bring on holiday' ],
            test        : ['resultbooking'  , 'Please answer the security question' ]
        }
        for( key in info ) {
            if(document.Booking[key].value == '')
            {
                divresult = document.getElementById(info[key][0])
                divresult.className = 'validation'
                divresult.textContent = info[key][1];
                document.Booking[key].focus();
                return false;
            }
        }
}
        </script>
 

La validación de la pregunta de seguridad, creo que le meteré Ajax ya que estoy y queda mejor.
Lo de Preventdeafult lo dejo un rato ya que veo que se ha abierto un debate paralelo.
Antes de este nuevo codigo usaba innerhtml y estaba intentando poner que cuando escribe en el input que esta en focus que se elimine el texto de validación, no es necesario pero queda bien, intentaba meter otra funcion dentro de la funcion usando onchange y poner innerhtml como display none:
document.getElementById("resultado").style.display = "none";
me imagino en este caso sería textContent.display = "none";

Este es el html relevante:

Código HTML:

<form class="instant" name="Booking" id="solicitud" 
action="<?php echo $_SERVER['SCRIPT_NAME'];?>" method="post" onSubmit="return OnSubmitBooking();">
<label for="name">Name:</label><input id="name" name="name" title="Please enter your name" onchange="quitarfocus()" type="text" size="50">
<div id="namebox"></div>
 

Edited, I changed textcontent to innerhtml as innerhtml is supported by all browswers and works, does not have any html:
divresult.innerHTML = info[key][1];
http://www.davidtong.me/innerhtml-in...html-and-text/