problema al validar un form

[yconoclasta]

Hola:

Ya llevo un rato revisando y aun no encuentro cómo solucionar este error.

Estoy intentando validar un form y lo hace a medias, en firefox marca los campos, pero a pesar de que no pasan, envia el form. Y en Iexplorer ni siquiera hace la validación y envía el form.

Qué hago ????

les paso las ligas del form y el js

http://idi.com.mx/simulador/simulador.html
http://idi.com.mx/simulador/validar.js

Gracias.

[caricatos]

Hola yconoclasta:

Intenté ver el código y no he podido... ¿Porqué no lo pegas en un mensaje...?

Saludos

[yconoclasta]

Gracias, pues es un poco largo, por eso no quería pegarlo, pero si eso facilita.

Código PHP:

  function ValidaNumero(Campo, pOpcion) 
{
    var checkOK = "0123456789.,";
    var checkStr = QuitaFormato(Campo.value);
    var allValid = true;
    var decPoints = 0;
    var allNum = "";
  for (i = 0;  i < checkStr.length;  i++) {
    ch = checkStr.charAt(i);
    for (j = 0;  j < checkOK.length;  j++)
      if (ch == checkOK.charAt(j))
        break;
    if (j == checkOK.length) {
      allValid = false;
      break;
    }
    if (ch == ".") {
      allNum += ".";
      decPoints++;
    }
    else if (ch != ",")
      allNum += ch;
  }
  if (!allValid) {
    alert("Escriba sólo dígitos en este campo.");
    Campo.value ="0.00"
    Campo.focus();
    Campo.select();
    return (false);
  }
 
  if (decPoints > 1) {
    alert("Escriba un número válido en este campo.");
    Campo.value ="0.00"
    Campo.focus();
    Campo.select();
    return (false);
  }
 
  if (QuitaFormato(Campo.value) == "")
        Campo.value = "0.00";
 
  if (QuitaFormato(Campo.value) >     9999999999.99)
  {
    alert("El importe no puede ser mayor a 9,999,999,999.99");
    Campo.value ="0.00"
    Campo.focus();
    Campo.select();
    return (false);
  }
 
  if (pOpcion && QuitaFormato(Campo.value) != "")
        Campo.value = FormateaNumero(Campo.value);
  return (true);
}
 
 
function FormateaNumero(Numero) 
{
 
    if (Numero == "0")
    {
        Numero = "0.00"
    }
    
    var numStr = QuitaFormato(Numero);
    var pDecimal = numStr.indexOf(".");
    var nDecimal = SubCadena(numStr, pDecimal, 3);
  var pNegativo = numStr.indexOf("-");
  if (pNegativo > -1)    
        var nEntero = SubCadena(numStr, 1, pDecimal - 1 );
    else
       var nEntero = SubCadena(numStr, 0, pDecimal );    
    var lEntero = nEntero.length;
    var vCEntero = lEntero;
    var nNumero = ""
    while (true) {
        if (vCEntero - 3 > 0) {
            nNumero = "," + SubCadena(nEntero, vCEntero - 3, 3) + nNumero            
            vCEntero = vCEntero - 3;
        }
        else {
            nNumero = SubCadena(nEntero, 0, vCEntero) + nNumero;
            break;
        }
    }
    nNumero = nNumero + nDecimal;
    if (pNegativo > -1)    
      nNumero = "-" + nNumero;
    return (nNumero);
}
 
 
function SubCadena(pCadena, pInicio, pCuantos) 
{
var vSubcadena
var vCicCadena
   
  vSubcadena = "";
  for (vCicCadena = 0; vCicCadena < pCuantos; vCicCadena++) {
    vSubcadena = vSubcadena + pCadena.charAt(pInicio + vCicCadena); 
    }
  return vSubcadena;
}
 
 
function QuitaFormato(NumFormateado) 
{
var numAux = "";
var numSCero = "";
var vCicNumero;
 
for (vCicNumero=0; vCicNumero < NumFormateado.length; vCicNumero++)
    if (NumFormateado.charAt(vCicNumero) != "0") {
      numSCero = SubCadena(NumFormateado, vCicNumero, NumFormateado.length);
      break;
    }
for (vCicNumero=0; vCicNumero < numSCero.length; vCicNumero++)
    if (numSCero.charAt(vCicNumero) != ",")
        numAux = numAux + numSCero.charAt(vCicNumero);
if (numAux.indexOf(".") == -1)
    numAux = numAux + ".00";
var vPos = numAux.indexOf(".");
var vLon = numAux.length;
var varCero = "00";
if (vLon - vPos < 3)
    numAux = numAux + SubCadena(varCero, vLon - vPos - 1, 2);
if (numAux.charAt(0) == ".") {
    numAux = "0" + numAux;
}
return (numAux);
}
 
 
function EsCero(Campo,Mensaje)
{
    if (parseFloat(QuitaFormato(Campo.value)) <= 0 )
    {
        alert (Mensaje);
        Campo.focus();
        Campo.select();
        return (false);
    }
    else
    {
        return (true);
    }
}
 
 
function SeleccionaTexto(Campo)
 
{
    //Si el valor del campo es diferente de ""
    if (Campo.value != "")
    {
        Campo.select();
    }
}
 
 
function ValidaCampos()
{
    //Obtiene la clave del tipo de crédito que ha sido seleccionado
    TipoCredito = document.form1.ddlDestinoCredito.options[document.form1.ddlDestinoCredito.selectedIndex].value;
    if (TipoCredito == "0")
    {
        alert ("Debe seleccionar el Destino de Crédito");
        document.form1.ddlDestinoCredito.focus();
        return false;
    }
 
    //Obtiene la clave de la plaza que ha sido seleccionado
    Plaza = document.form1.ddlCiudad.options[document.form1.ddlCiudad.selectedIndex].value;
    if (Plaza == "(Seleccione una Ciudad)")
    {
        alert ("0");
        document.form1.ddlCiudad.focus();
        return false;
    }
 
    //Obtiene la clave del tipo de simulador seleccionado
    TipoSimulador = document.form1.ddlCalculadoPor.options[document.form1.ddlCalculadoPor.selectedIndex].value;
    if (TipoSimulador == "0")
    {
        alert ("Debe seleccionar la opción por la cual desea hacer la simulación");
        document.form1.ddlCalculadoPor.focus();
        return false;
    }
    
    //Verifica que el importe no este en ceros
    if (TipoSimulador == "Comprar Casa")
        mensaje = "¿Cuál es el Valor del Bien?";
    if (TipoSimulador == "Construir Casa")
        mensaje = "¿Cuáles son los Ingresos Mensuales?";
    if (TipoSimulador == "Remodelar Casa o Departamento")
        mensaje = "¿Cuál es el Importe Solicitado?";
    if (EsCero(document.form1.txtMonto,mensaje)== false)
        return false;
    
    //Manda llamar el resultado
    parent.frames["fraResultado"].location.href="resultado.aspx?TipoCredito=" + TipoCredito + "&TipoSimulador=" + TipoSimulador + "&Importe=" + document.form1.txtImporte.value + "&Plaza=" + Plaza;
    return true;
} 
   

[Cap.Buscapina]

el problema es cuando no ingresa dentro de ningun if (de la funcion ValidaCampos()) y por lo tanto la variable mensaje no existe.

declará

mensaje=""

como variable global y creo que puede funcar.


saludos

[yconoclasta]

Cap.Buscapina, ya ví lo que me dices, lo he corregido pero aun así no valida.

Alguna otra idea?

Cabe mencionar que en el explorer me marca error al cargar la página, si me posiciono y selecciono el campo de monto. El error es en la linea 46 y dice que "se esperaba un objeto"

[Cap.Buscapina]

ups , ahora caigo

probá cambiar esta linea

<form name="form1" method="post" action="simulador.aspx" id="form1" class="cotizador" onsubmit="return ValidaCampos()">

suerte