INNER JOIN php Frage an die Freaks ;)

[Medium]

Hallo,

habt mir ja schon mal geholfen

ich habe Artikel bzw. Produkte -> Tabelle tx_item_produkt
ich habe Kategorien -> Tabelle tx_item_Cat
ind eine Hilfstabelle -> tx_item_produkt_category_mm

über die Variable $auswahl möchte ich später stuern welche Kategorien angezeigt werden sollen.

in tx_item_produkt_category_mm wird jedes Artikel-Kategorie-Verhältnis in eine Zeile geschrieben



mein Code sieht so aus:

Code: Alles auswählen

			$sql = "SELECT *
            FROM tx_item_produkt
            INNER JOIN tx_item_produkt_category_mm ac ON ( ac.uid_local = tx_item_produkt.uid )
			INNER JOIN tx_item_Cat c ON ( c.uid = ac.uid_foreign )";

            if ($auswahl == '') {
                        }else {
                        $sql .="WHERE ac.uid_foreign IN ($auswahl)";
            }

			$abfrage = mysqli_query($verbindung, $sql);

			while($articel = mysqli_fetch_assoc($abfrage)){

                        echo "<tr>";
						echo "<td>$articel&#91;uid&#93; </td>";
						echo "<td>$articel&#91;title&#93; </td>";
						echo "<td>$articel&#91;text&#93;</td>";
                        echo "<td>$articel&#91;category&#93;</td>";
						echo "</tr>";

			&#125;
			echo "</table>";
			mysqli_free_result&#40;&#41;;

leider gibt er mir wieder Artikel die in mehreren Kategorien sind sopplet usw. aus

[Outman]

Was steht denn bei dein Skript in der Variable: $auswahl? Sind dies Zahlen mit Komma getrennt? Gegebenenfalls könntest du mit distinct bzw. GROUP BY was machen. Ich finde auf den ersten Blick deine Bezeichnungen von den Spalten sehr unpassend.

Verwende für die Zukunft nicht uid für id. Denn uid kann man immer dazu interpretieren, dass dies eine User-ID ist. Desweiteren würde ich an deiner Stelle auch Tabellennamen nie abkürzen wie zum Beispiel "tx_item_produkt_category_mm ac" dadurch verliert man einfach den Überblick. Du kannst dies im Nachhinein sobald du fertig bist, abändern.

Schreibe lieber:

... (`tx_item_produkt_category_mm`.`uid_local` = `tx_item_produkt`.`uid` ) ...


und lasse dir mal $sql bevor du die Abfrage von der Mysql Datenbank machst ausgeben. Bei * solltest du noch angeben, welche Tabelle du hier meinst. Beispiel:

`tx_item_produkt_category_mm`.*

[nerd]

SELECT DISTINCT ... !

[Outman]

SELECT DISTINCT ... !

hatte ich bereits geschrieben

[nerd]

stimmt. liegt daran das ich grundsaetzlich nur die erste zeile pro beitrag lese. spart ne menge zeit

[mgutt]

1.) tx_item_produkt_category_mm hast Du weil jeder Artikel auch in mehreren Kategorien stehen kann, korrekt?

2.) Warum INNER JOIN? Gibt es Artikel ohne Kategorien? Wenn nein wäre ein simpler JOIN die bessere Wahl bzw. noch besser die implizite Schreibweise:

Code: Alles auswählen

// alle artikel aus allen kategorien
$sql = "SELECT *
				FROM (tx_item_produkt p, tx_item_produkt_category_mm pc, tx_item_Cat c)
				WHERE pc.uid_local = p.uid
				AND c.uid = pc.uid_foreign";
// nur aus bestimmten kategorien
if ($auswahl) { // wenn $auswahl == '' dann ist das gleich false ;)
	$sql .= ' AND pc.uid_foreign IN(' . mysql_real_escape_string($auswahl) . ')';
}

3.) Ich finde Deine Spalten und Tabellennamen reichlich verwirrend. Was soll der Quatsch mit der uid? Nenn das doch einfach product_id und cat_id und nicht local und foreign.