7

Desde un formulario envío un campo valor, y quiero procesar dicho campo siempre y cuando no sea vacío y no exceda la longitud de 15 caracteres. Hice esto, pero igual muestra PROCESAR.

if (strlen(!empty($_POST['valor'])) <= 15) {
    echo "PROCESAR";
}
2
  • a ver tienes un problema de logica !empty($_POST['valor']) devolvera true y luego haces strlen(true') obviamente siempre va a ser < 15 el 20 jun. 2018 a las 17:49
  • ¿Qué valores puede tener la variable $_POST['valor']?
    – user22721
    el 20 jun. 2018 a las 18:19

4 respuestas 4

7

Necesitas dos condiciones separadas. No lo puedes hacer todo junto como lo tienes. Como te explicaron en un comentario. La condcion !empty($_POST['valor']) va a evaluar siempre a true o false. Cuando evalueas la segunda condicion, siempre va a ser strlen(true) o strlen(false), y esos dos valores van a evaluar verdadero porque los dos son menores que 15. La forma correcta seria asi:

if (!empty($_POST['valor']) && strlen($_POST['valor']) <= 15) {
    echo "PROCESAR";
}
1
  • 3
    Podes explicar un poco mejor porque?
    – gbianchi
    el 20 jun. 2018 a las 18:01
5

Tu problema

Primero y antes de saltar a dar una respuesta voy a explicar porque no funciona tu código.

De acuerdo a la documentación empty() nos regresará un booleano y strlen() nos regresará un entero, esto quiere decir que si hacemos

strlen(true)

o

strlen(false)

Nos regresará 0 y 1 respectivamente, como sabemos 1 y 0 son menores o iguales a 15 por lo que tu condición pase lo que pase siempre será verdadera.


Respuesta

Por otro lado lo que no toman en cuenta ninguna de las respuestas es que si $_POST['valor'] es '0', la validación !empty($_POST['valor']) nos dará como resultado false por lo que no entrará en la condición.

Si necesitamos que el valor no esté vacío podemos hacerlo con el comparador !== de esta forma

$valor = $_POST['valor'];
if ($valor !== '' && strlen($valor) <= 15) {
    echo "PROCESAR";
}

Lo que nos dará paso a otro problema, ¿qué pasa si la variable no existe o es null?

Yo sugiero que para no tener falsos negativos revisemos que la variable exista y luego revisamos que no sea '', de esta forma:

$valor = $_POST['valor'];
if (isset($valor) && $valor !== '' && strlen($valor) <= 15) {
    echo "PROCESAR";
}
2
  • Con el ternario funciona tanto si $valor es igual a: 0, '0', NULL, FALSE, ''... funciona en todos los casos.
    – A. Cedano
    el 20 jun. 2018 a las 18:51
  • @A.Cedano no, rextester.com/BTJQC84092. A menos que yo no esté entendiendo el problema :)
    – user22721
    el 20 jun. 2018 a las 18:54
4

Otra posibilidad sería usar un operador ternario, del siguiente modo:

$status=(!empty($_POST['valor'])) && (strlen($_POST['valor']) <= 15) ? "PROCESAR" : "NO PROCESAR";

Ejemplos:

En los ejemplos que siguen, la variable $status adquiere un valor u otro según el caso:

$valor=NULL; //$_POST['valor']

$status=(!empty($valor)) && (strlen($valor) <= 15) ? "PROCESAR" : "NO PROCESAR";
echo $status;
echo PHP_EOL;

Salida:

NO PROCESAR

$valor="PALABRADEMASDEQUINCE";
$status=(!empty($valor)) && (strlen($valor) <= 15) ? "PROCESAR" : "NO PROCESAR";
echo $status;    
echo PHP_EOL;

Salida:

NO PROCESAR

$valor="PALABRADEQUINCE";
$status=(!empty($valor)) && (strlen($valor) <= 15) ? "PROCESAR" : "NO PROCESAR";
echo $status;

Salida:

PROCESAR

3

Saludos debería ser de este modo

$var = 'foos';

if(!empty($var) && strlen($var) >3){
  echo "exito";
}else{
  echo "fallo";
}

Con !empty indicas que mientras la variable no este vacía con el

operador && siginifica AND es un operador lógico; se esta usando para indicar que para que la instrucción se ejecute necesita que ambas comparaciones ocurran

if ((!empty($_POST['valor']) && strlen($_POST['valor']) <= 15) {
    echo "PROCESAR";
}

Como tu enunciado dicta que para que la variable sea válida no debe estar vacia y no debe ser mayor a 15 entonces primero lo comprobamos con la función empty pero negandola ya que el resultado que te devuelve es booleano, true o false, posterior al hacer esa revisión ahora procedemos con la longitud y si ambas condiciones se cumplen entonces accedemos al valor del if en caso contrario con alguna de las condiciones que no se cumpla te mandrá al return que pongas en el false

2
  • el operador && no indica que despues sigue otra instruccion.. significa AND y es un operador logico.
    – gbianchi
    el 20 jun. 2018 a las 17:59
  • cierto @gbianchi lo agregaré pues no me supe explicar
    – user75901
    el 20 jun. 2018 a las 18:00

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.