Syntax-Problem MySQL mit INNER JOIN

Thonixx

Ich habe ein Problem, das ich irgendwie einfach nicht checke.
Das INNER JOIN kenn ich recht gut und ich denke das muss wohl richtig sein, da es bei anderen Beispielen funktioniert hat.

Nur bei diesem einen SQL Statement habe ich ein Syntax-Problem, welches ich nicht begreife.
Es handelt sich um:

PHP

SELECT
    `surveys`.`id` as `sid`,
    `surveys`.`datum` as `dat`,
    `shops`.`name` as `snam`,
    `seller`.`name` as `nam`,
    `seller`.`fname` as `fnam`
FROM
    `surveys`
INNER JOIN
    `shops` ON `surveys`.`shop` = `shops`.`id` AND `seller` ON `suverys`.`seller` = `seller`.`id`

Und es gibt folgenden Fehler aus:

Zitat von localhost

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ON `suverys`.`seller` = `seller`.`id`' at line 10

Die Tabellen existieren und die Verbindung muss funktionieren, die Verbindungspunkte stimmen überein und es müsste auch etwas ausgeben.
Danke für Hilfe.

crAzywuLf

probiers mal so:

SQL

SELECT
    `surveys`.`id` as `sid`,
    `surveys`.`datum` as `dat`,
    `shops`.`name` as `snam`,
    `seller`.`name` as `nam`,
    `seller`.`fname` as `fnam`
FROM
    `surveys`
INNER JOIN
    `shops` ON (`surveys`.`shop` = `shops`.`id`)
INNER JOIN
    `seller` ON (`surveys`.`seller` = `seller`.`id`)

Alles anzeigen

bin mir aber auch nicht sicher wie das bei 3 Tabellen war^^

Thonixx

Danke, habs aber jetzt so:

SQL

SELECT
    `surveys`.`id` as `sid`,
    `surveys`.`datum` as `dat`,
    `shops`.`name` as `snam`,
    `seller`.`name` as `nam`,
    `seller`.`fname` as `fnam`
FROM
    `surveys`
INNER JOIN
    `shops` ON `surveys`.`shop` = `shops`.`id`
INNER JOIN
    `seller` ON `surveys`.`seller` = `seller`.`id`

Alles anzeigen

Hatte zuerst als Übergangslösung mal das:

SQL

SELECT
    `surveys`.`id` as `sid`,
    `surveys`.`datum` as `dat`,
    `shops`.`name` as `snam`,
    `seller`.`name` as `nam`,
    `seller`.`fname` as `fnam`
FROM
    `surveys`, seller, shops
WHERE
    surveys.seller = seller.id AND surveys.shop = shops.id

Was von beidem ist jetzt schlauer?
Ich würde mal die JOIN Lösung bevorzugen, aber du?

EDIT:
Bei dir ist ein kleiner Schreibfehler drin
Es heisst surveys und nicht suverys. Aber danke für die Hilfe.

crAzywuLf

Zitat

Was von beidem ist jetzt schlauer?
Ich würde mal die JOIN Lösung bevorzugen, aber du?

Ist beides gleich performant, ich persöhnlich finde die multi-select Variante besser, weil Sie mir einfach übersichtlicher erscheint.

Thonixx

Multi-Select?
Also das mit dem Where?

crAzywuLf

Zitat von Thonixx

Multi-Select?
Also das mit dem Where?

jop...

Thonixx

Aber warum findest du das besser oder übersichtlicher?
Ich finde persönlich die INNER JOIN Sache viel übersichtlicher, da man zumal mit OUTER JOIN, LEFT JOIN, RIGHT JOIN und anderen Dingen mehr Funktionalität hat, als nur mit dem WHERE.

Was spricht also gegen JOIN und was für WHERE?

crAzywuLf

Zitat von Thonixx

Aber warum findest du das besser oder übersichtlicher?

Geschmackssache. INNER Join und die WHERE Variante sind gleich schnell. Ich mag es halt lieber wenn die Tabellen auf einen Haufen sind und die Bedingungen auf einem, außerdem ist es weniger zu schreiben

Zitat von Thonixx

Ich finde persönlich die INNER JOIN Sache viel übersichtlicher, da man zumal mit OUTER JOIN, LEFT JOIN, RIGHT JOIN und anderen Dingen mehr Funktionalität hat, als nur mit dem WHERE.

Was spricht also gegen JOIN und was für WHERE?

LEFT JOIN und RIGHT JOIN machen doch völlig andere Sachen.

synaptic

ich mag auch nen multi-select lieber ich hab lieber nen arsch voll where-bedingungen, als nur einmal join irgendwo stehen
find ich weitaus übersichtlicher

Thonixx

Ja aber ich verstehe nicht ganz, warum du lieber zeilenweise WHERE Bedinungen hast, wenn auch einfach ein JOIN reicht.
Wurde JOIN nicht extra deswegen entwickelt?

synaptic

wat weiß ich wofür der entwicklet wurd..
ich find zeilenweise joins auch kacke

schreib mal des hier als join auf

Code

select s.name as 'skill', ms.wert
        from mitarbeiter m, rolle r, mit_skill ms, skill_roll sr, skills s
    where m.id = ms.mitarb_id
        and   r.id = sr.roll_id
        and   ms.skill_id = sr.skill_id
        and m.name = 'klopotek'
        and r.name = 'entwickler'
        and s.id = ms.skill_id
        and basic = 0

Thonixx

Okay, sagen wirs mal so, manchmal ist JOIN gut und manchmal einfach ein WHERE.
Ich entscheide einfach je nach Situation

SinnlosS

Ich persönlich bevorzuge auch joins weil ich sie ebenfall viel übersichtlicher finde, das sind "unterschiedliche" where-bedingungen und die gehören für mich getrennt.
Bei den einen geht es um die Bedingungen der Tabellenverknüpfungen, in den anderen um fest Wert "von außen".

PHP

$sql = "
        SELECT
            ia.id,ia.vorgabe,ia.preis,ia.start,ia.stop,ia.bezahlt,ia.erhalten,ia.finished,
            sb.id,sb.email,sb.name,sb.passwort,sb.url,
            sn.id,sn.name,sn.logo,
            a.id,a.aktion,
            ifnull(MAX(id.datum),'2070-12-12 23:59:59'),ifnull(SUM(id.done),0),
            ifnull(ac.id,0),ifnull(ac.name,'---')
        FROM
            ind_assign ia
            INNER JOIN
                sn_band sb
                    ON
                    sb.id=ia.sn_band_id
                INNER JOIN 
                    sn
                        ON
                        sn.id=sb.nwk
            INNER JOIN
                aktionen a
                    ON
                    a.id=ia.aktion_id
            LEFT JOIN
                ind_done id
                    ON
                    id.ind_assign_id=ia.id
            LEFT JOIN
                ind_account ac
                    ON
                    ac.id=ia.ind_sub_id
        WHERE
            (ia.ind_id=?
            OR
            ia.ind_sub_id=?)
            AND
            ia.finished<2
        GROUP BY
            sn.name ASC,a.aktion ASC,sb.email ASC";

Alles anzeigen

Ka, also da möchte ich meine Inner Joins nicht missen.

Generell ist es aber letztendlich sowieso Geschmackssache, solange man da keine Vorgaben hat soll man das natürlich so machen wie man selber gut mit klarkommt.

Tags