Cita:
Iniciado por cristo995 
o. si me sirvio retornando true.
gracias: carlos_belisario funciona 100% :) :)
ya me estaba desesperando.
pero entonces la parte de return true;
es como si fuera la parte contraria del codigo cierto..?
o esq de alguna u otra forma va return true; ??
no me queda muy claro...
Te pongo una variante
Código Javascript
:
Ver original<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<script type="text/javascript">
//<![CDATA[
function Validar_CampoVacio(){
var mensaje = "";
var mensaje_tel = "";
var params = Validar_CampoVacio.arguments;
for (var i = 0; i < params.length; i++){
if(document.getElementById(params[i]).value == ""){
mensaje += "Debe rellenar el campo " + (params[i]) + "\n\r";
}
}
var campoTel = document.getElementById('telefono');
if(!validarNumero(campoTel.value)){
if(campoTel.value != ""){
mensaje_tel += "Telefono debe ser un número";
}
}
// para sacar todos los mensajes juntos
if((mensaje != "")||(mensaje_tel != "")){
alert(mensaje + mensaje_tel);
return false; // recién aqui devolvés false
}
}
function validarNumero(input){
return (!isNaN(input)&&parseInt(input)==input);
}
//]]>
</script>
<title></title>
</head>
<body>
<form name="prueba" action="test.php" onsubmit="return Validar_CampoVacio('nombre', 'apellido','telefono')" id="prueba">
Nombre:<input type="text" name="nombre" id='nombre' value="" /><br />
<br />
Apellido:<input type="text" name="apellido" id="apellido" value="" /><br />
<br />
Telefono:<input type="text" name="telefono" id="telefono" value="" /><br />
<br />
<button type="submit">enviar formulario</button></form>
</body>
</html>
Primero un par de correciones sobre tu código original, en
getElementById('telefono).value falta el cierre de la comilla en teléfono
ya que le puiste id a nombre y telef, ponéselo a apellido
Cuando llamás una función con return, el evento espera una respuesta de dicha función, en el caso del onsubmit (que implica justamente eso, hacer un submit, y el button dentro de un form efectivamente los hace), de no recibir un false, la acción del submit sigue su curso natural y dirije al action (que digamos, tampoco fué definido en el form).
Luego podés sacar un mensaje individualmente, o, generar un mensaje en el error y posteriormente evaluar si el mensaje no es = ""; y sacar el mensaje de error trás hacer todas las validaciones (que incluso podrían ser más)
Otro detalle, en lo propuesto pos Carlos, un simple isNaN() te daría como falso tanto para un entero como para un float, pero intuyo que por ser un telefono solo querés enteros, de ahi que yo usé otra función para evaluar eso.
Finalmente, deberías mejorar la validación de campos vacios, ya que una sucesión de espacios en blanco con la barra espaciadora no es una cadena vacía precisamente.
Hay otras variantes, pero esta que puse es lo que m´´as se asemeja a la que hiciste.
Saludos