Dzień dobry

Mam dwie tabele w bazie:

reklama:

id reklamodawca rodzaj wymagana_widownia ilosc_emisji ilosc_dni cena Rosnąco kara

i

kontrakty_reklamy:

id id_gracza id_reklamodawcy czas_konca brakujace_emisje status

Chciałbym wyświetlić wszystkich reklamodawców, z wyjątkiem tych, którzy mają trwający kontrakt (`status`='trwa') z danym graczem.

Po prostu dołącz (JOIN) drugą tabelę i w klauzuli WHERE umieść odpowiedni warunek:

[SQL] pobierz, plaintext 
SELECT a.id, a.col_a, a.col_b, ..., a.col_x
FROM table_a AS a
LEFT JOIN table_b AS b
WHERE b.STATUS != "trwa";
[SQL] pobierz, plaintext 

Ja stworzyłem takie coś:

SELECT *
FROM reklamy
LEFT JOIN kontrakty_reklamy ON reklamy.id = kontrakty_reklamy.id_reklamodawcy
WHERE kontrakty_reklamy.id_gracza IS NULL
OR kontrakty_reklamy.id_gracza !=1
OR kontrakty_reklamy.id_gracza =1
AND STATUS != 'trwa'
LIMIT 0 , 30

Czy to jest poprawne?

(próbowałem i wydaje mi się, że dokładnie o to mi chodziło, ale jestem Ciekaw opinii "z zewnątrz"

Nie jest poprawne. Nigdy nie miesza się AND i OR bez używania nawiasów.

Takie zapytanie spełnia moje oczekiwania:

SELECT * FROM `reklamy` WHERE NOT EXISTS(
SELECT id_reklamodawcy FROM `kontrakty_reklamy` WHERE `reklamy`.`id` = `kontrakty_reklamy`.`id_reklamodawcy` AND `kontrakty_reklamy`.`id_gracza`='1' AND `kontrakty_reklamy`.`status`='trwa'
);

(akurat w tym przypadku id.gracza=1, oczywiście będzie to de facto id.gracza={$id_gracza}, ale to już szczegół

Użyj LEFT JOINa z NULLem zamiast NOT EXISTS - będzie wydajniej. Duzo wydajniej.

Użyj LEFT JOINa z NULLem zamiast NOT EXISTS - będzie wydajniej. Duzo wydajniej.