[MySql] Join/Sum falsche Ergebnisse

[pape]

Hi =)

Es gibt eine user-table (User) und 3 Tutorial-Tables (Text, Video und Other).

Jedes Tutorial hat die Attibute rating_positive and rating_negative
und ist über eine user_id eindeutig einem User zugeordnet.

Ich möchte die 10 besten User mit den meisten Tutorials haben und dazu noch die summe der Bewertungen ihrer Tutorials positiv und negativ.

Ich habe das hier versucht, da kommen zwar die richtigen User bei raus, aber die Werte für die summe der Tutorials und der Bewertungen ist viel zu hoch (ca 375 mal so hoch, wie sie sein sollten..)

SELECT
	u.id AS user_id,
	(COUNT(t.id) + COUNT(v.id) + COUNT(o.id)) AS tutorials_count, 
	(SUM(t.rating_positive) + SUM(v.rating_positive) + SUM(o.rating_positive)) AS pos,
	(SUM(t.rating_negative) + SUM(v.rating_negative) + SUM(o.rating_negative)) AS neg
FROM 
	user u LEFT JOIN trick t ON u.id = t.submitter_id
LEFT JOIN video v ON u.id = v.submitter_id
LEFT JOIN other o ON u.id = o.submitter_id
GROUP BY u.id
ORDER BY tutorials_count DESC
LIMIT 10

Was mache ich falsch?

Grüße
pape

[sonar]

Ganz spontan: Nimm mal statt LEFT JOIN ein normales JOIN …

[pape]

Hi,

was genau meinst du mit "normales Join"?
Wenn ich das Schlüsselwort "LEFT" weg lasse bekomme ich die gleichen Ergebnisse.

Das hier:

SELECT
	u.id AS user_id,
	(COUNT(t.id) + COUNT(v.id) + COUNT(o.id)) AS tutorials_count, 
	(SUM(t.rating_positive) + SUM(v.rating_positive) + SUM(o.rating_positive)) AS pos,
	(SUM(t.rating_negative) + SUM(v.rating_negative) + SUM(o.rating_negative)) AS neg

FROM user u, trick t, video v, other o
WHERE u.id = t.submitter_id
AND u.id = v.submitter_id
AND u.id = o.submitter_id
GROUP BY u.id
ORDER BY tutorials_count DESC
LIMIT 10

liefert auch die gleichen (falschen) Ergebnisse.

[thomas_E]

da ist ein Logikfehler in deinem Query. da gehören gar keine Joins rein.

versuch es mal so:

SELECT
  submitter_id AS user_id,
  COUNT(*) AS tutorials_count,
  SUM(rating_positive) AS pos,
  SUM(rating_negative) AS neg
FROM (
    SELECT submitter_id, rating_positive, rating_negative FROM trick
    UNION ALL
    SELECT submitter_id, rating_positive, rating_negative FROM video
    UNION ALL
    SELECT submitter_id, rating_positive, rating_negative FROM other
  ) AS temptable_all_tutorials
GROUP BY user_id
ORDER BY tutorials_count DESC
LIMIT 10;

wieso hast du die Daten eigentlich auf drei Tabellen verteilt? unterscheiden sie sich in den weiteren Spalten? oder lediglich wegen der bedeutung der Inhalte?

[pape]

Hi & Danke!

Der Query liefert die richtigen Ergebnisse und leuchtet mir auch ein. :-)

Wenn ich nun aber noch u.* (also alle User-Daten) will, dann brauche ich noch einen Join mit der user-table.

Danke!

wieso hast du die Daten eigentlich auf drei Tabellen verteilt? unterscheiden sie sich in den weiteren Spalten? oder lediglich wegen der bedeutung der Inhalte?

ganz genau. :-)
Zudem liegt noch ein ORM dazwischen (Doctrine).

Beste Grüße
pape

[thomas_E]

Wenn ich nun aber noch u.* (also alle User-Daten) will, dann brauche ich noch einen Join mit der user-table.

ja.
Nur für die Daten aus deinem bisherigen Query war es halt nicht notwendig.