[ex³]

Ich habe folgenden Query:

        $query = 'SELECT ' .
        'n.id, n.seo, n.headline, n.post, n.postdate, n.author,' .
        'COUNT(c.id) comments ' .
        'FROM newspost n ' .
        'LEFT JOIN newscomment c ON n.id = c.newsid ' .
        'WHERE n.postdate < ? AND n.textstatus = ? ' .
        'GROUP BY n.id ' .
        'ORDER BY n.postdate DESC';

Dadurch bekomme ich auf einen Schlag alle Newsposts und die dazugehörige Kommentaranzahl.

Jetzt werde ich aber allmählich von Kommentarspam zugeschissen und möchte Kommentare per E-Mail Benachrichtigung freischalten. Dazu gibt es in der Tabelle newspost das Feld textstatus was 0 oder 1 sein kann.

Mit dem obigen Query bekomme ich alle dazugehörigen Kommentare, allerdings werden dadurch auch Kommentare dazugerechnet die auf Status 0 sind. Ich möchte mit dem COUNT aber nur die Anzahl der aktiven Kommentare zusammengezählt haben. Das WHERE auszubauen hilft nichts.

Geht das nur mit zwei Queries?

[Artemis]

Erstmal kann man das strukturiert viel besser angucken:

SELECT
  n.id,
  n.seo,
  n.headline,
  n.post,
  n.postdate,
  n.author,
  COUNT(c.id) comments
FROM
  newspost n
LEFT JOIN
    newscomment c
  ON
    n.id = c.newsid
WHERE
    n.postdate < ?
  AND
    n.textstatus = ?
GROUP BY
  n.id
ORDER BY
  n.postdate
DESC
 

Ich verstehe dich gerade nicht. Soll jetzt bei den Kommentaren auf Status 0 überprüft werden, oder bei den Posts?

[ex³]

Mit dem COUNT sollen nun Kommentare berücksichtigt werden, die den c.textstatus = 1 haben. Der n.textstatus ist für die Newspost

[Artemis]

Kannste mal nen kleinen MySQL-Dump geben, hab Lust ein bisschen zu probieren.

Achja, welche MySQL-Version?

[Basti]

Zitat von ex³

Mit dem COUNT sollen nun Kommentare berücksichtigt werden, die den c.textstatus = 1 haben. Der n.textstatus ist für die Newspost

Dann würd ich es mit SUM(c.textstatus) probieren.

Basti

[Artemis]

LOL, auf die einfachsten Sachen kommt man nicht.

Dann hat sich das mit dem Dump wohl erledigt.

[ex³]

Dann würd ich es mit SUM(c.textstatus) probieren.

Ja das ist wirklich genial.

Allerdings hab ichs jetzt doch über 2 Queries gemacht.

Zitat von ex³

Ja das ist wirklich genial.

nicht ganz, denn damit verändert sich die semantik des 'textstatus'. stati haben die dumme eigenschaft, sich mit der zeit zu erweitern, daher fliegt dir diese abfrage dann schön um die ohren, wenn es auf einmal einen textstatus=-1, textstatus=2 gibt.

um gar nicht erst in versuchung zu kommen, solche status-spalten für etwas anderes als ihren ursprünglichen zweck zu verwenden, bietet sich der datentyp ENUM an - macht die datenbank-daten auch besser lesbar und vermeidet magic numbers.

Allerdings hab ichs jetzt doch über 2 Queries gemacht.

das ist noch weniger genial, denn du verlierst viel performance. das sind, vorausgesetzt du enthältst uns keine informationen vor, auch keine 2, sondern n zusätzliche queries, wobei n die anzahl der newsposts ist. dadurch erhöht sich der aufwand des algorithmus von O(n^0) auf O(n^1), und das ist schlecht.

COUNT darf einen (einschränkenden) ausdruck behinhalten, daher lautet die richtige lösung:

SELECT COUNT(c.textstatus=1 [AND foo=bar [OR [...]]] ) ...

grüße
axo