Error de logica? Listar cd con canciones

[ccsaiKo]

Hola como están ?
Bueno , espero que puedan ayudar a este humilde joven jaja .
Estoy creando un listado de CD y dentro de cada CD sus respectivas canciones.
Un ciclo FOR para listar los CD y dentro de cada CD UN CICLO CON SUS CANCIONES.

Deberia quedar asi :

CD1
cancion 1 del cd1
cancion 2 del cd1
cancion 3 del cd1

CD2
cancion 1 del cs2
cancion 2 del cs2
cancion 3 del cs2 ... sucesivamente.

Eh aqui el error y me lanza esto :

CD1
cancion 1 del cd1
cancion 2 del cd1
cancion 3 del cd1

CD2
cancion 1 del cd1
cancion 2 del cd1
cancion 3 del cd1
cancion 1 del cs2
cancion 2 del cs2
cancion 3 del cs2 ... y asi pasa con el CD3 Y sucesivos.. donde en el 3 me pone las canciones del c1 cd2 y cd3.


Imagen del script.
http://www.subirimagenes.com/otros-listadocanciones-7146395.html

PD:
Los script de las request a mysql están en otro archivo en sus clases, .. estoy seguro de que lo que falla es un detalle del ciclo.
Muchas gracias.

[linuxzero]

Anda depurando el código, fijate de hacer esto para ver los datos de los discos que trae:

Código PHP:

print_r($discos);
 

Asi nos aseguramos que no este trayendo duplicados en los discos.

Porque en sí el código de la página no esta mal, me huele que el problema esta en

Código PHP:

$discografia->getDiscos();
 

o en:

Código PHP:

$discografia->getCanciones();
 

fijate si podes postear en código de estos 2 métodos.

Saludos!

[ccsaiKo]

Gracias por la ayuda campeon,
eh aqui el script de obtener los discos:
http://www.subirimagenes.com/otros-discos-7149005.html

y aqui de obtener las canciones:
http://www.subirimagenes.com/otros-canciones-7149013.html

[linuxzero]

El código no parece tener nada malo, empeza depurando el codigo haciendo print_r() de los vectores a ver que te esta trayendo, haciendo eso vas a encontrar la solución facilmente.

Probalo y contame ;)

[ccsaiKo]

lo probare :) gracias.

[linuxzero]

Cualquier cosa avisame ;)

[lubtufano]

Me parece que el problema es que el método get_canciones($id_disco) guarda el resultado de la consulta SQL en el array $canciones pero sin borrar los datos guardados en la consulta anterior. Esto es, en el bucle for se llama al método get_canciones() del objeto $discografia la primera vez este método guarda los nombres de las canciones del primer disco en $canciones y luego retorna su valor, en el segundo llamado guarda los nombres de las canciones del segundo disco pero sin eliminar los primeros guardados con lo que al retornar $canciones este array ahora contiene las canciones de los dos discos y en el tercer llamado contendra las canciones de los tres discos y asi sucesivamente.
Una forma de solución es poner $this->canciones= false; al principio del método get_canciones() asi siempre se eliminaria el contenido anterior de $canciones y se retornaria solo las canciones del disco actual.
Un saludo.

[ccsaiKo]

Gracias. problema resuelto.
era lo que me indicaste .. deje al principio del metodo, this->canciones=false;
ahora tengo la siguiente pregunta... por que pasa eso ?.
osea básicamente el metodo de canciones con los discos son los mismos.
...
por que en el de los discos no me guarda los registros anteriores, y en el de las caniones si ?

Te pasaste, muchas gracias.

[Nemutagk]

es muy sencillo, checando tu código estas guardando todas las canciones de todas las peticiones en una variable array, por lo tanto cada vez que haces una nueva petición se agregan las canciones a las peticiones anteriores

Código PHP:

class miclase {
     private $canciones = array();
 
     public function getCanciones() {
          //código SQL, etc...
 
          //lo que haces aquí es agregar un nuevo item al array
          $this->canciones[] = $cancion; 
          //Por lo tanto si ya tienes por ejemplo 10 canciones cuando vuelvas a 
          //hacer otra petición la siguiente canción será el item 11 y así sucesivamente
 
          //En estos casos es mejor una variable local
          //Así no importa las peticiones, siempre estará vacía la variable canciones
          $canciones = array();
          while($cancion = mysql_fetch_assoc($sq)) {
                    $canciones[] = $cancion;
          }
 
          return $canciones;
     }
}
 

ahora, no se porque "$this->canciones" la tienes como variable global (dentro de la clase), a menos que la uses en otro método no le veo sentido, en todo caso la variable debería ser local para no tener estos problemas

por que en el de los discos no me guarda los registros anteriores, y en el de las caniones si ?

Muy sencillo, porque solo haces una petición de los discos disponibles en cambio, las canciones haces una petición por cada disco, si haces mas de 1 petición a los discos, tendrás el mismo problema...

[ccsaiKo]

extramadamente agradecido, comprendi mi gran error.
algo nuevo que aprendi haha.. entonces.. si la variable solo se usa en un metodo, no tengo para que tenerla global. ya que no se ustilizara en otros metodos.

entonces, el metodo construct esta demas y el array global tambien.
Mil gracias campeones !