Abfrage optimieren

[bazubi]

Hallo,

ich würde gerne folgenden Quellcode optimieren:

PHP-Code:


 //getting all systems and assigning them to a smarty value
$query = @mysql_query('SELECT o.id AS id, o.ip AS ip, o.tme AS tme, s.name AS name FROM onlinestate AS o, systems AS s WHERE o.id = s.id ORDER BY o.tme DESC');

$data = array();
$ids  = array();
while ($arr = @mysql_fetch_assoc($query)) {
            if (! in_array($arr['id'], $ids)) {
                $data[] = $arr;
                $ids[]  = $arr['id'];
            }
        } 


Der Quellcode soll folgendes tun: Aus der Tabelle "onlinestate" sollen alle Datensätze ausgelesen werden. Innerhalb dieser Tabelle wird jeder Zeitpunkt eines Seitenaufrufs gespeichert. Der Name der Seite soll aus der Tabelle "systems" gelesen werden. Allerdings will ich jetzt nicht alle Datensätze aus "onlinestate" sondern nur den letzten Zeitpunkt pro Seite. Das macht im Moment die if (! in_array(...)) Abfrage. Kann man diese if Verzweigung noch irgendwie umgehen, in dem man den SQL Befehl verändert?

[c4]

Ich habe auf meiner Seite auch eine kleine Seitenaufrufstatistik (sic!):
--> http://dbCF.de/hits-per-page/

Würde ich hier die letzten Zugriffe wissen wollen, dann würde das so aussehen:

SELECT dbcf_hits_per_page.id, title, MAX(last_time)
FROM dbcf_hits_per_page INNER JOIN dbcf_hits_per_page_ips ON dbcf_hits_per_page.id=dbcf_hits_per_page_ips.id
GROUP BY dbcf_hits_per_page.id

Die Ausgabe sieht dann wie folgt aus:

id  title                                       MAX(last_time)
1   Hintergrundbilder                           1101280576
3   Start                                       1101279977
12  14895 EDV Abkürzungen und ihre Wortlaute    1101280493
38  ColorFader mit PHP                          1101280996

Da bei mir alte Einträge gelöscht werden, variiert diese Anzeige ständig, klar.

Deine Abfrage würde also ungefähr so aussehen:

SELECT o.id, o.ip, o.tme, s.name
FROM onlinestate INNER JOIN systems ON onlinestate.id=systems.id
ORDER BY o.tme DESC

Das mit dem AS bla kannst Du Dir sparen, solange Du nicht o.id und s.id rausholst.

Alles klarofix?

[bazubi]

Danke für die Antwort.

Einen Inner Join hatte ich ja praktisch schon mit dem where s.id = o.id. Das funktioniert auch soweit. Deine Abfrage (die zweite) macht im Ergebniss die gleiche wie meine. Ich bekomme dann ungefähr die folgende Ergebnisstabelle:

id    ip       tme       name
------------------------------------
1     xyz      1008000   seite1
1     xyz      1007000   seite1
2     xyz      1006000   seite2
1     xyz      1005000   seite1
3     xyz      1004000   seite3
2     xyz      1003000   seite2

D.h. ich bekomme jeden Hit für jede Seite. Ich will aber pro Seite nur den letzten Hit. Es soll also so ausehen:

id    ip       tme       name
------------------------------------
1     xyz      1008000   seite1
2     xyz      1006000   seite2
3     xyz      1004000   seite3

(Die IP ist im Prinzip unwichtig es könnten also auch verschiedene IPs sein)

Ich habe zwischenzeitlich auch nochmal versucht ein MAX() mit einzubauen, erhalte aber bei der Abfrage:

 SELECT o.id, o.ip, MAX( o.tme ) AS tme, s.name
FROM onlinestate AS o, systems AS s
WHERE o.id = s.id

nur die Fehlermeldung

#1140 - Mixing of GROUP columns (MIN(),MAX(),COUNT()...) with no GROUP columns is illegal if there is no GROUP BY clause

[c4]

Hatte ganz vergessen MAX() in Deine Abfrage einzubinden.

Versuchen wir es mal damit:

SELECT o.id, MAX(o.tme) tme, s.name
FROM onlinestate o INNER JOIN systems s ON o.id=s.id
GROUP BY s.id
ORDER BY s.name

Wenn auch das nicht klappt, dann solltest Du Dich lieber selber damit beschäftigen (Doku MySQL). Für mich ist das ohne Daten schlecht testbar.

[bazubi]

Danke, jetzt klappt es bestens. Mit GROUP BY hatte ich es schonmal versucht, allerdings hatte ich da keinen INNER JOIN sondern nur ein WHERE. Das in Verbindung mit einem ORDER BY hat sich gebissen.

Ich werde es mir dick hinter die Ohren schreiben. Bei 2 Tabellen nur noch JOINs und keine WHEREs mehr! ;)