problema al validar dato

mager3 · #1 (**permalink**) 01/03/2013, 11:32

Hola a Todos...

estoy validando con la siguiente funcion para que no ingresen datos vacios:

Código PHP:

Ver originalfunction valida_datos() {
      global $form_result;
      
      if( ( empty( $_POST ) == 0 ) && empty( $_POST['form1'] ) ) {
          return false;
      }
 
      $message = "Favor de completar los siguientes campos: \n";
      $result = true;
      if( empty( $_POST['txtobservaciones'] ) ) {
           $message .= " - Debes diligenciar las observaciones\n";
           echo "<html><head></head>". "<body onload=\"javascript:history.back()\">"."</body></html>";
           $result = false;
      }

pero en txtobservaciones puede ir valor 0 ó 1 , cuando selecciono 1 no hay problema, pero si selecciono 0 me arroja que debo diligenciar el campo , a pesar que Cero es un valor válido , lo que no debe permitir es que el campo vaya vacio...

creo que la falla va aca: f( ( empty( $_POST ) == 0 ), como puedo cambiar esta parte para que el Valor Cero sea valido?

Ekel · #2 (**permalink**) 01/03/2013, 11:39

$_POST

es una variable global.. y es un array.

es como si fuera

$_POST = array('valor' => 0);

y si haces echo $_POST['valor']; <- eso te devolvería 0

lo que tienes que hacer es elegir un index para devolver un valor

$_POST == 0 <- es erronio

$_POST['campo'] == 0 <- es correcto.

mager3 · #3 (**permalink**) 01/03/2013, 11:46

gracias ekel comprendo que por alli este el problema, pero dime algo, en que me estoy equivocando con la sintaxis que me arroja el siguiente error cuando lo cambio asi como me indicas...

Código PHP:

Ver originalif( ( count ($_POST['txtobservacion']) == 0  && empty( $_POST['form1'] ) ) {
          return false;
      }

Parse error: syntax error, unexpected T_VARIABLE in C:\Program Files\VertrigoServ\www\empleo\seguimiento_prueba.p hp on line 15

la linea 15 es la del if

Ekel · #4 (**permalink**) 01/03/2013, 11:51

a ver..

explicame que quieres lograr con el código?

count(); <- es para contar los indices en el array..

si haces $_POST['txtobservacion'] <- quieres saber si eso esta vacio? o no tienen nada? o qué?

si quieres comprobar que este vacio es:

Código PHP:

Ver originalif(isset($_POST['txtobservacion'])){
echo 'campo vacio';
}

mager3 · #5 (**permalink**) 01/03/2013, 11:55

el error era por ( adicional, lo corregi pero como lo cambie ya no entra a la funcion de validar datos

Código PHP:

Ver originalif  ( count ($_POST['txtpuntuacion']) >= 0  && empty( $_POST['form1'] ) ) {
          return false;
      }

mager3 · #6 (**permalink**) 01/03/2013, 11:57

mira mi codigo es este completo: con este codigo valido que no hayan campos vacios en el formulario, si estan vacios se retorna y muestra mensaje de debes llenar los datos XXXXXX , si estan llenos los campos permite el INSERT

Código PHP:

Ver original<?php
include ("conexion.php");
session_start();
$documento=$_SESSION['documento_user'];
$id_prueba=$_GET['id_prueba'];
$id_convocatoria=$_GET['i'];
$id_cargo=$_GET['c'];
$documento_postulante=$_GET['p'];
$observacion=$_POST['txtobservaciones'];
$puntuacion=$_POST['txtpuntuacion'];
 
function valida_datos() {
      global $form_result;
      
      if(isset($_POST['txtobservacion']) && empty( $_POST['form1'] ) ) {
          return false;
      }
 
      $message = "Favor de completar los siguientes campos: \n";
      $result = true;
      if( empty( $_POST['txtobservaciones'] ) ) {
           $message .= " - Debes diligenciar las observaciones\n";
           echo "<html><head></head>". "<body onload=\"javascript:history.back()\">"."</body></html>";
           $result = false;
      }
       if( empty( $_POST['txtpuntuacion'] ) ) {
           $message .= " - Debes diligenciar la puntuacion obtenida\n";
           echo "<html><head></head>". "<body onload=\"javascript:history.back()\">"."</body></html>";
           $result = false;
           }
        if( !$result ) {
           $form_result = sprintf( "<script>alert( '%s' );</script>", str_replace( "\n", "\\n", $message ) );
      }
      
      return $result;
}      
$form_result = "";
$consulta_prueba=mysql_query("SELECT descrip_prueba FROM pruebas WHERE cod_prueba='$id_prueba'");
 
  while($row = mysql_fetch_array($consulta_prueba))
  {
    $descripcion_prueba=$row[0];
  }
mysql_free_result($consulta_prueba);
 
    if( valida_datos()){   // realizo validaciones que los campos no esten vacios
    mysql_query("INSERT INTO detalle_pruebas(id_convocatoria,id_cargo,id_prueba,observaciones_prueba,puntuacion,documento_postulante,documento_profesional)
    VALUES('$id_convocatoria','$id_cargo','$id_prueba','$observacion','$puntuacion','$documento_postulante','$documento')");
    echo "<META HTTP-EQUIV='Refresh' CONTENT='0;URL=seguimiento.php?documento=$documento'>";
}
echo $form_result;
 
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Ingresar Novedades</title>
<style type="text/css">
<!--
.Estilo16 {color: #FFFFFF;
    font-weight: bold;
}
body {
    background-image: url(imagenes/fondo.jpg);
}
.rojo {
background-color: #D64938;
color: #ffffff;
}
.verde {
background-color: #009900;
color: #ffffff;
}
-->
</style>
<script src="SpryAssets/SpryValidationSelect.js" type="text/javascript"></script>
<link href="SpryAssets/SpryValidationSelect.css" rel="stylesheet" type="text/css" />
<style type="text/css">
<!--
a:link {
    color: #FFFFFF;
    text-decoration: none;
}
a:visited {
    text-decoration: none;
    color: #FFFFFF;
}
a:hover {
    text-decoration: underline;
    color: #FFFFFF;
}
a:active {
    text-decoration: none;
    color: #FFFFFF;
}
.Estilo17 {
    color: #D64938;
    font-weight: bold;
}
.Estilo18 {
    color: #009900;
    font-weight: bold;
}
-->
</style></head>
 
<body>
<table width="1132" border="0" align="center">
  <tr>
    <td align="left" bgcolor="#5AA3D0"><span class="Estilo16"><img src="imagenes/LogotipoCoopsanaCurvasBlanco.jpg" alt="Coopsana" width="247" height="72" longdesc="Coopsana IPS" />Gestión Humana</span></td>
  </tr>
  <tr>
    <td align="left" bgcolor="#5AA3D0"><font color="ffffff"><?php echo "Bienvenido ".$_SESSION['nombre_user']; ?> </font> </td>
  </tr>
</table>
<form id="form1" name="form1" method="post" action="<?php if( !empty( $_POST['txtobservaciones'] ) ) { echo $_POST['txtobservaciones']; } ?>">
  <table width="797" border="0" align="center">
    <tr>
      <td colspan="4" bgcolor="#5AA3D0"><div align="center"><span class="Estilo16">INGRESAR RESULTADO</span></div></td>
    </tr>
    <tr>
      <td colspan="4">&nbsp;</td>
    </tr>
    <tr>
      <td width="139" bgcolor="#5AA3D0"><span class="Estilo16">Tipo de Prueba</span></td>
      <td colspan="3">&nbsp;<b><?php echo $descripcion_prueba; ?></b></td>
    </tr>
    <tr>
      <td bgcolor="#5AA3D0"><span class="Estilo16">Observación</span></td>
      <td colspan="3"><label>
        <textarea name="txtobservaciones" cols="75" rows="8" id="txtobservaciones"></textarea>
      </label></td>
    </tr>
    <tr>
      <td bgcolor="#5AA3D0"><span class="Estilo16">Puntuación</span></td>
      <td width="259">
        <select name="txtpuntuacion" id="txtpuntuacion">
          <option class="rojo"></option>
          <option class="rojo">0</option>
          <option  class="verde">1</option>
        </select>
      </td>
      <td width="177"><span class="Estilo17">Valor 0: No Cumple</span></td>
      <td width="204"> <span class="Estilo18">Valor 1: Si Cumple</span></td>
    </tr>
    <tr>
      <td colspan="4">&nbsp;</td>
    </tr>
    <tr>
      <td colspan="4"><label>
        <div align="center">
          <input type="submit" name="enviar" id="enviar" value="Enviar" />
          <table width="181" border="0" align="center">
            <tr>
              <td width="93" bgcolor="#D64938"><div align="center"><span class="Estilo16"><a href="principal_administrador.php">Ir a Principal</a></span></div></td>
              <td width="10" bgcolor="#FFFFFF">&nbsp;</td>
              <td width="67" bgcolor="#D64938"><div align="center" class="Estilo16"><a href="javascript:history.back(1)">Regresar</a></div></td>
            </tr>
          </table>
        </div>
      </label></td>
    </tr>
  </table>
</form>
<p>&nbsp;</p>
</body>
</html>

Ekel · #7 (**permalink**) 01/03/2013, 12:02

el problema esta en que lo validas en una funcion, no lo hagas en funcion.

mager3 · #8 (**permalink**) 01/03/2013, 12:50

perfecto, asi lo hare, muchas gracias por la cooperacion !!!!