11

El caso

Resulta que haciendo unos test me he encontrado con un comportamiento algo inusual o que antes no había experimentado.

El tema es que la función switch permite hacer comparaciones de forma consecutiva del tipo:

//...
switch ($i) {
    case 0:
    case 1:
    case 'a':
    echo "...";
    break;
//...

En el manual de php aparecen ejemplos similares.

La teoría es que si el valor de $i es 0, 1, o a debería activar el echo.

En el caso que muestro:

  • Los valores 0 o a retornarían 0.
  • Los valores 1 o b retornarían 1.
  • Los valores 2 o c retornarían 2.

Pero no se que motivo hace que no de los resultados esperados con el siguiente ejemplo:

class Test {
    const A = 0;
    const B = 1;
    const C = 2;

    public function select($value)
    {
        switch($value) {
            case 'a':
            case self::A:
                return self::A;
                break;
            case 'b':
            case self::B:
                return self::B;
                break;
            case 'c':
            case self::C:
                return self::C;
                break;
            default:
                return 3;
        }
    }
}

$test = new Test();
echo $test->select('b');

Un ejemplo funcional en un Sandbox.

El problema

Todos los integer retornan su valor correcto, pero los strings siempre retornan 0.

La pregunta

¿Por qué motivo los valores con string no retornan el valor que le corresponde?

2
  • 2
    No es un problema del switch en sí mismo, sino de la forma en que PHP hace las comparaciones. Yo plantée una pregunta con un problema similiar... la conclusión fue que al hacer una comparación no estricta de una cadena, ésta es parseada a int, y termina siendo convertida en un cero (0).
    – A. Cedano
    el 14 may. 2018 a las 11:00
  • @A.Cedano eso estoy viendo. Ni me había planteado que podía ser eso. Gracias por comentar!
    – OscarR
    el 14 may. 2018 a las 11:02

2 respuestas 2

5

El problema viene de una conversión de tipos, en la documentación podemos encontrar lo siguiente:

Operadores de comparación:

Si se compara un número con un string o la comparación implica strings numéricos, entonces cada string es convertido en un número y la comparación realizada numéricamente. Estas reglas también se aplican a la sentencia switch. La conversión de tipo no tiene lugar cuando la comparación es === o !== ya que esto involucra comparar el tipo así como el valor.

¿Qué esta sucediendo?

Primero veamos que sucede cuando forzamos el String b a número

$value = 'b';
echo (int) $value; // 0

Sí vemos la documentación de Conversión de string a números vemos lo siguiente:

El valor es dado por la parte inicial del string. Si el string empieza con un dato numérico válido, éste será el valor empleado. De lo contrario, el valor será 0 (cero).

Por lo tanto si repasamos el switch y los reultados el ejemplo:

class Test {
    const A = 0;
    const B = 1;
    const C = 2;

    public function select($value)
    {  
      switch($value) {
        case 'a':  // a == b : false
        case self::A:  // 0 == 0 : true
            return self::A;
            break;
        case 'b':  // b == b : true
        case self::B:  // 1 == 0 : false
            return self::B;
            break;
        case 'c': // c == b : false
        case self::C: // 3 == 0 : false
            return self::C;
            break;
        default:
            return 3;
      }
    }
}

¿Cómo resolverlo?

Para mi la forma más apropiada es forzar a strings en el case, ya que podríamos recibir números como string y si forzamos tipos estas comparacion estricta será false (string '1'=== int 1 : false).

Ejemplo:

    switch($value) {
        case 'a':  // a == b : false
        case (string) self::A:  // b == 0 : false
            return self::A;
            break;
        case 'b':  // b == b : true
        case (string) self::B:  // b == 0 : false
            return self::B;
            break;
    }
4
  • Super claro!! Me tenía esto enredado toda la mañana y no terminaba de ver donde estaba el error. Gracias por el aporte.
    – OscarR
    el 14 may. 2018 a las 11:22
  • @OscarR Te añado una forma de solucionarlo, justo estaba editando.
    – Xerif
    el 14 may. 2018 a las 11:24
  • Pasa los test perfectamente Xerif. Muchas gracias por la info.
    – OscarR
    el 14 may. 2018 a las 11:56
  • 1
    Me alegro de que lo solucionaras, es posible que la respuesta contenga algun error de formato o ortografía, respondí desde el móvil, si detectais alguno invito a que lo corrijáis como Cedano hizo, gracias.
    – Xerif
    el 14 may. 2018 a las 12:05
4

El problema puede estar en que, según la documentación la sentencia switch/case funciona con comparaciones flexibles. con lo que esto supone a la hora de comparar cadenas.

Puedes ver un poco más de información en esta respuesta

Una manera de resolverlo es "forzando" a que haga comparaciones estrictas:

 class Test {
        const A = 0;
        const B = 1;
        const C = 2;

        public function select($value)
        {

            switch($value) {
                case 'a':
                case $value === self::A:
                    return self::A;
                    break;
                case 'b':

                case $value === self::B:
                    return self::B;
                    break;
                case 'c':
                case $value === self::C:
                    return self::C;
                    break;
                default:
                    return 3;
            }
        }
    }

    $test = new Test();
    echo $test->select('c');
4
  • lo mismo. Acabo de actualizar y veo que ya han dado respuesta... :)
    – Jakala
    el 14 may. 2018 a las 10:53
  • 1
    Genial Ernesto. Como comentaba en la pregunta, ni me había planteado que podía ser eso. Gracias por la respuesta! En el mismo tiempo estaban respondiendo y dando solución por el chat
    – OscarR
    el 14 may. 2018 a las 11:04
  • Ernesto, cambio la respuesta aceptada. La respuesta de Xerif se adapta mejor a los diferentes test con distintos formatos en el argumento. No obstante, gracias por el aporte.
    – OscarR
    el 14 may. 2018 a las 11:49
  • 1
    ¡Por supuesto! !yo mismo le he dado un voto positivo a @Xerif! Encantado de ayudar
    – ErnestoRM
    el 14 may. 2018 a las 12:02

Tu Respuesta

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

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