Validación de formularios

[roal40]

Buenas, necesito ayuda, estoy intentando hacer que se validen unos campso de texto de un formulario, pero segun lo que elijas, hay tal cantidad de forms, el problema es que tengo este codigo:

Código HTML:

<script type="text/javascript">
//funcion que es llamada en el evento submit del formulario
function fntValidar(){
	//obtener el formulario para utilizarlo en la validacion
	var frmFormulario=document.forms['formcontacto'];
	//contar la cantidad de elementos que contiene el formulario
	var iElementos=frmFormulario.elements.length;
			
	//recorrer todos los elementos del formulario
	for(var iCont=0;iCont<iElementos;iCont++){
		//obtener el elemento actual para utilizarlo
		var objElemento=frmFormulario.elements[iCont];
				
		//validar unicamente los elementos del tipo "text" (campos de texto)
		if(objElemento.type=='text'){
			if(objElemento.style.display=="block"){
			/*estamos utilizando la funcion trim (funcion no propia de JavaScript)
			para eliminar los espacios en blanco al inicio y final de una cadena*/
			if(trim(objElemento.value)==''){
				//mostramos un mensaje al usuario
				alert('Por favor, complete todos los campos del formulario.');
				//enfocamos el campo que exta vacio
				objElemento.focus();
				//borramos el contenido del campo (podria contener espacios en blanco)
				objElemento.value='';
				//devolvemos false para que el formulario no sea procesado
				return false;
			}
			
		}
		}
	}
			
	//llegamos hasta aqui solo en caso de que todos los campos no esten vacios
	//le preguntamos al usuario si desea almacenar los datos
	if(!confirm('�Desea almacenar los datos actuales?')){
		//el usuario indica que no desea almacenar los datos
		//entonces devolvemos false para que el formulario no sea procesado
		return false;
	}
}
</script> 

Como ven he añadido la linea:

Código HTML:

		if(objElemento.type=='text'){
			if(objElemento.style.display=="block"){

El problema es que siempre me sale como correcto, no se que hacer, ayuda porfavor

[djaevi]

creo que el problema esta en el if (objElemento.style.display == "block") generalmente si mal no recuerdo para acceder a esta propiedad (sin usar jQuery) debes haberla seteado antes de hacer la consulta x ej:

Esto no deberia funcionar:

Código Javascript:

var miDiv = document.getElementById("idElemento");
if (miDiv.style.display == "block") // Resto del codigo
 

Esto puede ser que funcione: (no estoy seguro :P)

Código Javascript:

var miDiv = document.getElementById("idElemento");
miDiv.style.display = "block";
if (miDiv.style.display == "block") // Resto del codigo
 

Con jQuery puedes consultar la propiedad sin declararla previamente:

Código Javascript:

if ($("#idElemento").css("display") == "block") // Resto del codigo
 

Saludos

[roal40]

Ya nada, lo acabé averiguando yo jajaja

[emprear]

Iniciado por roal40

Ya nada, lo acabé averiguando yo jajaja

No le veo la gracia, en todo caso agradecé a quien intentó darte una respuesta y mostrá como lo resolviste en beneficio de otros usuarios