Cómo comprobar el valor de retorno de la función si es verdadero o falso

Question

Tengo aquí una función que valida campos en un formulario si están vacíos.

function ValidateForm() 
{ 
    jQuery('span.error_msg').hide(); 
    var success = true; 
    jQuery("#shippingF input").each(function() 
     { 
      if(jQuery(this).val()=="") 
      { 
       jQuery(this).next().show(); 
       success = false; 
      } 
    }); 
    return success; 
} 

Ahora quería utilizar esa función aquí:

function post(url,formId) { 
     jQuery("#process").html('<img src="<?php echo get_bloginfo('wpurl').'/wp-content/plugins/'.basename(dirname(__FILE__)).'/images/co/ajax-loader.gif'; ?>" alt="loading" title="ajax-loader" width="16" height="16" class="alignnone size-full wp-image-134">'); 
    jQuery.post(url, jQuery('#' + formId).serialize(), function(d) { 
     jQuery('html,body').animate({scrollTop: jQuery("#scrollhere").offset().top},'slow'); 
     jQuery("#response").html('<center><p style="height:820px"><span style="color:black;font-weight:bold;font: 11px arial,verdana,sans-serif;"><b>Loading available payment getaways..</b></span><br/><img src="<?php echo get_bloginfo('wpurl').'/wp-content/plugins/'.basename(dirname(__FILE__)).'/images/co/8-1.gif'; ?>" width="220" height="19" /></p></center>'); 
     jQuery("#response").load("<?php echo get_bloginfo('wpurl').'/wp-content/plugins/'.basename(dirname(__FILE__)).'/checkout_payment.php'; ?>", function() { Cufon.refresh(); }); 
     jQuery("#response").attr("style","height:1030px"); 
    }); 
} 

Lo probé y me ocurra esto.

function post(url,formId) { 
ValidateForm(); 
if(ValidateForm() == 'false') { 
    jQuery('html,body').animate({scrollTop: jQuery("#shippingF").offset().top},'slow'); 
} else { 
     jQuery("#process").html('<img src="<?php echo get_bloginfo('wpurl').'/wp-content/plugins/'.basename(dirname(__FILE__)).'/images/co/ajax-loader.gif'; ?>" alt="loading" title="ajax-loader" width="16" height="16" class="alignnone size-full wp-image-134">'); 
    jQuery.post(url, jQuery('#' + formId).serialize(), function(d) { 
     jQuery('html,body').animate({scrollTop: jQuery("#scrollhere").offset().top},'slow'); 
     jQuery("#response").html('<center><p style="height:820px"><span style="color:black;font-weight:bold;font: 11px arial,verdana,sans-serif;"><b>Loading available payment getaways..</b></span><br/><img src="<?php echo get_bloginfo('wpurl').'/wp-content/plugins/'.basename(dirname(__FILE__)).'/images/co/8-1.gif'; ?>" width="220" height="19" /></p></center>'); 
     jQuery("#response").load("<?php echo get_bloginfo('wpurl').'/wp-content/plugins/'.basename(dirname(__FILE__)).'/checkout_payment.php'; ?>", function() { Cufon.refresh(); }); 
     jQuery("#response").attr("style","height:1030px"); 
    }); 
} 
} 

El problema es que, la validación está trabajando .. Sin embargo, la función se ejecuta .post() aunque hay campo vacío. Supongo que está en la condición if/else ... ¿hay una mejor manera de lograr esto?

Gracias.

Answer 1

false != 'false' 

Por buenas medidas, puesto el resultado de validar en una variable para evitar la doble validación y usar eso en la declaración IF. De esta manera:

var result = ValidateForm(); 
if(result == false) { 
... 
} 

Answer 2

que está comparando el resultado con una cadena ('falso') no es la constante integrada negativo (falso)

sólo tiene que utilizar

if(ValidateForm() == false) { 

o mejor aún

if(!ValidateForm()) { 

también ¿por qué llamas a validateForm dos veces?

Answer 3

ValidateForm devuelve boolean, no es string.
Cuando hace esto if(ValidateForm() == 'false'), es lo mismo que if(false == 'false'), que no es verdadero.

function post(url, formId) { 
    if(!ValidateForm()) { 
     // False 
    } else { 
     // True 
    } 
} 

Answer 4

Sintaxis incorrecta. No puede comparar un booleano con una cadena como "falso" o "verdadero". En su caso, sólo prueba que es inversa:

if(!ValidateForm()) { ... 

Usted podría de ensayo frente a la constante falsa, pero es bastante feo y generalmente mal visto:

if(ValidateForm() == false) { ... 

Answer 5

No es necesario llamar ValidateForm() dos veces, ya que está por encima. Usted sólo puede hacer

if(!ValidateForm()){ 
.. 
} else ... 

Creo que va a resolver el problema lo anterior parece que su comparación true/false a la cadena equivalente 'false'.