Duda con Consulta SQL

[ezekkiel]

Hola!,
Tengo algo que me esta rompiendo la cabeza..
Tengo una tabla con pagos de usuarios:

id_user_paid id_user user_paid paid_date
1 9 0.00 2011-12-20
2 8 1.00 2011-12-20

y la siguiente consulta:

SELECT users_data.num_doc, CONCAT(users_data.last_name, ', ', users_data.first_name) AS nombre,
CONCAT('$', ROUND( (training.training_cost - (training.training_cost * payment_plans.payment_plan_discount / 100)),2)) AS tot_a_pagar,
CONCAT('$', ROUND( SUM(users_payment.user_paid), 2)) AS tot_pagado,
CONCAT('$', ROUND( (training.training_cost - (training.training_cost * payment_plans.payment_plan_discount / 100)) - (SUM(users_payment.user_paid)),2)) AS saldo,
MAX(users_payment.paid_date) AS ultimo_pago
FROM users_data
JOIN users_payment ON users_data.id_user = users_payment.id_user
JOIN users_payment_plans ON users_data.id_user = users_payment_plans.id_user
JOIN payment_plans ON users_payment_plans.payment_plan_id = users_payment_plans.payment_plan_id
JOIN users_trainings ON users_data.id_user = users_trainings.id_user
JOIN training ON users_trainings.id_training = training.training_id
GROUP BY users_data.num_doc

el campo tot_pagado tendria que ser 1,00 pero me sale 4,00..
Porque puede ser?

[quimfv]

Pon ejemplos de los datos de todas las tablas... sin ver nada solo se puede decir que alguna relacion te esta generando que ese valor se lea 4 veces....

GROUP BY users_data.num_doc, CONCAT(users_data.last_name, ', ', users_data.first_name)

para ser estrictos te falta esto en el group by.... ¿puede ser que tengas users con el mismo num_doc y nombres distintos?

[ezekkiel]

Hola!, gracias por la respuesta.
para prueba solo tengo 2 usuarios, cada uno con un dni y un nombre.

como decis probe con:
GROUP BY users_data.id_user, CONCAT(users_data.last_name, ', ', users_data.first_name)
y no da el numero que deberia dar..

Tabla users_data:
id_user | first_name | last_name | birth_place | birth_date | id_type_doc | num_doc
8 | Ezequiel | Rodriguez | NULL | 1983-11-16 | 1 | 33333333
9 | Ignacio | Perez | NULL | 1978-02-14 | 1 | 22222222

Tabla users_payment:
id_user_paid | id_user | user_paid | paid_date
1 | 9 | 0.00 | 2011-12-20
2 | 8 | 1.00 | 2011-12-20
2 | 8 | 15.00 | 2011-12-20

Tabla users_payment_plans:
id_user_payment_plan | id_user | payment_plan_id
2 | 8 | 1
3 | 9 | 1

Tabla payment_plans:
payment_plan_id | payment_plan_cod | payment_plan_name | payment_plan_discount
1 | A | Débito directo de caja de ahorro o cuenta corrient... | 10

Tabla users_trainings:
id_user_training | id_user | id_training | pre_inscription_date | activate_date
1 | 8 | 1 | 2011-12-18 | 2011-12-18
2 | 9 | 1 | 2011-12-18 | 2011-12-18

Tabla training:
training_id | created_date | training_name | duration | enrollment | training_cost
1 | 2011-12-18 | Desarrollo Y Constitución Subjetiva En Tiempo De ... | 9 | 400 | 2520

El resultado de la consulta:
33333333 | Rodriguez, Ezequiel | $2268.00 | $65.00 | $2203.00 | 2011-12-20
22222222 | Perez, Ignacio | $2268.00 | $0.00 | $2268.00 | 2011-12-20

Espero haberme explicado.. el total pagado deberia ser $16.00 y no $65.00 como dice, no se que estoy haciendo mal.

[gnzsoloyo]

Es un problema que se produce cuando se usan funciones agrupadas sobre diferentes columnas provenientes de diferentes tablas en un JOIN multitabla. En realidad lo que sucede es que realiza los JOIN antes de hacer la operación de la función, y como la cardinalidad es superior a 1, genera un producto cartesiano no previsto, antes de hacer la el agrupamiento.
La única solución es hacer que las diferentes tablas de datos se consoliden antes de usarlas en un JOIN, de modo que cuando este se haga la relación tenga cardinalidad 1 en todos los casos.
Hace unos días alguien preguntó por un caso similar, sólo que era con tres tablas, nada más.
La idea sería mas o menos:

Código MySQL:

SELECT 
    UD.num_doc, 
    CONCAT(UD.last_name, ', ', UD.first_name) nombre,
    CONCAT('$', ROUND((T.training_cost - (T.training_cost * PP.payment_plan_discount/100)),2)) tot_a_pagar,
    CONCAT('$', ROUND(UP.user_paid, 2)) tot_pagado,
    CONCAT('$', ROUND((T.training_cost - (T.training_cost * PP.payment_plan_discount / 100)) - (UP.user_paid),2)) saldo,
    UP.paid_date ultimo_pago
FROM 
    users_data UD
    INNER JOIN 
        (SELECT MAX(paid_date), SUM(user_paid) user_paid, id_user, paid_date 
        FROM users_payment 
        GROUP BY id_user) UP 
        ON UD.id_user = UP.id_user
    INNER JOIN users_payment_plans UPP 
        ON UD.id_user = UPP.id_user
    INNER JOIN payment_plans PP 
        ON UPP.payment_plan_id = PP.payment_plan_id
    INNER JOIN users_trainings UT 
        ON UD.id_user = UT.id_user
    INNER JOIN training T 
        ON UT.id_training = T.training_id
GROUP BY UD.num_doc
 

Por cierto: Trata de usar ALIAS para simplificar la sintaxis, y procura no usar JOIN sólo. Define mejor la operación para que el parser utilice el algoritmo más eficiente.

[ezekkiel]

Gracias quimfv y gnzsoloyo!, creo haber entendido lo que me decis gnzsoloyo
ya me acostumbraré a los alias.. jaja.

Muchas gracias a los 2!
Slds.

Eze.