[SOLUCIONADO] PHP OO Problema al generar un valor aleatorio sin que se repita

[guardarmicorreo]

Tengo el siguiente método que lo que hace es generar un número aleatorio.

Para que no genere dos números aleatorios iguales, lo que hago es guardar cada número aleatorio en sesión, luego comparo el número con el array en sesión, si no está entonces retorno el valor y sino vuelvo a generar otro número aleatorio.

El problema es que hay un tramo del código en el que genera un valor nulo, entre que compara el valor e introduce la pregunta en sesión.

El resultado es que hay preguntas que se quedan con valor null.

En el código explico dónde creo que hay errores. Los cuales por más que los he buscado, pensado y analizado no los encuentro.

La función en el controlador que obtiene tanto amigos aleatorios como preguntas aleatorias sobre dicho amigo.

Código PHP:

/*
    obtiene un amigo aleatorio y cinco preguntas sobre el
*/
function play_start()
{
    if ( isset( $_SESSION['user_twitter'] ) AND $_SESSION['twitter_status'] == true )
    {
        //vacio la sesion con los numeros obtenidos aleatoriamente
        //y la declaro de nuevo como array para utilizarla en el array_push del metodo
        //getrandomvalues()
        unset( $_SESSION['randomresult']['count'] );
        
        //obtengo los amigos en una sesion
        gettruefriendslist();
        
        //instancio la clase personal
        $newget                     = new Twitter_main();
        
        //obtengo un numero de amigo aleatorio
        $getfriendrandom            = $newget->getrandomvalue( $_SESSION['gettruefriendslist'] );
        
        //guardo el amigo aleatorio en sesion
        $_SESSION['randomresult']['friend'] = $_SESSION['gettruefriendslist'][$getfriendrandom];
        
        //instancio la clase Conectar
        $newget2                    = new Conectar();
        
        //selecciono las preguntas y las guardo en array
        $select                     = "SELECT * FROM questions";
        
        $result_conexion            = $newget2->conectar( $select ); 
 
        while ($result = mysqli_fetch_array( $result_conexion ))
        {
            $getquestions[] = $result;
        }
        
        
        //vacio la sesion con las preguntas
        //y la declaro de nuevo como array para utilizarla en el array_push
        unset( $_SESSION['randomresult']['questions'] );
        $_SESSION['randomresult']['questions']= array();
        
        //VUELVO A VACIAR la sesion con los numeros obtenidos aleatoriamente
        //y la declaro de nuevo como array para utilizarla en el array_push del metodo
        //getrandomvalues()
        unset( $_SESSION['randomresult']['count'] );
        
        $_SESSION['randomresult']['questions']= array();
        //declaro questions como array
        $questionsrandom = array();
        
        //obtengo todas las etiquetas
        $tags = $newget->gettags();
    
        //busco 5 preguntas aleatorias
        //y sustituyo [nombre] por el valor que deseo finalmente
        for($i=0; $i<=4 ; $i++)
            
        {
            //obtengo 5 numeros aleatorios
                        //aquí es donde comienza el error
            $getquestionrandom      = $newget->getrandomvalue ( $getquestions );
            
                        //no estoy seguro si el error está aquí al guardar de esta manera el valor en questionsrandom
            //guardo las preguntas segun los numeros anteriores
            array_push($questionsrandom, $getquestions[$getquestionrandom]);
            
            //instancio al metodo para hacer las sustituciones
            $questionsrandom[$i]['question'] = $newget->setvaluestags($tags,
            "@".$_SESSION['randomresult']['friend']['screen_name'], $questionsrandom[$i]['question']);
        }
        //finalmente lo guardo en sesion
        array_push($_SESSION['randomresult']['questions'], $questionsrandom ); 
    }
        
}
 

Aquí es donde genero los valores aleatorios, tanto para obtener el amigo como para las preguntas

Código PHP:

/*
        genera un valor aleatorio de un array
    */
    public function getrandomvalue ( $imput )
    {
        if( empty( $_SESSION['randomresult']['count'] ) 
            OR !is_array( $_SESSION['randomresult']['count'] ) )
        {
            $_SESSION['randomresult']['count'] = array();
        }
        if ( is_array( $imput ) )
        {
            //cuento los valores del array
            $this->result = count ( $imput );
            
            //resto un valor para que random no escoja un valor no encontrado en el array
            $this->result = $this->result - 1;
            
            //elijo un numero al azar
            $this->result = mt_rand( 0, $this->result );
            
            /*$this->result = array_rand( $imput );*/
            $inarray = in_array( $this->result, $_SESSION['randomresult']['count'] );
            //no se si puede estar aquí el error, es otra posibilidad que he analizado pero no encuentro nada
            if( $inarray === false )
            {
                //guardo el valor en count
                array_push( $_SESSION['randomresult']['count'], $this->result );
                //finalmente retorno el numero escogido
                return $this->result;
            }           
            else
            {
                $this->getrandomvalue( $imput );
            }
        }
    }
 

La estructura del array que utilizo para las preguntas aleatorias son

Código PHP:

$getquestions array(10){
  [0, 1, 2, ...] array(6){
   [0]=> string(...)
   ["id_question"] => string(...)
   [1]=> string(...)
   ["question"] => string(...)
   [2]=> string(...)
   ["id_typequestions"] => string(...)
   }
}
 

Quien necesite más de mi código solo tiene que pedirlo y si no compromete al sistema entonces lo aportaré sin problemas.

¿Alguien me puede ayudar a entender por qué obtengo preguntas con valor null?

[Cuervoo]

A simple vista no se ve nada (o yo no vi nada) muy evidente. Como usas clases y métodos que no están en el código no se puede copiar y probar qué puede llegar a ser.

Pero si te sirve, te puedo decir que si no usas un debugger podés usar echos en forma de "breackpints" para ir siguiendo los pasos, una forma no muy elegante pero a mi cada tanto me ayuda.

Fijate las variables que setea, los valores que va comparando, por donde se va metiendo.
Sobre todo que cuando entra a if ( $inarray === false ), devuelva un valor correcto. Y despues todos los valores que agrega array_push($questionsrandom, $getquestions[$getquestionrandom]) lo haga bien.

Un ejemplo sería, previo a esa linea.
echo "Agrega la pregunta: array_push($questionsrandom, {$getquestions[$getquestionrandom]})<br />";

Y vas a ir viendo si agrega la pregunta o un valor nulo. Si no es ahi vas haciendo lo mismo por distintos puntos donde pueda estar el error hasta que lo puedas aislar y solucionar.

[guardarmicorreo]

Iniciado por Cuervoo

A simple vista no se ve nada (o yo no vi nada) muy evidente. Como usas clases y métodos que no están en el código no se puede copiar y probar qué puede llegar a ser.

Pero si te sirve, te puedo decir que si no usas un debugger podés usar echos en forma de "breackpints" para ir siguiendo los pasos, una forma no muy elegante pero a mi cada tanto me ayuda.

Fijate las variables que setea, los valores que va comparando, por donde se va metiendo.
Sobre todo que cuando entra a if ( $inarray === false ), devuelva un valor correcto. Y despues todos los valores que agrega array_push($questionsrandom, $getquestions[$getquestionrandom]) lo haga bien.

Un ejemplo sería, previo a esa linea.
echo "Agrega la pregunta: array_push($questionsrandom, {$getquestions[$getquestionrandom]})<br />";

Y vas a ir viendo si agrega la pregunta o un valor nulo. Si no es ahi vas haciendo lo mismo por distintos puntos donde pueda estar el error hasta que lo puedas aislar y solucionar.

Muchas gracias por tu ayuda. He probado estos días todo tipo de maneras para atrapar el error (var_dumps, echo) en cada parte del código y he logrado saber dónde se genera precisamente el error pero no entiendo el por qué, por lo tanto no puedo solucionarlo de momento.

El error se genera en el momento que in_array no es false porque el valor ya existe en la lista de números generados, entonces se vuelve a generar un número aleatorio y ese número no existe, pero en la función donde recojo ese número aleatorio lo detecta como null.

Harto ya, voy a instalar xdebug, leer cómo se utiliza y ver cual es el motivo.

Sospecho que el motivo es que la función llama al método, este genera otro valor si el valor generado existe pero ese paso, el de generar un nuevo valor, resta un paso en el for de la función, por lo que hay una pregunta que se queda vacía.

Eso no tendría sentido, puesto que el método se vuelve a llamar a sí mismo y al no retornar valor, solamente volver a llamarse a si mismo ese paso/iteración quedaría al margen de la función, es decir, la función se quedaría esperando a que el método le de un valor y continuaría con la iteración del for.

No se si me he explicado bien.

En definitiva, veré a ver si con xdebug puedo solucionarlo y además la comprobación de repetidos la haré en la función en vez de en el método y si así funciona bien entonces mis sospechas se habrán hecho ciertas.

Saludos!!!

[gnzsoloyo]

Yo te propondría un ejercicio de análisis: ¿Que condiciones deberían cumplirse en esa llamada para obtener intencionalmente un NULL?

A veces, cuando recorriendo la lógica con datos reales, no se obtiene la respuesta de qué falla, es mejor mirarlo a la inversa...

[guardarmicorreo]

Iniciado por gnzsoloyo

Yo te propondría un ejercicio de análisis: ¿Que condiciones deberían cumplirse en esa llamada para obtener intencionalmente un NULL?

A veces, cuando recorriendo la lógica con datos reales, no se obtiene la respuesta de qué falla, es mejor mirarlo a la inversa...

En la función, justo después de la línea 64 del código que he puesto aquí he hecho dos var_dump()

Código PHP:

var_dump( $getquestionrandom );
var_dump( $_SESSION['randomresult']['count'] );
 

Lo curioso, es que el método funciona correctamente, genera los valores aleatoriamente y en caso de que haya alguno repetido vuelve a generar otro valor hasta que no esté repetido y los va guardando en la sesión count.

Por lo tanto lo raro, rarísimo, es que de los var_dump anteriores en la última ejecución en local me hayan indicado lo siguiente

Código PHP:

int 6
array (size=1)
  0 => int 6
            
 
                        
           
int 1
array (size=2)
  0 => int 6
  1 => int 1
            
 
                        
           
int 10
array (size=3)
  0 => int 6
  1 => int 1
  2 => int 10
            
 
                        
           
null
array (size=4)
  0 => int 6
  1 => int 1
  2 => int 10
  3 => int 9
            
 
                        
           
//esto es del xdebug
( ! ) Notice: Undefined index:  in C:\xampp\htdocs\xampp\web\videosfav\twitter2\control\controller_public.php on line 207
Call Stack
#   Time    Memory  Function    Location
1   0.0020  135888  {main}(  )  ..\index.php:0
2   1.0251  580864  html_start(  )  ..\index.php:23
3   1.0251  580880  play_start(  )  ..\html_public.php:133
            
 
                        
                       
int 4
array (size=5)
  0 => int 6
  1 => int 1
  2 => int 10
  3 => int 9
  4 => int 4
 

Si te fijas, lo curioso es que en sesión estan todos los valores correctamente pero en la función esta vez el 9 lo recoge como null :S

Eso es lo raro y no entiendo el por qué.

Para que un valor sea int pero el método retorne null o la función recoja null no se me ocurre ningún caso posible sin añadir código que ahí no está y yo no he escrito, cabe decir que el proyecto y código lo he hecho exclusivamente yo, nadie más ha tocado el código y no estoy utilizando framework.

EDITO y cabe decir que en la BD hay 11 preguntas, por lo tanto una novena pregunta sí tendría que haber existido.

[dashtrash]

Lo que te dicen los var_dump es correcto.No te debería parecer raro que una variable esté a null, y la otra no lo esté.Una no está tomando el valor de la otra, así que no dependen entre sí.
Al usar una superglobal ($_SESSION), la estás asignando valor directamente.A la otra variable se le asigna un valor según el valor retornado de una función.Si tiene un null, es porque la función puede retornar null.
Esto te deberia dar una pista:

Código PHP:

function a()
{
}
 
$b=a();
var_dump($b);
 

[guardarmicorreo]

Iniciado por dashtrash

Lo que te dicen los var_dump es correcto.No te debería parecer raro que una variable esté a null, y la otra no lo esté.Una no está tomando el valor de la otra, así que no dependen entre sí.
Al usar una superglobal ($_SESSION), la estás asignando valor directamente.A la otra variable se le asigna un valor según el valor retornado de una función.Si tiene un null, es porque la función puede retornar null.
Esto te deberia dar una pista:

Código PHP:

function a()
{
}
 
$b=a();
var_dump($b);
 

Tienes razón. Entiendo a qué te refieres. Pero si te fijas el método solo puede retornar un valor comprendido en el rango de preguntas disponibles. No puede retornar null pero a la vez en la sesión guardar por ejemplo el int 9. No tiene sentido que haga eso :S

Si tiene sentido entonces algo se está escapando a mi comprensión :S

[Cuervoo]

Te fijaste cual es el valor que tiene result cuando da null? Hacé que muestre $this->result y fijate si siempre es el mismo valor cuando da null o va cambiando. Seguile los pasos en las tres líneas que lo modificás.

Código PHP:

$this->result = count ( $imput );
            
//resto un valor para que random no escoja un valor no encontrado en el array
$this->result = $this->result - 1;
            
//elijo un numero al azar
$this->result = mt_rand( 0, $this->result );
 

En esas tres.

[dashtrash]

Iniciado por guardarmicorreo

Tienes razón. Entiendo a qué te refieres. Pero si te fijas el método solo puede retornar un valor comprendido en el rango de preguntas disponibles. No puede retornar null pero a la vez en la sesión guardar por ejemplo el int 9. No tiene sentido que haga eso :S

Si tiene sentido entonces algo se está escapando a mi comprensión :S

Yo veo una línea de ejecución que hace que la función no ejecute ningún return.Por lo que su valor devuelto será null...

[guardarmicorreo]

Iniciado por Cuervoo

Te fijaste cual es el valor que tiene result cuando da null? Hacé que muestre $this->result y fijate si siempre es el mismo valor cuando da null o va cambiando. Seguile los pasos en las tres líneas que lo modificás.

Código PHP:

$this->result = count ( $imput );
            
//resto un valor para que random no escoja un valor no encontrado en el array
$this->result = $this->result - 1;
            
//elijo un numero al azar
$this->result = mt_rand( 0, $this->result );
 

En esas tres.

He hecho var_dump() a las tres líneas.

El código en el método para los var_dump() ha sido el siguiente

Código PHP:

//cuento los valores del array
            echo "result count<br>";
            $this->result = count ( $imput );
            
            var_dump( $this->result );
            
            //resto un valor para que random no escoja un valor no encontrado en el array
            echo "result -1<br>";
            $this->result = $this->result - 1;
            
            var_dump( $this->result );
            //elijo un numero al azar
            echo "result mt_rand<br>";
            $this->result = mt_rand( 0, $this->result );
 

Mientras que el código para los var_dump() en la función ha sido el siguiente

Código PHP:

for($i=0; $i<=4 ; $i++)
            
        {
            //obtengo 5 numeros aleatorios         
            echo "iteración <b>". $i."</b><br>";
            $getquestionrandom      = $newget->getrandomvalue ( $getquestions );
                
            ?>
            <pre>
            <?php
                
                var_dump( $getquestionrandom );
                var_dump( $_SESSION['randomresult']['count'] );
                echo "fin de la iteración <b>". $i."</b><br>";
            
            //var_dump($getquestions[$getquestionrandom]);
            ?>
            </pre>
            <?php
            //guardo las preguntas segun los numeros anteriores
            array_push($questionsrandom, $getquestions[$getquestionrandom]);
            
            //instancio al metodo para hacer las sustituciones
            $questionsrandom[$i]['question'] = $newget->setvaluestags($tags,
            "@".$_SESSION['randomresult']['friend']['screen_name'], $questionsrandom[$i]['question']);
        }
 

Los resultados

Código PHP:

//estos primeros var_dump corresponden a la elección del usuario aleatoriamente
//por este motivo no se cuenta y no influye en el problema puesto que antes de
//entrar en las preguntas aleatorias la sesión count es vaciada.
result count
int 5
result -1
int 4
result mt_rand
int 2
            
                        
 
 //aquí es donde comienza a escoger las preguntas aleatorias                        
                                
 
                iteración 0
result count
int 11
result -1
int 10
result mt_rand
int 0
            
                        
 
                            
                                
 
                            
            
int 0
array (size=1)
  0 => int 0
                fin de la iteración 0
            
 
                iteración 1
result count
int 11
result -1
int 10
result mt_rand
int 10
            
                        
 
                            
                                
 
                            
            
int 10
array (size=2)
  0 => int 0
  1 => int 10
                fin de la iteración 1
            
 
                iteración 2
result count
int 11
result -1
int 10
result mt_rand
int 6
            
                        
 
                            
                                
 
                            
            
int 6
array (size=3)
  0 => int 0
  1 => int 10
  2 => int 6
                fin de la iteración 2
            
 
                iteración 3
result count
int 11
result -1
int 10
result mt_rand
int 0
            
                        
 
result count
int 11
result -1
int 10
result mt_rand
int 5
            
                        
 
                            
                                
 
                            
            
null
array (size=4)
  0 => int 0
  1 => int 10
  2 => int 6
  3 => int 5
                fin de la iteración 3
            
 
            
//esto es del xdebug
( ! ) Notice: Undefined index:  in C:\xampp\htdocs\xampp\web\videosfav\twitter2\control\controller_public.php on line 218
Call Stack
#   Time    Memory  Function    Location
1   0.0010  135904  {main}(  )  ..\index.php:0
2   1.0441  582568  html_start(  )  ..\index.php:23
3   1.0441  582584  play_start(  )  ..\html_public.php:133
 
                iteración 4
result count
int 11
result -1
int 10
result mt_rand
int 0
            
                        
 
result count
int 11
result -1
int 10
result mt_rand
int 3
            
                        
 
                            
                                
 
                            
            
null
array (size=5)
  0 => int 0
  1 => int 10
  2 => int 6
  3 => int 5
  4 => int 3
                fin de la iteración 4
            
 
            
//esto es del xdebug
( ! ) Notice: Undefined index:  in C:\xampp\htdocs\xampp\web\videosfav\twitter2\control\controller_public.php on line 218
Call Stack
#   Time    Memory  Function    Location
1   0.0010  135904  {main}(  )  ..\index.php:0
2   1.0441  582568  html_start(  )  ..\index.php:23
3   1.0441  582584  play_start(  )  ..\html_public.php:133
 

EDITO

La aparición de null no sigue un patrón. Puede aparecer en cualquier lugar y hasta en dos posiciones en una misma ejecución. Nunca tres, hasta el momento. Hay veces que no aparece null y lo realiza tal y como se espera.

[dashtrash]

La aparición de null no sigue un patrón. Puede aparecer en cualquier lugar y hasta en dos posiciones en una misma ejecución. Nunca tres, hasta el momento. Hay veces que no aparece null y lo realiza tal y como se espera.

Sigue un patrón: depende del número de llamas recursivas que hagas a la función.
Fíjate en...

Código PHP:

....
else
            {
                $this->getrandomvalue( $imput );
            }
 ...
 

No debería ser?....

Código PHP:

....
else
            {
                return $this->getrandomvalue( $imput );
            }
 ...
 

[guardarmicorreo]

Iniciado por dashtrash

Sigue un patrón: depende del número de llamas recursivas que hagas a la función.
Fíjate en...

Código PHP:

....
else
            {
                $this->getrandomvalue( $imput );
            }
 ...
 

No debería ser?....

Código PHP:

....
else
            {
                return $this->getrandomvalue( $imput );
            }
 ...
 

Pero cuando esa línea se ejecuta vuelve a comenzar desde arriba la ejecución del método y debería retornar en el if ¿no? :S

De todas formas voy a probarlo.

[guardarmicorreo]

Muchas gracias dashtrash!!

Ese era el problema, el return que no había en esa línea.

Después de 15 pruebas no retorna null.

Pero quiero entender por qué daba ese error, si lo lógico es que esa línea haga comenzar de nuevo la ejecución del método y retorne en el if o vuelva a ejecutar el método. Me gustaría entender el por qué, el motivo. Pero sí, definitivamente esa es la solución a este problema.

[dashtrash]

Si que retorna por el if.Pero, adonde retorna? A la anterior llamada recursiva.No retorna directamente al nivel más alto, donde tú esperas el valor de retorno.
En cuanto hay al menos 1 llamada recursiva, vas a tener en la pila:
[0 : Script que invoca el metodo]
[1: Metodo de busqueda.Produce una llamada recursiva]
[2: Metodo de busqueda.Retorna por el if]

Lo retornado en 2, no va directamente a 0.Va a 1.Y , en 1, el metodo no retorna el valor devuelto.Por lo que devuelve null.

[guardarmicorreo]

Muchas gracias a todos los que me han ayudado con este problema.

Muchas gracias dashtrash por explicarmelo tan bien!! :D

Lo doy por resuelto y es algo más que añado a mis conocimientos, lo valoro más que haber resuelto el problema :P

Gracias!!