Als erstes wird die Datenbank nach 10 zutreffenden ids befragt. Dieses Ergebnis wird in das PHP array $zehnids gesteckt.
Das Problem: Im Zweiten Schritt wird dieselbe Tabelle nach weiteren Spalten unter Benutzung der $zehnids-Werte abgefragt.
Doch die Zweite Abfrage bringt die Fehlermeldung:
Fatal error: Uncaught PDOException: SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'id' in where clause is ambiguous in G:
Wo ist der Fehler? Oder lässt sich die ganze Abfrage-Technik besser machen?
Der Grund, warum ich dieses Konstrukt verwende, ist: die spalte8 wird am Ende des Skripts (schritt3) hochgezählt, also geändert - wodurch bei der nächsten Abfrage andere ids als result gezogen werden. Darum ist es wichtig, immer mit den gleichen ids zu fragen.
Aha, der Klassiker. "Column 'id' is ambiguous" - das klingt erst mal kryptisch, ist aber ganz einfach.
Das heißt: In deiner SQL-Abfrage gibt es zwei Spalten mit dem Namen id, eine aus tabelle1 und eine aus tabelle2.
MySQL weiß also nicht, welche id du meinst.
Hier ist dein Fehler:
WHERE (tabelle1.autor = tabelle2.user AND `id` = ?)
Da musst du klar sagen, von welcher Tabelle die ID stammt. So:
WHERE (tabelle1.autor = tabelle2.user AND tabelle1.id = ?)
Damit weiß MySQL Bescheid, und der Fehler ist weg.
Aber noch was: Dein Code lässt sich deutlich sauberer schreiben. Du brauchst nicht für jede ID eine neue Abfrage zu machen. Das ist langsam und unübersichtlich.
Statt foreach kannst du alle IDs in einem Rutsch holen.
Zum Beispiel so:
<?php
$placeholders = str_repeat('?,', count($zehnids) - 1) . '?'; $sql = "SELECT t1.id, t1.frage, t1.gez, t2.look FROM tabelle1 t1 JOIN tabelle2 t2 ON t1.autor = t2.user WHERE t1.id IN ($placeholders)"; $stmt = $pdo->prepare($sql); $stmt->execute($zehnids); ?>
Dann bekommst du alle passenden Zeilen auf einmal, ohne die Schleife mit zig einzelnen Abfragen.
Und noch ein kleiner Hinweis: Du schreibst in deiner zweiten Abfrage:
<?php
$stmt->execute(array('$einzid')); ?>
Das ist falsch. Die Variable wird als String behandelt, weil du sie in Anführungszeichen setzt.
Richtig ist:
<?php
$stmt->execute(array($einzid)); ?>
Viele Grüße Nico
Nachricht wurde verfasst am 14.10.2025 um 09:36 Uhr von
