Opinion validación javascript

[Pantaláimon]

Todo este código sigue la misma estructura:

Código Javascript:

// name - namebox
    if(document.Booking.name.value == '')
    {
        divresult = document.getElementById('namebox')
        divresult.className = 'validation'
        divresult.innerHTML = 'Please fill in name';
        document.Booking.name.focus();
        return false;
    }
    // passport - passportbox
    if(document.Booking.passport.value == '')
    {
        divresult = document.getElementById('passportbox')
        divresult.className = 'validation'
        divresult.innerHTML = 'Please fill in your identification';
        document.Booking.passport.focus();
        return false;
    }
    // emailtrue - emailtruebox
    if(document.Booking.emailtrue.value == '')
    {
        divresult = document.getElementById('emailtruebox');
        divresult.className = 'validation'
        divresult.innerHTML = 'Please fill in email';
        document.Booking.emailtrue.focus();
        return false;
    }
    // repeat_email - repeat_emailbox
    if(document.Booking.repeat_email.value == '')
    {
       divresult = document.getElementById('repeat_emailbox');
       divresult.className = 'validation'
       divresult.innerHTML = 'Please repeat email';
       document.Booking.repeat_email.focus();
       return false;
    }
    // telmobile - telmobilebox
    if(document.Booking.telmobile.value == '')
    {
       divresult = document.getElementById('telmobilebox');
       divresult.className = 'validation'
       divresult.innerHTML = 'Please fill in mobile to bring on holiday';
       document.Booking.telmobile.focus();
       return false;
    }
    // test - resultbooking 
    if(document.Booking.test.value =='')
    {
        divresult = document.getElementById('resultbooking');
        divresult.className = 'validation'
        divresult.innerHTML = 'Please answer the security question';
        document.Booking.test.focus();
        return false;
    }
 

Intenta usar bucles cuando te ocurra esto para no escribir tanto. A la vez que luego al cambiar el código HTML te será más rápido cambiar el código javascript:

Código Javascript:

var inputs = {
        name        : "namebox",
        passport    : "passportbox",
        emailtrue   : "emailtruebox",
        repeat_email: "repeat_emailbox",
        telmobile   : "telmobilebox",
        test        : "resultbooking"
    }
    for( key in inputs ) {
        if(document.Booking[key].value == '')
        {
            divresult = document.getElementById(inputs[key])
            divresult.className = 'validation'
            divresult.innerHTML = 'Please fill in name';
            document.Booking[key].focus();
            return false;
        }
    }
 

Por otro lado, ¿return false es para evitar el submit pot defecto? Lo digo porque "return false" está en desuso. Deberías usar el método "preventDefault". Encontrarás información sobre este en tu buscador favorito.

Un saludo!