[SOLUCIONADO] Después de un UPDATE no continúa WHILE

[Benderzuelo]

Hola amig@s,

tengo un problema que no consigo resolver, tengo el siguiente SCRIPT

Código PHP:

$sql = "SELECT rn.*, s.saldo FROM renovaciones rn 
            LEFT JOIN saldo s 
            ON s.idUsuario = rn.idUsuario 
            AND rn.activo = 1";
            $cnx = new Mysql(_SERVER, _USER, _PASSWORD, _BBDD);
            
            $periodo = array('0'=>0, '1'=>'1 HOUR', '2'=>'2 HOUR', '3'=>'3 HOUR', '4'=>'4 HOUR', '5'=>'5 HOUR', '6'=>'6 HOUR', '10'=>'1 DAY', '100'=>'2 DAY');
            $intervalo = '';
            
            if($cnx->ejecutaSql($sql)){
                //Recorro todos los resultado
                while($row = $cnx->asociaFilas()){
                    //Si su saldo es mayor o igual a 0.30 que es el mínimo, renuevo si no no.
                    if($row['saldo'] >= 0.3 && $row['saldo'] != NULL){
                        //Preparo el intervalo dinámico para la consulta desde el array anterior
                        $intervalo = $periodo[$row['periodo']];
                        //Si la fecha de configuración y la de renovado son iguales es porque aún no ha renovado nunca
                        if($row['fecha'] == $row['fechaRenovado']){
                            //Entonces renuevo sin comprobar el intervalo
                            $sqlUpdate = "UPDATE renovaciones SET fechaRenovado = now() WHERE idUsuario = ".$row['idUsuario']." AND idAnuncio = ".$row['idAnuncio'];
                        }else{
                            //Si no compruebo el intervalo para renovar de nuevo
                            $sqlUpdate = "UPDATE renovaciones SET fechaRenovado = now() WHERE idUsuario = ".$row['idUsuario']." AND idAnuncio = ".$row['idAnuncio']." AND NOW() >= DATE_ADD(fechaRenovado, INTERVAL ".$intervalo.") ";
                        }
                        //Preparo las consultas para actualizar el anuncio y el saldo del cliente
                        $sqlUpdateAnuncio = "UPDATE anunciosCV SET time = ".time()." WHERE idUsuario = ".$row['idUsuario']." AND idAnuncio = ".$row['idAnuncio'];
                        $sqlUpdateSaldo = "UPDATE saldo SET saldo = ROUND(saldo-0.3,1) WHERE idUsuario = ".$row['idUsuario'];
                        //Si actualiza la renovacion entonces continuo si no muestro el fallo
 
 
                                                 //EN ESTA CONSULTA DEJA DE IR EL WHILE, PERO LA CONSULTA ES CORRECTA
 
                        if($cnx->ejecutaSql($sqlUpdate)){
                            //Si hay filas afectadas es por que el intervalo se ha cumplido si no muestro error
                            if($cnx->filasAfectadas() == 1){
                                //echo $cnx->filasAfectadas();
                                //Si actualiza el anuncio continuo si no muestro fallo
                                if($cnx->ejecutaSql($sqlUpdateAnuncio)){
                                    //Actualizo tabla de saldos
                                    if($cnx->ejecutaSql($sqlUpdateSaldo)){
                                        echo 'Actualizado el anuncio '.$row['idAnuncio']." del Usuario ".$row['idUsuario'].' en el intervalo '.$intervalo.'<br>';
                                    }else{
                                        echo 'Actualizado el anuncio '.$row['idAnuncio']." del Usuario ".$row['idUsuario'].' en el intervalo '.$intervalo.' pero no se descontó el saldo<br>'.$sqlUpdateSaldo;
                                    }
                                }else{
                                    echo 'Falla la consulta'.$sqlUpdate.'<br>';
                                }
                            }else{
                                echo 'No se actualiza por no cumplir el intervalo '.$intervalo.' con la consulta '.$sqlUpdate.'<br>';
                            }
                        }else{
                            echo 'Falla la consulta'.$sqlUpdateAnuncio.'<br>';
                        }
                    }else{
                        echo 'El Usuario '.$row['idUsaurio'].' no tiene saldo, el intervalo es '.$intervalo.'.<br>';
                    }
                }
            }else{
                echo 'Falla la consulta '.$sql.'<br>';
            }
 

En la consulta que he marcado en MAYÚSCULAS , si se ejecuta el update, no continúa con el WHILE, hay 3 registros para las pruebas y deberá de actualizar o no.. 3 registros, pero después de realizar la consulta no continua el WHILE, y la consulta está bien porque la imprimo y la compruebo directamente en phpmyadmin y va perfecta.

Si hay un echo 'vueltas' antes de ejecutar esa consulta y evidentemente en el código omito la consulta, me imprime 3 vueltas, sin embargo si no la omito solo da una vuelta.

He comprobado todas las consultas y todas van bien, se ejecutan y todo bien pero en la acción se corta el while..

A alguien se le ocurre algo?

Muchas gracias por adelantado.

[pateketrueke]

¿Es mi imaginación o reutilizas el mismo contexto/instancia de la primer consulta para derivar las siguientes?

No sé como hayas diseñado tu capa de interacción pero eso que haces huele mal, cada consulta debería tener su propia instancia para evitar colisiones.

[Benderzuelo]

¿Cómo?, entonces si hago 20 consultas en una porción de código tengo que abrir conexión 20 veces?, la verdad es que es algo que he hecho así de toda la vida y jamás he tenido problemas.

Aún así, nunca llega a la segunda consulta, en el momento de la ejecución de la primera para el bucle.

Igualmente lo voy a probar

[Benderzuelo]

Probado!!, Pasa lo mismo

Igualmente muchas gracias.

[pateketrueke]

Nadie dijo conexiones, estoy hablando de instancias, si tu programación es tan mala que necesita una conexión nueva por instancia entonces ya veo tu preocupación.

La cosa, es que usas la misma instancia para cada sub-consulta: ese podría ser el problema

¿Por qué?

Pues depende mucho de cómo hayas implementado dicha clase, si está bien hecha no debería preocuparte el tema de las conexiones.

Pero si está mal hecha entonces ya veo el punto de querer reusarla a más no poder.

Sin conocer exactamente qué hace la clase de tus consultas no puedo decir más.

Sólo interpreto lo que alcanzas a mostrar, nada más.

[Benderzuelo]

Ok, bueno alguien que quiera ayudar en vez de criticar?

[xfxstudios]

no se como tengas tus clases, pero porque tanto $cnx???? estas seguro que no es alli el problema pues creo que a eso se refiere nuestro amable amigo pateketrueke hahaha, pero bueno, revisa eso, recuerda que en php cada instancia es unica asi este llamando a la misma funcion o clase:

Código PHP:

$a = new NombreClase("Primero");
$b = new NombreClase("Segundo");
$c = new NombreClase();
 

ahora si hacemos esto:

Código PHP:

$a = new NombreClase("Primero");
$a = new NombreClase("Segundo");
$a = new NombreClase();
 

se sobreescribe la instancia hasta donde se

[Benderzuelo]

Muy amable xfxstudios

Instancia solo hay una, la llamo $cnx pero bueno llamarla $obj para que nos entendamos mejor y lo que hay en cada vez es ejecutar un $obj->ejecutaSql, equivanlente a mysqli_query();

Creo que es a eso a los que te refieres...

yo lo que hago es llamar varias veces a un método, no estoy instanciando varias veces el mismo objeto.

[pateketrueke]

Lo de la crítica fue únicamente porque sencillamente asumes que todo es correcto, pero prefieres no mirar hacia tu propio código.

yo lo que hago es llamar varias veces a un método, no estoy instanciando varias veces el mismo objeto.

El asunto es que puedes estar sobreescribiendo referencias internas, y con eso cancelas todos los punteros a la consulta previa, etc.

Creo que lo mejor sería que muestres el código de dicha clase y nos dejemos de adivinanzas ¿no te parece?

[Benderzuelo]

Código PHP:

class Mysql {
    
    private $server; // Servidor donde está alojada la BBDD
    private $user;   // Usuario que se conecta a la BBDD
    private $passwd; // Contraseña del usuario que se conecta a la BBDD
    private $bbdd;   // Nombre de la BBDD a la que se quiere conectar
    private $link;   // Puntero a la BBDD (por si se abre conexión a varias BBDD)
    private $error;  // Guarda el resultado de mysql_error()
    
function __construct($server, $user, $passwd, $bbdd) {
        $this->server = $server;
        $this->user   = $user;
        $this->passwd = $passwd;
        $this->bbdd   = $bbdd;
        $this->conecta();
    }
    
   private function conecta() {
        $this->link = mysqli_connect($this->server, $this->user, $this->passwd);
        mysqli_select_db($this->link,$this->bbdd);
        mysqli_query($this->link, "SET NAMES utf8"); 
    }
    
    
public function muestraError() {
        return $this->error;
    }
    
 
public function ejecutaSql($qry) {
        if (!$this->res = mysqli_query($this->link, $qry)) {
            //Preparo el error
            $this->error = mysqli_error($this->link);
            return false;
        }else{
            return true;
        }
    }
    
    
public function asociaFilas() {
        if (!$row = mysqli_fetch_assoc($this->res)) {
            $this->error = mysqli_error($this->link);
            return false;
        }else{
           return $row;
        }
    }
    
public function numeroFilas() {
        return mysqli_num_rows($this->res);
    }
    
    
public function filasAfectadas() {
        return mysqli_affected_rows($this->link);
    }
    
    
    
 
public function ultimoID() {
        return mysqli_insert_id($this->link);
    }
 
    
public function cierraCNX(){
        if(mysqli_close($this->link)){
            return true;
        }else{
            return false;
        }
    }
}
 

[kip13]

Hola, que te parece si en lugar de crear $cnx, e instanciarlo, mejor creas un metodo tal como ejecutaSql(), y lo inicias en el script, por ejemplo conectar_db() que retorne la conexion y de esa manera al momento de llamar al metodo ejecutaSql() no tendrias que usar ninguna instancia creada, ya que mantendrias la conexion a la base abierta desde el principio.

Y como dice pateketrueke de esa forma evitaras sobreescribir cualquier referencia interna, que quizas se este dando en este caso.

[pateketrueke]

Dicho y hecho: con cada consulta que se ejecuta sobreescribes la referencia $this->res inutilizando la consulta que ya se había almacenado ahí.

Definitivamente debes plantear una mejor solución, ya sea con funciones, un mejor entendimiento de orientación a objetos o cómo prefieras.

[Benderzuelo]

Llevo poco tiempo con POO pero de todos modos aunque puedas tener razón cuando sobreescribo $this->res ya no vuelvo a necesitar lo que había antes, y no es el problema que tiene este script, de hecho hay un portal entero de anuncios que funciona así sin ningún problema en absoluto.

Más ideas?

[pateketrueke]

Bueno, no se trata de que tu uses dicha referencia, la usa tu clase, el método ->asociaFilas() lee dicha referencia para continuar con el while(), entonces para cuando ejecutas la sub-consulta dentro del while() la consulta principal ya no funciona: porque ya no es la misma referencia.

¿En el portal que has hecho previamente abordas exactamente el mismo problema que aquí planteas?

De ser así ya lo has resuelto entonces ¿para qué abrir un tema en este foro si eso ya lo resolviste en tu portal tiempo atrás?

Pero bueno, así como respondes parece que eres un experto y no cometes errores: suerte con tu problema.

Más ideas?

Te estoy explicando tu problema pero te aferras a pensar que el problema no es ese, así no vamos a llegar a ningún lado.

[Benderzuelo]

Yo no soy experto pero parece que tu si y que tienes que dejar claro que eres dios para dormir bien por la noche.

He pedido ayuda, y si no estuviera seguro de lo que digo no te llevaría la contraria, al margen de eso si te llevo la contraria lo hago con respeto y no con ironias.

Cuando ejecuto el while, el resultado de asociafilas está guardado en $row por lo tanto no pasa nada, te digo que tengo estructuras exactamente iguales pero con consultas diferentes y funcionan.

Si te apetece seguir criticando sigue. Yo no llegaré a tu nivel de Dios en programación pero tampoco voy a bajar al nivel de empezar a discutir contigo puesto que no me aporta nada.

[xfxstudios]

Bueno fuera de lo que pasa, la respuesta ya te la dieron, yo tengo clases y funciones para ejecutar querys y no he tenido nunca un problema asi, pero cuando dices que tienes estructuras iguales a esta y te funcionan, entonces por que esta no???, revisa las otras estructuras y compara, asi podrás ver lo que no podemos nosotros al no estar frente a tu monitor.

Nadie esta discutiendo y nadie es Dios aquí, aunque no lo creas, cuando algún compañero te emite una respuesta es porque tiene media o completa idea de lo que esta respondiendo o ya a solventado problemas así, hasta yo aprendo de las respuestas que leo a diario, y lo que consideras critica, en realidad es la solución, pero todavía no la vez o tal vez es algún ; que no hemos visto, nadie discute, es solo que la diferencia esta en como interpretas lo que escribe cada quien y como lo escribe lo que te da a pensar eso, cuando no es así.

[Benderzuelo]

Puede que lleves razón pero hay formas y formas de decir las cosas, se puede decir lo mismo sin molestar a la persona que pregunta y llevando tanto tiempo como llevas en el foro en el fondo sabes que tengo razón en lo que estoy diciendo.

Hay mucha gente en el foro que para ayudar a parte de ofrecer su ayuda, hacen comentarios sobre los conocimientos de las personas que preguntan y no precisamente constructivamente, o comentarios humillantes hacia los códigos de los que preguntan.

Si no hubiera gente que tiene dudas no existiría este foro.

de todos modos, ya está... no lo voy a alargar más.

intentaré en otro foro.

Doy por cerrado el tema.

Gracias a todos por vuestras respuestas.

[pateketrueke]

Cuando ejecuto el while, el resultado de asociafilas está guardado en $row por lo tanto no pasa nada, te digo que tengo estructuras exactamente iguales pero con consultas diferentes y funcionan.

Te explico:

Sí, el resultado de asociaFilas() lo guardas en $row ¿pero entiendes cómo funciona internamente un while() y la relación implícita con tu método?

La cosa, es que en cada vuelta del while() se vuelve a invocar asociaFilas(), cada vez.

Por eso cuando eliminas la sub-consulta funciona de maravilla, pero dado que la sub-consulta altera las referencias internas de toda clase todo se rompe ahí.

Para que todo funcione como tienes en mente debes evitar sobre-escribir dichas referencias, ni más ni menos.

He pedido ayuda, y si no estuviera seguro de lo que digo no te llevaría la contraria, al margen de eso si te llevo la contraria lo hago con respeto y no con ironias.

También recuerda que estás en un foro de discusión, no es yahoo answers, ni taringa.

Si me quieres llevar la contraria sin argumentos bien por ti, pero te platico que tengo mucha más experiencia haciendo OOP y por eso el problema es muy fácil de identificar para mi.

Te sugiero que leas bien como funcionan las estructuras de un while(), para que veas como cada vez se ejecuta la función que devuelve un registro, etc.

Una solución directa podría ser pre-acumular en un array los registros dentro de la clase con un método, y entonces usar dicho array por fuera: así podrás ejecutar cientos de consultas con la misma clase sin chocar referencias entre si:

Código PHP:

// este método va en tu clase
public function allRows() {
  $out = [];
  // esto es lo mismo que ya haces, pero de forma más discreta
  while ($row = $this->asociaFilas()) {
    $out []= $row;
  }
  return $out;
}
 
// ...
$cnx->ejecutaSql($sql1);
$data = $cnx->allRows();
 
foreach ($data as $row) {
  $cnx->ejecutaSql($sql2);
  $subdata = $cnx->allRows();
 
  foreach ($subdata as $subrow) {
    // ...
  }
}
 

En dicho ejemplo usé foreach() que es más claro cuando trabajas sobre estructuras ya hechas, que a diferencia de while() no necesita llamar frecuentemente a la misma expresión, etc.

intentaré en otro foro.

Bueno, si eres de los que sólo ven el árbol cuando hay detrás un bosque entero entiendo tu partida.

Suerte y mucho éxito en lo tuyo.

[Benderzuelo]

De acuerdo pateketrueke, esto si lo veo una respuesta constructiva y te lo agradezco.

Mañana lo probaré y te comentaré el resultado.

[Benderzuelo]

pateketrueke, debo decir que tenías razón, y he revisado las otras estructuras que tengo en el portal en las cuales yo me apoyaba para rebatirte y si, son iguales pero lo que no vi y no me fijé es en que las consultas en las que se hace el WHILE, tienen una conexión distinta a las que se ejecutan dentro del while, por eso no fallaba, pero yo eso no lo vi.

Es un trabajo heredado, pensaba que lo conocía bien pero no es así.

Por lo tanto, te pido disculpas, y acepto que tenías razón.