Hacer una buena comprobacion con javascript

[patilanz]

Hola tengo las 2 siguientes funciones :

Código Javascript:

function comprobar_envio() {
    var n = form.name.value;
    var e = form.email.value;
    var p = form.phone.value;
    var t = form.tema.value;
    var m = form.message.value; 
    var errores;
    if(n.length > 0 && e.length > 0 && t.length > 0 && m.length > 0) {
        if(n.length < 3) {
            errores += '<font size="5" color="red">Името е невалидно!</font><br />';  
            
        }
        if(!isNaN(n)) {
            errores += '<font size="5" color="red">Името не трябва да съдържа цифри!</font><br />'
            var n = true;
        }
        if(comprobar_correo()) {
            errores += '<font size="5" color="red">Имейлът е невалиден!</font><br />';
            var e = true;
        }
        if(p.length > 0) {
            if(!/^([0-9]{3} ){2}([0-9]{3})$/.test(p) || !/^[0-9]{9}$/.test(p) || !/^(([0-9]{3})-){2}([0-9]{3})$/.test(p)) {
                errores += '<font size="5" color="red">Телефонът е невалиден!</font>';
            } 
            
                
            
        }
    } else {
            errores += '<font size="5" color="red">Попълнете всички задължителни полета!</font>';
    }
    return errores;
}
function enviar() {
    if(comprobar_envio()) {
        d = document.getElementById('errores');
        d.style.opacity = '0.9';
        d.style.height = '169px';
        var e =comprobar_envio();
        d.innerHTML = e.substr(9,e.length);
    }   
}
 

El error esta en que no me comprueba bien el error en comprobar el telefono con las expresiones regulares.
Y luego cuando el error ya no existe se queda alli y no desaparece ...
Alguna forma de solucionarlo ???

Las respuestas están en otro idioma pero esto creo que no sera algún problema...

Gracias :)

[dontexplain]

Tienes un error de lógica en el if

según lo que tienes establecido:

si no coincide con la primera regexp o si no coincide con la segunda o si no coincide con la tercera da error, eso significa que tendrá que coincidir con las tres a la vez para que no se ejecute el error, o sea, que el teléfono mágicamente sea de las tres formas a la vez, eso es imposible; para que eso tenga lógica sustituye los OR (||) por AND (&&)

!/^([0-9]{3} ){2}([0-9]{3})$/.test(p) && !/^[0-9]{9}$/.test(p) && !/^(([0-9]{3})-){2}([0-9]{3})$/.test(p)

Un saludo.