3

Buenas a todos!

Me topé con algo que me está haciendo renegar y aún no encuentro la solución:

A mi código ingresa un campo el cual si posee LETRAS lo anulo.

Es decir:

                        $valor= "1A8CF4D2414E000094136783";
                        #FILTRADO CAMPO 1


                        if( preg_match('/^[A-Z]+$/', $valor) )
                        { 

                            echo('Lectura posee letras.');
                            $valor = "0";

                        } else {
                            echo('Lectura NO posee letras.');
                        }

El problema es que siempre me devuelve el valor de else aunque sea verdedera la condición. Qué puede estar mal?

Desde ya muchas gracias!!

4 respuestas 4

2

Tu problema es que preg_match devuelve un entero, no un booleano.

preg_match() devuelve 1 si pattern coincide con el subject dado, 0 si no, o FALSE si ocurrió un error.

Advertencia Esta función puede devolver el valor booleano FALSE, pero también puede devolver un valor no booleano que se evalúa como FALSE.

Todo lo siguiente, se considera falso, y por lo tanto cae por el else:

el integer 0 (cero)
el float 0.0 (cero)
el valor string vacío, y el string "0"
un array con cero elementos

En tu caso, estas preguntando si la cadena solo contiene letras, cosa que es falso. entonces devuelve 0, que es falso. Debes comprobar la regex, una que te podria funcionar seria:

[A-Z]+
4
  • Buena puntuación lo de la advertencia
    – OscarR
    Commented el 4 oct. 2017 a las 19:50
  • No entiendo la primera parte de la respuesta. Tal cual estás diciendo 0 es un valor que se evalúa como falso, por lo tanto if (preg_match( $re, $valor)) { está perfecto... Sólo la corrección del regex para que no use ^ ni $ es lo que aplica en este caso... Y un detalle, no hace falta coincidir con múltiples letras, sobra con la primera: if (preg_match('/[a-záéíóúüñ]/iu', $valor)) { ... O, si se prefiere /[^\W_\d]/u
    – Mariano
    Commented el 26 nov. 2017 a las 11:35
  • Le explique porque siempre cai por el else @Mariano. Luego de eso puse una cadena que podia llegar a resolver su problema. pero no me centre en la regex, ya que vi que no estaba entendiendo el problema del if.
    – gbianchi
    Commented el 27 nov. 2017 a las 3:21
  • dijo que siempre salia por el else.. no me centre en la regex.. me centre en que podia tener una confunción de como funcionaba la funcion preg_match...
    – gbianchi
    Commented el 27 nov. 2017 a las 4:13
2

Por aquí hay gente con muuucha mas experiencia que yo en Regex. Pero voy a intentar responderte.

Estás preguntando si coincide con:

/^    Empieza por
[A-Z] Un rango de A a Z
+     Una o más coincidencias
$/    Termina por

Por lo que solo te está admitiendo cadenas del tipo KJBFKGJBKSBGBJKVJV.

Si eliminas los delimitadores ^ y $ va a buscar cualquier coincidencia sin tener en cuenta expresamente que debe iniciar o acabar con los caracteres indicados. Lo cual resolvería el problema.

$valor="1A8CF4D2414E000094136783";
#FILTRADO CAMPO 1

if(preg_match('/[A-Za-z]+/', $valor)) {
    echo 'Lectura posee letras.';
    $valor="0";
}
else {
    echo 'Lectura NO posee letras.';
}
8
  • 1
    O prueba con ver si es numérico. preg_match('/^[0-9]/i', $valor)
    – Gustavo
    Commented el 4 oct. 2017 a las 19:50
  • @Gustavo claro, yo lo haría así por lógica, si solo admites números, cualquier cosa distinta de un entero es suficiente. is_int($valor). Lo he puesto en base a su misma estructura.
    – OscarR
    Commented el 4 oct. 2017 a las 19:59
  • no @Gustavo, como OP no aclaro el formato de la cadena que viene, esa Regexp falla para cosas que no empiecen con un numero. y /i no sirve con numeros!
    – gbianchi
    Commented el 4 oct. 2017 a las 20:02
  • 1
    @OscarR esa fue mi primera idea, pero si son todos numeros y muy largo, va a dar overflow.. entonces tampoco sirve...
    – gbianchi
    Commented el 4 oct. 2017 a las 20:03
  • @gbianchi ¿en que escenario se produce eso? no me ha pasado, es por curiosidad.
    – OscarR
    Commented el 4 oct. 2017 a las 20:04
2
$valor= "1A8CF4D2414E000094136783";

if ( preg_match('/[A-Z]/i', $valor) )
{
    echo('Lectura posee letras.');
} else {
    echo('Lectura NO posee letras.');
}

El /i evita la distinción de mayúsculas y minúsculas.

2
  • avisa si te sirve ;) Commented el 4 oct. 2017 a las 19:43
  • Esta debería ser la respuesta aceptada.
    – Mariano
    Commented el 26 nov. 2017 a las 11:40
0

intenta esta solucion

$valor= "1A8CF4D2414E000094136783";
                    #FILTRADO CAMPO 1

if(!preg_match('/^[0-9]+$/', $valor) )
{ 

 echo('Lectura posee letras.');
 $valor = "0";

 } else {
     echo('Lectura NO posee letras.');
 }
1
  • 1
    Deberías explicar la motivación de la respuesta... qué está mal y por qué tu código soluciona el problema.
    – eferion
    Commented el 5 oct. 2017 a las 6:09

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.