Validador de montos

[Spychaser]

Buenas, antes que nada, queiro agradecer a Tunait por su javascript validador. Intente agregarle unas cosas mas que necesitaba en mi web, pero me ocurre que funciona correctamente pero en el caso de haber ingresado mal los montos luego siempre me dice que esta mal. Es como que no se vacia la variable.
Muchas gracias, aca va el codigo:

Código:

<script language="JavaScript">
//Script verificador de formularios por Tunait!
//Los campos obligados deben terminar con un *
//No verifica los checkbox ni los radio

function verificarFormu(cualo)
{
for(m=0;m<cualo.length;m++)
	{
	if(cualo.elements[m].id.charAt(cualo.elements[m].id.length-1) == "*")
	{
		if(cualo.elements[m].value=="")
			{
			alert("Debe poner un " + cualo.elements[m].id.substring(0,cualo.elements[m].id.length-1))
			cualo.elements[m].focus()
			return false
			break;
			}
	
		if(cualo.elements[m].id=="E-Mail" || cualo.elements[m].id=="E-Mail*")
			{
			pat=/^[\w\_\.çñ]{2,255}@[\w]{2,255}\.[a-z]{1,3}\.?[a-z]{0,3}$/
			if(!pat.test(cualo.elements[m].value))
				{
				alert("Introduzca un E-mail válido")
				cualo.elements[m].focus()
				return false
				}
		
			}
			if (cualo.elements[m].id=="Monto Min.*")
            {
             if(cualo.elements[m+1].value < cualo.elements[m].value)
               {
                 alert("El monto MAXIMO debe ser mayor al monto MINIMO")
                 cualo.elements[m].focus()
                
                return false
                break;
			   }
            }
		}
	else
	{
	continue;
	}
	}
	

}
</script>

[tunait]

Hola Spychaser

déjame ver el código del form, si es muy largo ponme sólo la parte de los campos de los montos a comparar

[Spychaser]

Aca esta una parte de mi form.
Utilizo

Código:

onKeyPress="return validar(event)"

para que solo ingresen numeros.
VALIDADOR NUMEROS:

Código:

<script type="text/javascript">
function validar(e) { // 1
    tecla = (document.all) ? e.keyCode : e.which; // 2
    if (tecla==8) return true; // 3
    patron =/\d/; // 4
    te = String.fromCharCode(tecla); // 5
    return patron.test(te); // 6
} 
</script>

FORM:

Código:

<td align="right"><span class="style2">Monto Max. :</span></td>
<td>
<input type="text" name="montomax" id="Monto Max.*" onKeyPress="return validar(event)" onFocus="foco(this);" onBlur="no_foco(this);" style="border:1px solid #CCCCCC"/>
</td>
</tr>
<tr>
<td align="right"><span class="style2">Monto Min. :</span></td>
<td>
<input type="text" name="montomin" id="Monto Min.*" onKeyPress="return validar(event)" onFocus="foco(this);" onBlur="no_foco(this);" style="border:1px solid #CCCCCC"/></td>

[tunait]

Por lo que veo tienes definido primero el monto máximo y después el mínimo. Así la posición del campo contra la que comparas es otro campo distintos

osea, tienes

montomaximo
montomimino
otrocampo
si tomas montominimo y lo comparas con su siguiente campo [m+1] lo estás comparando con otrocampo y no con montomaximo.

[Spychaser]

si es que no sabia como hacerlo de otra manera, entonces al tener los 2 campos de monto juntos me los valida.
El tema es que si ingresan mal los montos, luego cuando los pongan bien, le va a salir que esta mal.
Como puedo solucionarlo.
Muchas gracias, por la mano Tunait.

[tunait]

No, si los ponen bien no va a pasar eso.

¿ya has cambiado ese + por -??

cualo.elements[m-1].value

[Spychaser]

Buenas, cambie

Código:

if(cualo.elements[m-1].value < cualo.elements[m].value)

peor no me funciona, sigue el mismo error:
Al ingresar mal un monto y luego corregirlo te sigue indicando que esta mal el monto:
Muchas Gracias

[tunait]

Déjame una dirección donde poder verlo en marcha

[Spychaser]

Tunait:
aca te paso la direccion donde esta funcionando.
Muchas gracias

[tunait]

pues puse en montomax 5 y en montomin 6 y me salió una alerta de que el max debía ser mayor. Luego cambién el montomax a 7 y el form se envió sin problemas (recibirás un email absurdo, por cierto)

No sé, a mi me ha funcionado correcto

[Spychaser]

Tunait, muchas gracias por hacer esas pruebas, pero ocurre algo muy raro, prueba con 1000 y 1100, con esos valores no anda, nose porque.
Muchas gracias

[Spychaser]

Estuve probando y valida del 1-10 si insertas un numero mayor ocurre ese error.
Alguana idea.
Muchas gracias.

[Spychaser]

Alguna solucion?

[tunait]

Se me había olvidado

er, mira, a veces lo hace bien a veces mal.

Prueba una cosa ... cambia esto ...

Código:

if (cualo.elements[m].id=="Monto Min.*")
            {
             if(cualo.elements[m-1].value < cualo.elements[m].value)

por esto

Código:

if (cualo.elements[m].id=="Monto Min.*")
            { 
			montoMax = parseFloat(cualo.elements[m-1].value)
			montoMin = parseFloat(cualo.elements[m].value)
             if(montoMax <= montoMin)

Si no van a haber decimales en los montos puedes usar parseInt en vez de parseFloat (yo por las dudas puse el float)

me cuentas que tal

saludos

[Spychaser]

Muchas gracias Tunait, anda de 10!
Gracias

[tunait]

Estupendo me alegra que lo solucionaras.

Un saludo