[SOLUCIONADO] Problema con INNER JOIN

[dankko]

Hola, he realizado una búsqueda por el foro, pero no logro arreglar el fallo que tengo en esta sentencia SQL:

Pretendo sacar como resultado de estas cuatro tablas:

Código:

torneos
(
id_torneo
nombre_torneo
)

torneos_rondas
(
id_ronda
nombre_ronda
)

torneos_inscripcion
(
user_id
username
id_torneo
)

torneos_enfrentamientos
(
id_enfrentamiento
id_ronda
user_id_1
user_id_2
id_torneo
)

Resultado:

Enfrentamiento ronda 1

Usuario 1 - Usuario 2

(donde torneos_enfrentamientos.user_id_1 = torneos_inscripcion.user_id y donde torneos_enfrentamientos.user_id_2 = torneos.inscripcion.user_id)

La consulta que estoy sacando es esta:

Código:

$ssql = "SELECT username, id_enfrntamiento,id_ronda FROM torneos_enfrentamientos
	INNER JOIN torneos_inscripcion ON torneos_inscripcion.user_id = torneos_enfrentamientos.user_id_1 
ON torneos_rondas.id_ronda = torneos_enfrentamientos.id_ronda
	where torneos_enfrentamientos.id_torneo = '$id_torneo'";

Pero tengo algún error, ya que no logro sacar el resultado...

[Triby]

En tu select tienes id_enfrntamiento, creo que falta una e

Además, cuando una consulta no te funciona, siempre es bueno saber porqué; supongamos que estás usando la vieja y obsoleta librería mysql:

Código PHP:

$consulta = '/* Aquí tu consulta */';
$resultado = mysql_query($consulta) or die('Error en consulta: ' . mysql_error());
 

Con eso debes obtener un mensaje de error muy explicativo. Si no, entonces imprime la consulta y pruébala en PhpMyAdmin, ahí podrás ir afinando detalles hasta tenerla funcionado correctamente.

Agrego: También tienes un ON sin JOIN previo en la tercera línea de la consulta.

[dankko]

Hola Triby, gracias por responder...

Código:

$ssql = "SELECT username, id_enfrentamiento,id_ronda FROM torneos_enfrentamientos
	INNER JOIN torneos_inscripcion ON torneos_inscripcion.user_id = torneos_enfrentamientos.user_id_1 
	INNER JOIN torneos_inscripcion ON torneos_inscripcion.user_id = torneos_enfrentamientos.user_id_2
	INNER JOIN torneos_rondas ON torneos_rondas.id_ronda = torneos_enfrentamientos.id_ronda
	where torneos_enfrentamientos.id_torneo = '$id_torneo'";

El error que me produce es el siguiente:

[phpBB Debug] PHP Warning: in file C:/wamp/www/.../torneo-rondas.php on line 21: mysql_num_rows() expects parameter 1 to be resource, boolean given

[dankko]

Pongo la sentencia completa para veais donde se encuentra el error... Me lo marca en la parte en al que le digo que si no hay resultados mostrar una notificacion, pero seguramente esté en la consulta:

Código:

$conexion = conecta_base_datos();
$id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();
$ssql = "SELECT * FROM torneos_enfrentamientos
	INNER JOIN torneos_inscripcion ON torneos_inscripcion.user_id = torneos_enfrentamientos.user_id_1
	INNER JOIN torneos_rondas ON torneos_rondas.id_ronda = torneos_enfrentamientos.id_ronda		
	where torneos_enfrentamientos.id_torneo = '$id_torneo'";			
$rs = mysql_query($ssql);
if(mysql_num_rows($rs)> 0){
while ($fila = mysql_fetch_array($rs)){

echo 'partido nº ' . $fila["id_partido"] . '<br>';

echo '<a href="' . $fila["user_id_1"] . '">' . $fila["username"] . '</a>';
	echo ' - ';
echo '<a href="' . $fila["user_id_2"] . '">' . $fila["username"] . '</a><br>';

	
}
}else{
    //es que no se encontraron anuncios
    echo'<div class="alert alert-error">Aún no se han generado las rondas del torneo.</div>';
}

[Triby]

Si agregas la parte del or die() que te sugerí, ahí vas a ver el mensaje de error originado por la consulta.

[gnzsoloyo]

Separando la consulta que usas (y simplificándola un poco con uso de alias):

Código MySQL:

SELECT 
    *
FROM
    `torneos_enfrentamientos` TE
        INNER JOIN
    `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`
        INNER JOIN
    `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda`
where
    TE.`id_torneo` = '$id_torneo'
 

No existen errores sintácticos, por lo que si el PHP devuelve ese error es porque o la variable $id_torneo no contiene datos, o no hay registros que cumplan con las relaciones.

[dankko]

Iniciado por Triby

En tu select tienes id_enfrntamiento, creo que falta una e

Además, cuando una consulta no te funciona, siempre es bueno saber porqué; supongamos que estás usando la vieja y obsoleta librería mysql:

Código PHP:

$consulta = '/* Aquí tu consulta */';
$resultado = mysql_query($consulta) or die('Error en consulta: ' . mysql_error());
 

Con eso debes obtener un mensaje de error muy explicativo. Si no, entonces imprime la consulta y pruébala en PhpMyAdmin, ahí podrás ir afinando detalles hasta tenerla funcionado correctamente.

Agrego: También tienes un ON sin JOIN previo en la tercera línea de la consulta.

Perdón! estaba seguro de la que la tenía puesta, pero de todas formas está devolviendo el error porque no encuentra resultados...

Pero me parece raro porque

Le estoy pasando a través de $_GET[id] el ID del torneo y le doy el valor del GET a la variable $id_torneo.

[dankko]

Iniciado por gnzsoloyo

Separando la consulta que usas (y simplificándola un poco con uso de alias):

Código MySQL:

SELECT 
    *
FROM
    `torneos_enfrentamientos` TE
        INNER JOIN
    `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`
        INNER JOIN
    `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda`
where
    TE.`id_torneo` = '$id_torneo'
 

No existen errores sintácticos, por lo que si el PHP devuelve ese error es porque o la variable $id_torneo no contiene datos, o no hay registros que cumplan con las relaciones.

Lo raro es que si que hay registros que cumplen los requisitos:

Tabla torneos_enfrentamientos:

Código:

id_enfrentamiento | id_torneo | id_ronda | user_id_1 | user_id_2 | enfrentamiento_jugado
2 	7 	1 	63 	2 	NO 
3 	7 	1 	63 	2 	NO 
4 	7 	2 	2 	63 	NO

Y esto es lo que le pasa través de la URL, el ID del torneo, que es a lo que llamo con la variable $id_torneo: http://www.../torneo-rondas.php?id=7

Además, de como podría hacer para poder sacar el username correspondiente a user_id_1 y el username correspondiente a user_id_2, ya que al hacer INNER JOIN torneos_enfrentamientos ON torneos_enfrentamientos.user_id_1 = torneos_inscripcion.user_id, lo que está haciendo es darle el mismo nombre tanto a user_id_1 y a user_id_2.

[dankko]

Alguna idea?

[gnzsoloyo]

Iniciado por dankko

Lo raro es que si que hay registros que cumplen los requisitos:

Tabla torneos_enfrentamientos:

Código:

id_enfrentamiento | id_torneo | id_ronda | user_id_1 | user_id_2 | enfrentamiento_jugado
2 	7 	1 	63 	2 	NO 
3 	7 	1 	63 	2 	NO 
4 	7 	2 	2 	63 	NO

Y esto es lo que le pasa través de la URL, el ID del torneo, que es a lo que llamo con la variable $id_torneo: http://www.../torneo-rondas.php?id=7

Además, de como podría hacer para poder sacar el username correspondiente a user_id_1 y el username correspondiente a user_id_2, ya que al hacer INNER JOIN torneos_enfrentamientos ON torneos_enfrentamientos.user_id_1 = torneos_inscripcion.user_id, lo que está haciendo es darle el mismo nombre tanto a user_id_1 y a user_id_2.

la tabla de muestra resulta insuficiente para saber si las condiciones se cumplen.
Para verificarlo deben cumplirse los dos INNER JOIN al mismo tiempo, además de tener un id_ronda igual al indicado.
Lo que hay que verificar es primero qué es lo que realmente MySQL está recibiendo.
Ponlo de esta forma y pruébalo:

Código PHP:

$id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();
$ssql = "SELECT *
FROM `torneos_enfrentamientos` TE
        INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`
        INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda`
WHERE TE.`id_torneo` = $id_torneo";
// verificando la query
echo $ssql; 
$rs = mysql_query($ssql);
 

Luego, si muestra el valor del ID buscado en la consulta, toma la query completa y ejecutala en phpMyadmin.
Una vez hecho, veremos lo que da.

[dankko]

Código:

torneos_enfrentamientos

id_enfrentamiento | id_torneo | id_ronda | user_id_1 | user_id_2 | enfrentamiento_jugado
2 	7 	1 	63 	2 	NO 
3 	7 	1 	63 	2 	NO 
4 	7 	2 	2 	63 	NO

torneos_inscripcion

id (de la inscripcion) | user_id | username | nombre_real | email | estado | fecha | id_torneo

	223 	63 	pepito pepito periquito [email protected] 	SP 	2013-05-19 	7
	224 	2 	pepito2 pepit2o periquito2 [email protected] 	SP 	2013-05-19 	7

Te adjunto las dos tablas para ver que puede ser el dato que no esté relacionado.

He probado pero no devuelve nada, devulve el mensaje cuando no hay consultas que cumplan la función... Pero lo que no entiendo es que si hay datos que cumplen todo lo que se indica con INNER JOIN a la vez

En torneos_enfrentamientos > user_id_1, user_id_2 > user_id
En realidad lo que quiero es mostrar el aprtido correspondiente a la ronda con el ID = 1...

Como si se tratará de mostrar un partido de fútbol entre dos equipos

[gnzsoloyo]

Te adjunto las dos tablas para ver que puede ser el dato que no esté relacionado.

Las tablas no me sirven de mucho si no me das el CREATE TABLE que las genera para entender las relaciones.
Además, no me estás dando lo que te pedí: Lo que te muestra cuando pones:

Código PHP:

// verificando la query
echo $ssql;
 

Es una buena costumbre que cuando se te pide una cierta información claramente definida, aportes eso y no otras cosas.
Para que se entienda: Que tu estés seguro de que tu código y los formularios hacen algo, no quiere decir que sea realmente así. TE sorprendería la cantidad de veces que han insistido que todo está bien, y luego con un simple echo se ve que no es lo que creen (o si, pero con condiciones que generan error).

Haz ese echo, y postea lo que muestra en pantalla.
Luego de eso copialo y pruébalo en phpMyadmin, y postea el resultado.

Simplemente para que nosotros estemos seguros.

[dankko]

Perdona, te había entendido mal, no había visto lo de hacer un echo al $ssql. Ahora ya se lo que me quieres decir. Lo probaré.

Edit. El archivo php Me devuelve lo siguiente:

Código:

SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` WHERE TE.`id_torneo` = '7'SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` WHERE TE.`id_torneo` = '7'SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` WHERE TE.`id_torneo` = '7'SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` WHERE TE.`id_torneo` = '7'SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` WHERE TE.`id_torneo` = '7'SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` WHERE TE.`id_torneo` = '7'

En phpmyadmin me da el siguiente error del sintaxis:

Código:

#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT * FROM `torneos_enfrentamientos` TE INNER JOIN `torneos_inscripcion` TI ON TI.`u' at line 1

[gnzsoloyo]

Disculpa la carcajada, pero evidentemente algo estás haciendo mal. Bastante mal.
Te muestro lo que está quedando en tu query, poniéndola más estructuradamente de modo que se vea bien lo que está en ella:

Código MySQL:

SELECT * 
    FROM `torneos_enfrentamientos` TE 
        INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
        INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
     WHERE TE.`id_torneo` = '7'
 
        SELECT * 
        FROM `torneos_enfrentamientos` TE 
            INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
            INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
        WHERE TE.`id_torneo` = '7'
 
            SELECT * 
            FROM `torneos_enfrentamientos` TE 
                INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
                INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
            WHERE TE.`id_torneo` = '7'
 
                SELECT * 
                FROM `torneos_enfrentamientos` TE 
                    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
                    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
                WHERE TE.`id_torneo` = '7'
 
                    SELECT * 
                   FROM `torneos_enfrentamientos` TE 
                        INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
                        INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
                   WHERE TE.`id_torneo` = '7'
 
                            SELECT * FROM `torneos_enfrentamientos` TE 
                            INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
                            INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
                            WHERE TE.`id_torneo` = '7'
 

Lo ves más claramente?
¡Esa consulta repite cinco (5) veces la consulta encadenando una detrás de la otra!
Obviamente o hay un loop indebido en alguna parte, o la variable no se está reinicializando donde debe.
La verdadera consulta, descartada toda la basura, es:

Código MySQL:

SELECT * 
FROM `torneos_enfrentamientos` TE 
    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
WHERE TE.`id_torneo` = '7';
 

Obviamente no puedo decirte exactamente dónde estas cometiendo el error, porque en el código que no has posteado no hay nada que pueda estar generando este error de construcción de la consulta...
En cualquier caso, para poder seguir con lo que iba diciendo, toma lo que te estoy posteando al final, y prueba ESO en phpMyadmin, a ver qué es lo que la consulta realmente debería devolver.
En cuanto al resto, hay que ver por qué se está generando tal desfasaje. ¿Podrías postear el código completo que usaste para esta prueba?

[dankko]

He cambiado el nombre a la tabla torneos_enfrentamientos por torneos_partidos.

Código:

<?php
$conexion = conecta_base_datos();
$id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();

$ssql = "SELECT *
FROM `torneos_partidos` TE
       INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`
       INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda`
WHERE TE.`id_torneo` = '$id_torneo'" or die('Error en consulta: ' . mysql_error());			
$rs = mysql_query($ssql);
if(mysql_num_rows($rs)> 0){
while ($fila = mysql_fetch_array($rs)){

    // verificando la query
    echo $ssql;
}
}else{
    //es que no se encontraron anuncios
    echo'<div class="alert alert-error">Aún no se han generado las rondas del torneo.</div>';
}
?>

El resultado de la consulta correcta devuelve lo siguiente:

[dankko]

Te lo adjunto en imagen para que sea más claro.

[gnzsoloyo]

Bueno, por empezar (ahora no tengo tiempo de verlo más profundamente, luego me pongo a analizar el resultado del query), al menos ya sabemos por qué se replicó la consulta:
¿Me puedes decir por qué pusiste el echo dentro del while?
Si te fijas con cuidado, yo lo puse inmediatamente después de cargar la consula en la variable y antes de enviarla a la base.

En definitiva, ese ha sido un fallo accidental, pero tenlo en cuenta: Las cosas no se ponen en cualquier parte...

Respecto al resultado de la consulta, lo que puedo ver es que te devuelve resultados indebidos o aparentemente duplicado, porque estás filtrando por id_ronda solamente, pero no lo estás haciendo por id_torneo, ni relacionando los torneos, con los que se retornan todas las rondas Nº 7, sin importar a qué torneo pertenecen... y eso no creo que esté bien.

Yo probaría esto:

Código MySQL:

SELECT * 
FROM `torneos_enfrentamientos` TE 
    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1` 
    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` AND TR.`id_torneo` = TE.`id_torneo`
WHERE TE.`id_torneo` = '7';
 

[dankko]

Porque al ver el echo por inercia lo puse debajo del while, donde tenía el resultado de la muestra de partidos.

La forma que uso para relacionarlo con el torneo es darle al valor $id_torneo el valor enviado por get

Código:

$id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();

Entonces lo que hace es devolverme las rondas del torneo, los partidos, y los usuarios que están en el torneo con el ID = 7. Pero es la única forma con la que lo relaciono...

He probado con tu consulta pero no me arroja ningún dato. Jugando con dicha consulta he probado a hacer lo siguiente, aunque es un poco chapucilla pero igual puede ayudar en algo o darnos alguna pista:

Código:

<?php
$conexion = conecta_base_datos();
$id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();

$ssql = "SELECT * 
FROM `torneos_partidos` TE 
    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`
    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` 
WHERE TE.`id_torneo` = '$id_torneo' and TE. id_ronda = '1'" or die('Error en consulta: ' . mysql_error());			
$rs = mysql_query($ssql);

while ($fila = mysql_fetch_array($rs)){

    // verificando la query
	echo '<a href="' . $fila[user_id_1] . '">' . $fila[username] . '</a>';
	echo ' vs ';
	echo '<a href="' . $fila[user_id_2] . '">' . $fila[username] . '</a><br>';
}

?>

He pensado en hacer varias consultas por ronda, para así sacar el número de partidos que corresponden a cada ronda del torneo indicado por la variable $id_torneo...

El resultado que me ofrece es el siguiente:

Código HTML:

dankko2 vs dankko2
dankko2 vs dankko2
dankko2 vs dankko2
dankko2 vs dankko2
 

Parece que está mostrando un resultado correcto, ya que muestra los cuatro partidos que corresponden a la ronda con el ID=1 que están en el torneo con el ID=7.

Ya te digo, es un poco chapucillas pero igual puede ayudar a descifrar algún error.

[dankko]

Puede ser que esto último que realicé me funcione aunque tenga que mostrar ronda por ronda, pero no se como podría hacer para que a user_id_1 y user_id_2 los relacione con su correspondiente "username" de la tabla de usuarios.

La estructura para esta tabla es simple

Código:

user_id | username |        email                | avatar |
2           dankko         [email protected]      one.jpg
63         dankko2       [email protected]      ...jpg

La relación tambien se hace con la tabla de torneos_inscripciones que llama a user_id y a la tabla partidos, en la que user_id_1 corresponde a al user_id de cada usuario...

[dankko]

Alguna idea?

[gnzsoloyo]

Puede ser que esto último que realicé me funcione aunque tenga que mostrar ronda por ronda, pero no se como podría hacer para que a user_id_1 y user_id_2 los relacione con su correspondiente "username" de la tabla de usuarios.

Es medio de manual: Requiere invocar dos veces a la tabla usuario, a fin de relacionar cada invocación con uno de los ID. Obviamente requiere el uso de alias para cada llamada.

Código MySQL:

<?php
$conexion = conecta_base_datos();
$id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();
 
$ssql = "SELECT TI.id_torneo, TR.id_ronda, TR.nombre_ronda, TE.id_enfrentamiento , TE.`user_id_1`, U1.username User1, TE.`user_id_2`, U2.username User2 ";
$ssql .= "FROM `torneos_partidos` TE ";
$ssql .= "    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`";
$ssql .= "    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` ";
$ssql .= "    INNER JOIN `usuarios` U1 ON TE.`user_id_1` = U1.user_id";
$ssql .= "    INNER JOIN `usuarios` U2 ON TE.`user_id_1` = U1.user_id";
$ssql .= "    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` ";
$ssql .= "WHERE TE.`id_torneo` = '$id_torneo' ";
$ssql .= " ORDER BY id_torneo, TR.id_ronda, U1.username, U2.username";          
 
echo $ssql;
 
$rs = mysql_query($ssql) or die('Error en consulta: ' . mysql_error());
 
while ($fila = mysql_fetch_array($rs)){
 
    // verificando la query
    echo '<a href="' . $fila[user_id_1] . '">' . $fila[User1] . '</a>';
    echo ' vs ';
    echo '<a href="' . $fila[user_id_2] . '">' . $fila[User2] . '</a><br>';
}
 
?>
 

Algunas notas:
1) Por cuestiones de eficiencia y eficacia, es mejor que no uses "*" para hacer consultas, especialmente con JOINS, ya que duplican las columnas y traen mucho que no vas a usar. Es preferible invocar uno a uno los campos a usar. Por eso estoy poniendo la lista de campos en el SELECT, a modo de sugerencia.
2) Para etiquetar el código existen los Highlights en la edición. No pongas "CODE", usa el Highlight correspondiente al código que posteas (PHP, en este caso), de ese modo el contenido se resalta mejor y algunos errores notorios se perciben a simple vista.
3) Poner los dos parámetros buscados (ronda y tornoe), no es una "chapucería", como dices. Es necesario, porque un sólo parámetro, como te hice notar sobre el JOIN, no es suficiente para obtener los datos. En el caso del JOIN, porque diferentes torneos pueden tener iguales números de ronda, con lo que si no pones ambas condiciones terminas haciendo un producto cartesiano parcial... lo que es un espanto.

PD: No he revisado la sintaxis del SQL que te posteo, por lo que puede contener errores.
No hagas Copy+Paste, eso no funciona jamás. Verifica que esté todo bien escrito, y verifica, sobre todo, que el SQL quede bien armado, haciendo un echo como ya te he indicado.
Nunca asumas que algo está bien porque crees que está bien o porque lo hizo alguien que conoce más.
Todos cometemos errores (o sino los médicos no tendrían problemas de mala praxis).

[dankko]

Me refería a chapuza al tener que usar varias llamadas a la consulta pero con diferentes id_ronda y poniendo el nombre de la ronda fuera de la consulta.

He probado con tu consulta, haciendole algunas modificaciones porque tenía algún error:

Código PHP:

<?php
    $conexion = conecta_base_datos();
    $id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();
     
    $ssql = "SELECT TI.id_torneo, TR.id_ronda, TR.nombre_ronda, TE.id_partido , TE.`user_id_1`, U1.username User1, TE.`user_id_2`, U2.username User2 ";
    $ssql .= "FROM `torneos_partidos` TE ";
    $ssql .= "    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`";
    $ssql .= "    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` ";
    $ssql .= "    INNER JOIN `usuarios` U1 ON TE.`user_id_1` = U1.user_id";
    $ssql .= "    INNER JOIN `usuarios` U2 ON TE.`user_id_1` = U1.user_id";
    //$ssql .= "    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` ";
    $ssql .= " WHERE TE.`id_torneo` = '$id_torneo' ";
    $ssql .= " ORDER BY TR.id_ronda";             
     
    echo $ssql;
     
    $rs = mysql_query($ssql) OR die('Error en consulta: ' . mysql_error());
     
    while ($fila = mysql_fetch_array($rs)){
     
        // verificando la query
        echo '<a href="' . $fila[user_id_1] . '">' . $fila[User1] . '</a>';
        echo ' vs ';
        echo '<a href="' . $fila[user_id_2] . '">' . $fila[User2] . '</a><br>';
    }
     
    ?>
 

La línea que comento estaba repetida y algún campo no era correcto y prefería ordenarlo por las rondas.

De todas formas no me ofrece el resultado correcto, ya que me hace un emparamiento con el resto de usuarios de la tabla de usuarios aunque no estén isncritos en el torneo y repite varias veces el mismo partido:



He probado modificando la consulta a ver si lograba conseguir otro resultado pero sin éxito:

Código PHP:

<?php
    $conexion = conecta_base_datos();
    $id_torneo = (isset($_GET["id"])) ? $_GET["id"] : exit();
     
    $ssql = "SELECT TI.id_torneo, TR.id_ronda, TR.nombre_ronda, TE.id_partido , TE.`user_id_1`, U1.username User1, TE.`user_id_2`, U2.username User2 ";
    $ssql .= "FROM `torneos_partidos` TE ";
    $ssql .= "    INNER JOIN `torneos_inscripcion` TI ON TI.`user_id` = TE.`user_id_1`";
    $ssql .= "    INNER JOIN `torneos_rondas` TR ON TR.`id_ronda` = TE.`id_ronda` ";
    $ssql .= "    INNER JOIN `usuarios` U1 ON TE.`user_id_1` = U1.user_id";
    $ssql .= "    INNER JOIN `usuarios` U2 ON TE.`user_id_1` = U1.user_id";
    $ssql .= "    INNER JOIN `torneos_partidos` TP ON TP.`id_partido` = '$id_torneo'";  
    $ssql .= " WHERE TE.`id_torneo` = '$id_torneo' ";
    $ssql .= " ORDER BY TR.id_ronda";    
     
    echo $ssql;
     
    $rs = mysql_query($ssql) OR die('Error en consulta: ' . mysql_error());
     
    while ($fila = mysql_fetch_array($rs)){
     
        // verificando la query
        echo '<a href="' . $fila[user_id_1] . '">' . $fila[User1] . '</a>';
        echo ' vs ';
        echo '<a href="' . $fila[user_id_2] . '">' . $fila[User2] . '</a><br>';
    }
     
    ?>
 

Debería mostrar dos partidos de la ronda 1 y un partido de la ronda 2.

[gnzsoloyo]

¿Podrías pasarnos los cREATE TABLE de las tablas y un volcado de datos de muestra para poder realizar unas pruebas?

Código MySQL:

SHOW CREATE TABLE `torneos_partidos`;
SHOW CREATE TABLE `torneos_inscripcion`;
SHOW CREATE TABLE `torneos_rondas`;
SHOW CREATE TABLE `usuarios`;
SHOW CREATE TABLE `torneos_partidos`;
 

Para el volcado de datos podrías usar el phpMyadmin.

Se hace muy difícil darte la solución correcta del problema de SQL sin tener elementos reales.
Además, sería conveniente seguir este tema en el foro de MySQL, ya que el problema es ajeno a PHP. Es más tema de bases de datos.

[dankko]

Okey.


Volcado

torneos_partidos:

Código SQL:

CREATE TABLE `torneos_partidos` (
 `id_partido` INT(7) NOT NULL AUTO_INCREMENT,
 `id_torneo` INT(7) NOT NULL,
 `id_ronda` INT(7) NOT NULL,
 `user_id_1` INT(7) NOT NULL,
 `user_id_2` INT(7) NOT NULL,
 `partido_jugado` VARCHAR(250) NOT NULL DEFAULT 'NO',
 `ganador` INT(4) NOT NULL DEFAULT '0',
 PRIMARY KEY (`id_partido`)
) ENGINE=InnoDB AUTO_INCREMENT=5 DEFAULT CHARSET=latin1
 

usuarios:

Código SQL:

CREATE TABLE `foro_users` (
 `user_id` mediumint(8) UNSIGNED NOT NULL AUTO_INCREMENT,
 `user_type` tinyint(2) NOT NULL DEFAULT '0',
 `group_id` mediumint(8) UNSIGNED NOT NULL DEFAULT '3',
 `user_permissions` mediumtext COLLATE utf8_bin NOT NULL,
 `user_perm_from` mediumint(8) UNSIGNED NOT NULL DEFAULT '0',
 `user_ip` VARCHAR(40) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_regdate` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `username` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `username_clean` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `name1` VARCHAR(35) COLLATE utf8_bin NOT NULL,
 `name2` VARCHAR(35) COLLATE utf8_bin NOT NULL,
 `team_id` INT(7) NOT NULL,
 `twitter_user` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_password` VARCHAR(40) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_passchg` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_pass_convert` tinyint(1) UNSIGNED NOT NULL DEFAULT '0',
 `user_email` VARCHAR(100) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_email_hash` BIGINT(20) NOT NULL DEFAULT '0',
 `user_birthday` VARCHAR(10) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_lastvisit` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_lastmark` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_lastpost_time` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_lastpage` VARCHAR(200) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_last_confirm_key` VARCHAR(10) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_last_search` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_warnings` tinyint(4) NOT NULL DEFAULT '0',
 `user_last_warning` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_login_attempts` tinyint(4) NOT NULL DEFAULT '0',
 `user_inactive_reason` tinyint(2) NOT NULL DEFAULT '0',
 `user_inactive_time` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_posts` mediumint(8) UNSIGNED NOT NULL DEFAULT '0',
 `user_lang` VARCHAR(30) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_timezone` DECIMAL(5,2) NOT NULL DEFAULT '0.00',
 `user_dst` tinyint(1) UNSIGNED NOT NULL DEFAULT '0',
 `user_dateformat` VARCHAR(30) COLLATE utf8_bin NOT NULL DEFAULT 'd M Y H:i',
 `user_style` mediumint(8) UNSIGNED NOT NULL DEFAULT '0',
 `user_rank` mediumint(8) UNSIGNED NOT NULL DEFAULT '0',
 `user_colour` VARCHAR(6) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_new_privmsg` INT(4) NOT NULL DEFAULT '0',
 `user_unread_privmsg` INT(4) NOT NULL DEFAULT '0',
 `user_last_privmsg` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_message_rules` tinyint(1) UNSIGNED NOT NULL DEFAULT '0',
 `user_full_folder` INT(11) NOT NULL DEFAULT '-3',
 `user_emailtime` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 `user_topic_show_days` SMALLINT(4) UNSIGNED NOT NULL DEFAULT '0',
 `user_topic_sortby_type` VARCHAR(1) COLLATE utf8_bin NOT NULL DEFAULT 't',
 `user_topic_sortby_dir` VARCHAR(1) COLLATE utf8_bin NOT NULL DEFAULT 'd',
 `user_post_show_days` SMALLINT(4) UNSIGNED NOT NULL DEFAULT '0',
 `user_post_sortby_type` VARCHAR(1) COLLATE utf8_bin NOT NULL DEFAULT 't',
 `user_post_sortby_dir` VARCHAR(1) COLLATE utf8_bin NOT NULL DEFAULT 'a',
 `user_notify` tinyint(1) UNSIGNED NOT NULL DEFAULT '0',
 `user_notify_pm` tinyint(1) UNSIGNED NOT NULL DEFAULT '1',
 `user_notify_type` tinyint(4) NOT NULL DEFAULT '0',
 `user_allow_pm` tinyint(1) UNSIGNED NOT NULL DEFAULT '1',
 `user_allow_viewonline` tinyint(1) UNSIGNED NOT NULL DEFAULT '1',
 `user_allow_viewemail` tinyint(1) UNSIGNED NOT NULL DEFAULT '1',
 `user_allow_massemail` tinyint(1) UNSIGNED NOT NULL DEFAULT '1',
 `user_options` INT(11) UNSIGNED NOT NULL DEFAULT '230271',
 `user_avatar` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_avatar_type` tinyint(2) NOT NULL DEFAULT '0',
 `user_avatar_width` SMALLINT(4) UNSIGNED NOT NULL DEFAULT '0',
 `user_avatar_height` SMALLINT(4) UNSIGNED NOT NULL DEFAULT '0',
 `user_sig` mediumtext COLLATE utf8_bin NOT NULL,
 `user_sig_bbcode_uid` VARCHAR(8) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_sig_bbcode_bitfield` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_from` VARCHAR(100) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_icq` VARCHAR(15) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_aim` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_yim` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_msnm` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_jabber` VARCHAR(255) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_website` VARCHAR(200) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_occ` text COLLATE utf8_bin NOT NULL,
 `user_interests` text COLLATE utf8_bin NOT NULL,
 `user_actkey` VARCHAR(32) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_newpasswd` VARCHAR(40) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_form_salt` VARCHAR(32) COLLATE utf8_bin NOT NULL DEFAULT '',
 `user_new` tinyint(1) UNSIGNED NOT NULL DEFAULT '1',
 `user_reminded` tinyint(4) NOT NULL DEFAULT '0',
 `user_reminded_time` INT(11) UNSIGNED NOT NULL DEFAULT '0',
 PRIMARY KEY (`user_id`),
 UNIQUE KEY `username_clean` (`username_clean`),
 KEY `user_birthday` (`user_birthday`),
 KEY `user_email_hash` (`user_email_hash`),
 KEY `user_type` (`user_type`)
) ENGINE=InnoDB AUTO_INCREMENT=54 DEFAULT CHARSET=utf8 COLLATE=utf8_bin
 

torneos_rondas:

Código SQL:

CREATE TABLE `torneos_rondas` (
 `id_ronda` INT(7) NOT NULL AUTO_INCREMENT,
 `nombre_ronda` VARCHAR(250) NOT NULL,
 PRIMARY KEY (`id_ronda`)
) ENGINE=InnoDB AUTO_INCREMENT=7 DEFAULT CHARSET=latin1
 

torneos_inscripcion:

Código SQL:

CREATE TABLE `torneos_inscripcion` (
 `id` INT(7) NOT NULL AUTO_INCREMENT,
 `user_id` INT(7) NOT NULL,
 `username` VARCHAR(250) NOT NULL,
 `nombrereal` VARCHAR(100) NOT NULL,
 `apellidos` VARCHAR(250) NOT NULL,
 `email` VARCHAR(250) NOT NULL,
 `estado` VARCHAR(250) NOT NULL DEFAULT 'SP',
 `fecha` DATE NOT NULL,
 `id_torneo` INT(7) NOT NULL,
 PRIMARY KEY (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=227 DEFAULT CHARSET=latin1
 

A la tabla de usuarios le tengo de nombre "foro_users"

[gnzsoloyo]

Por un lado, tienes inconsistencia de datos en una tabla, lo que genera un resultado erróneo:

Código MySQL:

mysql> SELECT *
    -> FROM torneos_partidos;
+------------+-----------+----------+-----------+-----------+----------------+---------+
| id_partido | id_torneo | id_ronda | user_id_1 | user_id_2 | partido_jugado | ganador |
+------------+-----------+----------+-----------+-----------+----------------+---------+
|          2 |         7 |        1 |        63 |         2 | NO             |       0 |
|          3 |         7 |        1 |        63 |         2 | NO             |       0 |
|          4 |         7 |        2 |         2 |        63 | NO             |       0 |
+------------+-----------+----------+-----------+-----------+----------------+---------+
3 rows in set (0.00 sec)
 

Como puedes ver, si bien el partido es distinto entre los dos primeros, aparecen los mismos datos de roda, torneo, y los usuarios en el mismo orden, y eso no puede ser. Si son partidos distintos del mismo torneo, entonces el esquema de partidos no está bien definido.

Esto se produce al cruzar las dos:

Código MySQL:

mysql> SELECT *
    -> FROM torneos_partidos TP
    ->     INNER JOIN torneos_rondas TR ON TP.id_ronda = TR.id_ronda;
+------------+-----------+----------+-----------+-----------+----------------+---------+----------+--------------------------+
| id_partido | id_torneo | id_ronda | user_id_1 | user_id_2 | partido_jugado | ganador | id_ronda | nombre_ronda             |
+------------+-----------+----------+-----------+-----------+----------------+---------+----------+--------------------------+
|          2 |         7 |        1 |        63 |         2 | NO             |       0 |        1 | Treintaidosavos de final |
|          3 |         7 |        1 |        63 |         2 | NO             |       0 |        1 | Treintaidosavos de final |
|          4 |         7 |        2 |         2 |        63 | NO             |       0 |        2 | Dieciseisavos de final   |
+------------+-----------+----------+-----------+-----------+----------------+---------+----------+--------------------------+
3 rows in set (0.00 sec)
 

Con esa inconsistencia, el resultado es:

Código MySQL:

mysql> SELECT
    ->     TP.id_torneo,
    ->     TR.id_ronda,
    ->     TR.nombre_ronda,
    -> --    TE.id_enfrentamiento -- ,
    ->     TP.user_id_1,
    ->     TI1.username User1,
    ->     TP.user_id_2,
    ->     TI2.username User2
    -> FROM torneos_partidos TP
    ->     INNER JOIN torneos_rondas TR ON TP.id_ronda = TR.id_ronda
    ->     INNER JOIN torneos_inscripcion TI1 ON TP.user_id_1 = TI1.user_id AND TP.id_torneo = TI1.id_torneo
    ->     INNER JOIN torneos_inscripcion TI2 ON TP.user_id_2 = TI2.user_id AND TP.id_torneo = TI2.id_torneo
    -> WHERE TP.id_torneo = 7 AND TP.id_ronda = 1;
+-----------+----------+--------------------------+-----------+---------+-----------+--------+
| id_torneo | id_ronda | nombre_ronda             | user_id_1 | User1   | user_id_2 | User2  |
+-----------+----------+--------------------------+-----------+---------+-----------+--------+
|         7 |        1 | Treintaidosavos de final |        63 | dankko2 |         2 | dankko |
|         7 |        1 | Treintaidosavos de final |        63 | dankko2 |         2 | dankko |
+-----------+----------+--------------------------+-----------+---------+-----------+--------+
2 rows in set (0.00 sec)
 

Para descartar la inconsistencia, la consulta propuesta debería quedar así:

Código MySQL:

mysql> SELECT DISTINCT
    ->     TP.id_torneo,
    ->     TR.id_ronda,
    ->     TR.nombre_ronda,
    -> --    TE.id_enfrentamiento -- ,
    ->     TP.user_id_1,
    ->     TI1.username User1,
    ->     TP.user_id_2,
    ->     TI2.username User2
    -> FROM torneos_partidos TP
    ->     INNER JOIN torneos_rondas TR ON TP.id_ronda = TR.id_ronda
    ->     INNER JOIN torneos_inscripcion TI1 ON TP.user_id_1 = TI1.user_id AND TP.id_torneo = TI1.id_torneo
    ->     INNER JOIN torneos_inscripcion TI2 ON TP.user_id_2 = TI2.user_id AND TP.id_torneo = TI2.id_torneo
    -> WHERE TP.id_torneo = 7 AND TP.id_ronda = 1;
+-----------+----------+--------------------------+-----------+---------+-----------+--------+
| id_torneo | id_ronda | nombre_ronda             | user_id_1 | User1   | user_id_2 | User2  |
+-----------+----------+--------------------------+-----------+---------+-----------+--------+
|         7 |        1 | Treintaidosavos de final |        63 | dankko2 |         2 | dankko |
+-----------+----------+--------------------------+-----------+---------+-----------+--------+
1 row in set (0.00 sec)
 

En realidad el problema es que tienes mal los datos de una tabla y eso genera errores totales, además que los últimso dos join requerían una doble condición.
Por otro lado, si ambos partidos son reales, y corresponden a enfrentamientos Ida/Vuela, no pueden tener los mismos usuarios en el mismo orden. user_id_1 y user_id_2 deben invertirse entre ambos partidos.

Más allá de esto, a mi entender el modelo de datos en general está mal planteado, porque no logra cumplir con los requisitos que necesita una base aplicada a un sistema de torneos por rondas, ni tampoco está bien diseñado el subesquema referido a las inscripciones y los datos de los torneos.
En este último caso hay datos que deberían componer relaciones y no aparecen como tales.
Peor esto último es tema off-topic de PHP. Es asunto de diseño de bases de datos y habría que tratarlo en otro foro.

[dankko]

Me ha funcionado a la perfección con la consulta que me has dejado, sólo he tenido que realizar algún cambio. Eres un genio gnzsoloyo, muchísimas gracias!!
El sistema es un poco manual y sencillo, pero creo que por ahora con esto nos valdrá.

Haré cambios en el sistema de inscripciones para evitar futuros problemas, muchas gracias!

[gnzsoloyo]

[Triby]

Iniciado por dankko

Eres un genio gnzsoloyo, muchísimas gracias!!

No estoy seguro si su paciencia es mayor que su conocimiento, pero, definitivamente, sí es un genio.