No me reconoce las funciones de un archivo externo

[rvelezb]

Hola, tengo varias funciones en un archivo externo con extensión ".js", y en mi página tengo puesto de donde lo tiene que coger así:

Código HTML:

<script src="validar.js" type="text/javascipt"></script> 

Pero más adelante cuando llamo a alguna de las funciones que tengo en ese archivo me da error, bueno, corrijo, no da error, simplemente no hace lo que tiene que hacer, lo de error lo he descubierto yo gracias a un inspector de código de esos, que me dice: comprobarEmail is not defined, comprobarNumero is not defined, y validarFormulario is not defined, ¿qué es lo que hago mal?, ¿no está bien puesto para que incluya el código de ese archivo?.

Aquí están las llamadas a las distintas funciones JavaScript,

Código HTML:

<input type="text" name="telefono" size="20" maxlength="12" onblur="comprobarNumero(this)" />
<input type="text" name="email" size="30" maxlength="50" onblur="comprobarEmail(this)" />
<input type="button" name="btn_aceptar" value="Enviar datos" onclick="validarFormulario()"/> 

Y en el archivo javascript con las funciones tengo esto:

Código:

// JavaScript Document
function validarFormulario()
{
	campos = new Array("nombre", "apellidos", "telefono");  
	for(i=0; i<campos.length; i++)
	{
		obj = eval("document.f." + campos[i]);
		if (obj.value == "")
		{
			alert ("Ha de rellenar el campo ["+obj.name+"]");
			obj.focus();
			return;
		}
	}
	document.f.submit();
}
function comprobarNumero(obj)
{
	if (obj.value != "")
		if (isNaN(obj.value))
		{
			alert("Debe introducir un número en el campo " + obj.name);
			obj.focus();
			return;
		}
}
function comprobarEmail(obj)
{
       var hayAlgo = true;
	if (obj.value =="")
		return true;
	else
		obj.value = obj.value.toLowerCase(); // pasamos el campo a minúsculas

	// validar la cuenta de correo usando una expresión regular (RegExp)
	if (obj.value.search(/^[\w-\.]+@([\w-]+\.)+[\w-]{2,4}$/ig))
       {
		hayAlgo = false;
		alert ("La cuenta no es válida, debes escribirla de forma: [email protected]");
		obj.focus();
		return false;
	}
	return true;
}

[Adler]

Hola

¿Están en el misma carpeta el .js y su llamada?

Suerte

[rvelezb]

Sí, están en la misma carpeta ambos.

[Adler]

Ok, el metodo onblur es un poco incomodo para el usuario, opinión personal, así que por que no validas todo OnSubmit

Intentando solucionar tu problema añade esto

return comprobarNumero(this)
return comprobarEmail(this)
return validarFormulario()

Suerte

[Un_Tico]

Hola rvelezb, el problema es que tienes errores en el archivo js con tus funciones por lo que el script no va ha funcionar, por pro otro lado te recomiendo no usar la funcion eval a menos que fuera necesario(es este punto tienes el error), aca te dejo una recomendacion de codigo para esa funcion

Código:

function validarFormulario()
{
	campos = new Array("nombre", "apellidos", "telefono");  	
	for(var i=0;i<document.f.elements.length;i++){
		if(document.f.elements[i].type=='text'){
			if(document.f.elements[i].value==''){
				alert ("Ha de rellenar el campo ["+document.f.elements[i].name+"]");
				document.f.elements[i].focus();
				return;	
			}
		}
		
	}
	document.f.submit();
}

Me parece un poco mas sencilla y la puedes usar para validar campos de texto extra...

con la llamada al archivo no hay nada de malo, si aun no usas firefox para trabajar con tus scripts te recomiendo que lo uses junto con un complemento par de complementos de desarrollador(jsview,web developer tool bar) se que te seran utiles..

Saludos y suerte

[rvelezb]

¿Eso dónde lo pongo?, ¿cómo retorno de cada una de las funciones de JavaScript?.

[rvelezb]

Iniciado por Un_Tico

Hola rvelezb, el problema es que tienes errores en el archivo js con tus funciones por lo que el script no va ha funcionar, por pro otro lado te recomiendo no usar la funcion eval a menos que fuera necesario(es este punto tienes el error), aca te dejo una recomendacion de codigo para esa funcion

Código:

function validarFormulario()
{
	campos = new Array("nombre", "apellidos", "telefono");  	
	for(var i=0;i<document.f.elements.length;i++){
		if(document.f.elements[i].type=='text'){
			if(document.f.elements[i].value==''){
				alert ("Ha de rellenar el campo ["+document.f.elements[i].name+"]");
				document.f.elements[i].focus();
				return;	
			}
		}
		
	}
	document.f.submit();
}

Me parece un poco mas sencilla y la puedes usar para validar campos de texto extra...

con la llamada al archivo no hay nada de malo, si aun no usas firefox para trabajar con tus scripts te recomiendo que lo uses junto con un complemento par de complementos de desarrollador(jsview,web developer tool bar) se que te seran utiles..

Saludos y suerte

Es raro que tenga errores ahí en el código de las funciones JScript, ya que el código ese puesto dentro de una página, en vez de en un archivo aparte funciona bien, no da ningún tipo de problema.

Sí, utilizo el Firefox con la barra de herramientas de Web Developer y además tengo instalada la extensión Firebug.

Un saludo.

[rvelezb]

Vale, ya sé donde estaba el fallo, era una tontería, estaba aquí:

Código:

<script src="validar.js" type="text/javascipt"></script>

En el tipo de archivo, puse "javascipt", en vez de javascript, me había comido a 'r'.