Alle Postings & Kommentare in einem Query ausgeben

Teron Gerofied

Hallo Leute,

habe hier noch eine Frage zu SQL Querys wo ich leider nichtmal einen Lösungsansatz hätte ...

Ich habe folgendes Problem:

Ich habe eine Tabelle mit Postings und eine mit Antworten (Kommentaren)

Ich möchte diese wie folgt abfragen:

-Posting 1
-- Kommentar 1.1
-- Kommentar 1.2
-Posting 2
-Posting 3
-- Kommentar 3.1
-Posting 4
-- Kommentar 4.1
-- Kommentar 4.2

und das ganze möchte ich in einem Query haben ...

normalerweise sieht bei mir eine Abfrage so aus:

PHP

$result = mysql_query("SELECT * FROM ....");
while($row = mysql_fetch_array($result))
{
  // irgendwas machen
}

Meine Frage ist jetzt wie so eine Abfrage und eine weiterverarbeitung in php aussehen müsste

lg
mathias

jojo87

schau dir mal joins an

Code

Select dies,das,jenes
From tabelle1
Join tabelle2 On tabelle1.id = tabelle2.fremdschluessel

Teron Gerofied

Joins sind mir schon bekannt, aber wie muss die verarbeitung in php dazu aussehen ?

dazu bräuchte ich ja theoretisch zwei schleifen ineinander und ich hab echt keinen plan wie das trotz der joins aussehen kann

weil es können bei einem posting ja zB 10 kommentare sein und beim nächsten nur 2 ... ist ungefähr klar wie ich das mein ?

synaptic

du kannst dir alternativ den join auch in nem view speichern und den dann abfragen und später kannste die daten auslesen, wie aus ner normalen tabelle.

jojo87

bastel dir doch mal einen funktionierenden query.
Dann gibst du das ergebnis mal per var_dump aus und schaust, was dir dazu einfällt.
Sollte dir nix einfallen => Ausgabe posten und wir überlegen mal zusammen

synaptic: Einen weiteren Query sollte ersich sparen.
Zwar ist das für den konreten Fall erstemal wurscht, aber allgemein gilt: sowenig querys wie möglich.
Denn die sind der Performance Flaschenhals einer jeden Anwendung

Teron Gerofied

Ja das will ich eben vermeiden, habe von hausaus schon fast 5-6 querys / seite, deswegen will ichs niedrig halten.

Zu der Sache mit dem Query: das ist das Problem, ich habe nicht die geringste Idee zur Lösung des Problems ... deswegen bringt mir var_dump auch nix ... weil ein einfacher select query mit nem join is ja kein problem, aber was mach ich weiter damit ?

synaptic

ging ja nur darum ihm das ganze etwas übersichtlicher zu gestalten... außerdem kann doch mysql nicht direkt ne query auf nen result abfeuern (soweit ich mich entsinne) und ich glaub auch derzeit nicht, dass der overhead von nem view so immens ist, dass es da so sehr ins gewicht fällt

teron du kannst auch das ergebnis vom join in ein assoziatives array stopfen und dad so oft durchlaufen wie du magst..
mach dir mal grundgedanken, schau dir das ergebnis an und stell konkrete fragen, wir sind hier ja net um deine anwendung zu entwickeln!

Teron Gerofied

na gut ich hätte da schon eine idee ...

ich mach einfach ne normale schleife von den postings und stopf alle kommentare die je geschrieben wurden in ein eigenes array und bastel das dann im php code zusammen ... das ist zwar nur ein select mehr ... aber ich hab dabei irgendwie angst, dass das soviel meiner performance frisst ... was würdet ihr dazu sagen ?

jojo87

also was denn nun, kriegst dus hin dir den query (select mit nem Join) zu basteln oder nicht?
Wenn ja: den mal ausführen und das Ergebnis per var_dump() ausgeben.
Wenn nein: Warum sagst dus dann?
Ich kann dir grad schwer folgen...

Teron Gerofied

So ich hab jetzt mal eben so ein var_dump erstellt ...

So solls aussehen:

- Posting 1
--- Kommentar 1
--- Kommentar 2
- Posting 2
--- Kommentar 1
--- Kommentar 2
--- Kommentar 3
- Posting 3
--- Kommentar 1

So siehts aus, wenn der Query so aussieht:

SQL

SELECT * FROM posting JOIN comment ON posting_id = com_posting_id

Code

array(6) {
  [0]=>
  array(14) {
    [0]=>
    string(1) "1"
    ['posting_id']=>
    string(1) "1"
    [1]=>
    string(1) "1"
    ['posting_user_id']=>
    string(1) "1"
    [2]=>
    string(9) "Posting 1"
    ['posting_text']=>
    string(9) "Posting 1"
    [3]=>
    string(1) "1"
    ['com_id']=>
    string(1) "1"
    [4]=>
    string(1) "1"
    ['com_posting_id']=>
    string(1) "1"
    [5]=>
    string(1) "1"
    ['com_user_id']=>
    string(1) "1"
    [6]=>
    string(13) "Kommentar 1/1"
    ['com_text']=>
    string(13) "Kommentar 1/1"
  }
  [1]=>
  array(14) {
    [0]=>
    string(1) "1"
    ['posting_id']=>
    string(1) "1"
    [1]=>
    string(1) "1"
    ['posting_user_id']=>
    string(1) "1"
    [2]=>
    string(9) "Posting 1"
    ['posting_text']=>
    string(9) "Posting 1"
    [3]=>
    string(1) "2"
    ['com_id']=>
    string(1) "2"
    [4]=>
    string(1) "1"
    ['com_posting_id']=>
    string(1) "1"
    [5]=>
    string(1) "1"
    ['com_user_id']=>
    string(1) "1"
    [6]=>
    string(13) "Kommentar 1/2"
    ['com_text']=>
    string(13) "Kommentar 1/2"
  }
  [2]=>
  array(14) {
    [0]=>
    string(1) "2"
    ['posting_id']=>
    string(1) "2"
    [1]=>
    string(1) "1"
    ['posting_user_id']=>
    string(1) "1"
    [2]=>
    string(9) "Posting 2"
    ['posting_text']=>
    string(9) "Posting 2"
    [3]=>
    string(1) "3"
    ['com_id']=>
    string(1) "3"
    [4]=>
    string(1) "2"
    ['com_posting_id']=>
    string(1) "2"
    [5]=>
    string(1) "1"
    ['com_user_id']=>
    string(1) "1"
    [6]=>
    string(13) "Kommentar 2/1"
    ['com_text']=>
    string(13) "Kommentar 2/1"
  }
  [3]=>
  array(14) {
    [0]=>
    string(1) "2"
    ['posting_id']=>
    string(1) "2"
    [1]=>
    string(1) "1"
    ['posting_user_id']=>
    string(1) "1"
    [2]=>
    string(9) "Posting 2"
    ['posting_text']=>
    string(9) "Posting 2"
    [3]=>
    string(1) "4"
    ['com_id']=>
    string(1) "4"
    [4]=>
    string(1) "2"
    ['com_posting_id']=>
    string(1) "2"
    [5]=>
    string(1) "1"
    ['com_user_id']=>
    string(1) "1"
    [6]=>
    string(13) "Kommentar 2/2"
    ['com_text']=>
    string(13) "Kommentar 2/2"
  }
  [4]=>
  array(14) {
    [0]=>
    string(1) "2"
    ['posting_id']=>
    string(1) "2"
    [1]=>
    string(1) "1"
    ['posting_user_id']=>
    string(1) "1"
    [2]=>
    string(9) "Posting 2"
    ['posting_text']=>
    string(9) "Posting 2"
    [3]=>
    string(1) "5"
    ['com_id']=>
    string(1) "5"
    [4]=>
    string(1) "2"
    ['com_posting_id']=>
    string(1) "2"
    [5]=>
    string(1) "1"
    ['com_user_id']=>
    string(1) "1"
    [6]=>
    string(13) "Kommentar 2/3"
    ['com_text']=>
    string(13) "Kommentar 2/3"
  }
  [5]=>
  array(14) {
    [0]=>
    string(1) "3"
    ['posting_id']=>
    string(1) "3"
    [1]=>
    string(1) "1"
    ['posting_user_id']=>
    string(1) "1"
    [2]=>
    string(9) "Posting 3"
    ['posting_text']=>
    string(9) "Posting 3"
    [3]=>
    string(1) "6"
    ['com_id']=>
    string(1) "6"
    [4]=>
    string(1) "3"
    ['com_posting_id']=>
    string(1) "3"
    [5]=>
    string(1) "1"
    ['com_user_id']=>
    string(1) "1"
    [6]=>
    string(13) "Kommentar 3/1"
    ['com_text']=>
    string(13) "Kommentar 3/1"
  }
}

Alles anzeigen

jojo87

ach komm, das kannst du

Code

posting-array erstellen
foreach (dbergebnisse as ergebnis)
[INDENT]if (postingId in postingArray )
[INDENT]neuen kommentar dem posting in postingarray zufügen
[/INDENT]else
[INDENT]posting dem posting-array zufügen
kommentar dem posting in posting-array zufügen
[/INDENT][/INDENT]Ende

SinnlosS

Ich würde das ganze über zwei Querys lösen (bzw. bei Nutzung von mysqli oo (o.ä.) noch komfortabler dank prepared statements, aber du nutzt ja noch mysql_*, richtig?).
Erstes Query liest du Postings aus und füllt sie in einen Array.
Zweites Query liest dann die Antworten aus und ergänzt das erstellte Posting-Array um die Antworten.

Wenn man das in einem Query per Join löst bläht man meiner Meinung nach nur das Ergebnis unnötig auf. In der Ergebnismenge sind ja zwangsläufig immer entweder die Spalten aus der Antworttabelle leer, oder die Spalten aus der Postingtabelle sind doppelt-, drei, ... zig-facher Content. Da nehme ich lieber ein Query mehr in Kauf und halbiere dafür den Speicherbedarf für meine Ergebnismenge(n).

Teron Gerofied

Zitat von SinnlosS

Ich würde das ganze über zwei Querys lösen (bzw. bei Nutzung von mysqli oo (o.ä.) noch komfortabler dank prepared statements, aber du nutzt ja noch mysql_*, richtig?).
Erstes Query liest du Postings aus und füllt sie in einen Array.
Zweites Query liest dann die Antworten aus und ergänzt das erstellte Posting-Array um die Antworten.

Wenn man das in einem Query per Join löst bläht man meiner Meinung nach nur das Ergebnis unnötig auf. In der Ergebnismenge sind ja zwangsläufig immer entweder die Spalten aus der Antworttabelle leer, oder die Spalten aus der Postingtabelle sind doppelt-, drei, ... zig-facher Content. Da nehme ich lieber ein Query mehr in Kauf und halbiere dafür den Speicherbedarf für meine Ergebnismenge(n).

So hatte ich mir das auch gedacht

jojo87

hach, offtopic ist was schönes
Meiner Erfahrung nach ist heutzutage der Speicherverbrauch deutlich weniger wichtig als die Geschwindigkeit der Anwendung.
Speicher kostet einfach nichts mehr.
Aber für den Threadersteller sollte das letztendlich sehr egal sein. Ihn wird weder der Speicherbedarf noch der minimale Geschwindigkeitsunterschied bei diesem Problem interessieren.

Teron Gerofied

Zitat von jojo87

hach, offtopic ist was schönes
Meiner Erfahrung nach ist heutzutage der Speicherverbrauch deutlich weniger wichtig als die Geschwindigkeit der Anwendung.
Speicher kostet einfach nichts mehr.
Aber für den Threadersteller sollte das letztendlich sehr egal sein. Ihn wird weder der Speicherbedarf noch der minimale Geschwindigkeitsunterschied bei diesem Problem interessieren.

Hab mir jetzt mal deinen Lösungsvorschlag angeschaut, stehe aber grade ein bisschen an ...

sollte das so aussehen:

PHP

$result = mysql_query( "SELECT * FROM posting LEFT JOIN comment ON posting_id = com_posting_id" );
echo '<dl>';
$res= array();
while( $row = mysql_fetch_assoc( $result ) )
{
	$res[] = $row;
}
foreach ($res as $r)
{
	if (in_array($r['posting_id'], $r) )
		echo '<dd>'.$r['com_text'].'</dd>';
	else {
		echo '<dt>'.$r['posting_text'].'</dt>';
		echo '<dd>'.$r['com_text'].'</dd>';
	}
}
echo "</dl>";

Alles anzeigen

jojo87

naja, fast:

PHP

$res = mysql_query('SELECT...');
$postings = array();
while ($row = mysql_fetch_object($res)) {
  if (isset($postings[$row->posting_id])) {
    $postings[$row->posting_id]['comments'][] = array(
      'id'     => $row->comment_id,
      'text'  => $row->comment_text
    );
  } else {
      $postings[$row->posting_id] = array(
        'posting_id' => $row->posting_id,
        'comments' => array()
      );
}
// Ausgabe

Alles anzeigen

Teron Gerofied

ok, wenn ich das probier und $postings mit var_dump ausgebe kommt folgendes:

Code

array(4) {
  [1]=>
  array(2) {
    ['posting_id']=>
    string(1) "1"
    ['comments']=>
    array(1) {
      [0]=>
      array(2) {
        ['id']=>
        NULL
        ['text']=>
        NULL
      }
    }
  }
  [2]=>
  array(2) {
    ['posting_id']=>
    string(1) "2"
    ['comments']=>
    array(2) {
      [0]=>
      array(2) {
        ['id']=>
        NULL
        ['text']=>
        NULL
      }
      [1]=>
      array(2) {
        ['id']=>
        NULL
        ['text']=>
        NULL
      }
    }
  }
  [3]=>
  array(2) {
    ['posting_id']=>
    string(1) "3"
    ['comments']=>
    array(0) {
    }
  }
  [4]=>
  array(2) {
    ['posting_id']=>
    string(1) "4"
    ['comments']=>
    array(0) {
    }
  }
}

Alles anzeigen

jojo87

wenn du jetzt noch die attribute von $row richtig benennst (ich habe sie ja nur geraten ;)) dann sollte das so sein, wie dus brauchst

Teron Gerofied

ups, peinlich, das hab ich jetzt nicht mehr geschaut ... ok meine seite raubt mir einfach derzeit dezent den schlaf und nach 2 tagen dauercodierung war ich auch schon sehr übermüdet

Teron Gerofied

Ok, habe jetzt die Lösung des Problems:

Der Fehler war, dass beim immer das erste Kommentar ignoriert wurde weil das ja schon im ersten Posting steckt ... folgendes behebt den Fehler:

PHP

$postings = array();
while ($row = mysql_fetch_object($res)) {
	if (isset($postings[$row->posting_id])) {
		$postings[$row->posting_id]['comments'][] = array(
		  'id' => $row->com_id,
		  'text' => $row->com_text
		);
	} else {
      $postings[$row->posting_id] = array(
        'posting_id' => $row->posting_id,
		'posting_text' => $row->posting_text,
        'comments' => array()
      );
		if($row->com_id)
		$postings[$row->posting_id]['comments'][] = array(
		  'id' => $row->com_id,
		  'text' => $row->com_text
		);
	}
}

Alles anzeigen

EDIT: Jetzt wäre nur mehr eine Frage: Kann man mit SQL auch die Abfrage-Ergebnisse von einem JOIN limitieren, dass zB immer nur 2 Kommentare angezeigt werden .... ohne einer WHERE ... IN Anweisung ... unterstützt mein server nicht ...

Möchte das nicht in PHP machen, weil ich sonst unnötig viele Daten aus der DB ziehe

Alle Postings & Kommentare in einem Query ausgeben

Tags

Benutzer online in diesem Thema