Hoe maak ik hier een uitvoerbare regel van:

DELETE FROM banner WHERE autodel(in tabel banner_instellingen) = '1'AND banner_tot(in tabel banner) <= '0'

Oftuwel, in tabel banner moet de regel worden verwijderd, waar banner_tot een waarde van 0 of lager heeft, en er moet in de tabel banner_instellingen, bij autodel de waarde 1 staan.

Misschien twee select statements maken voor de twee voorwaarden.
Dan met de functie mysql_num_rows() in een if-statement de DELETE-opdracht uitvoeren

Origineel gepost door PascalB.

Misschien twee select statements maken voor de twee voorwaarden.
Dan met de functie mysql_num_rows() in een if-statement de DELETE-opdracht uitvoeren

Waarom 2 selects, gewoon een join erin zetten?

Afhankelijk van je SQL-skills natuurlijk

Maar, ik heb daar niet veel verstand van.
Kun je het voordoen?
Anders moet ik even een altanatief bedenken.

Effe zo uit m'n hoofd (kan het helaas nu neit testen):



$result1 = mysql_query("SELECT bannerId FROM banner_instelling WHERE autodel = 1 AND bannerId=".$JouBanner.")";

$result2 = mysql_query("SELECT bannerId FROM banner WHERE banner_tot = <=0 AND bannerId=".$JouBanner.")";

if ( mysql_num_rows($result1) == 1 && mysql_num_rows($result2) == 1 )
{
// DELETE QUERY
// ongeveer: DELETE FROM banner WHERE BannerId = $JouBanner
}

banner id hoeft niet volgens mij.
Hij moet ze allemaal checken, of ze daaraan voldoen, en diegene verwijderen.
Maar bedankt alvast voor de hulp iedereen

Oh. ik dacht dat het over 1 banner ging. Indien meerdere kun je dat stuk SQL weglaten :)

Moet ik dat SELECT bannerId ook weglaten?

Nee. Je moet wel iets selecteren wat aan de criteria voldoet. Dit moet dus 1 van de kolommen van de tabel zijn. Ik ga ervan uit dat je een BannerId hebt, dus heb ik die als voorbeeld toegevoegd.

Ok, en dan bij dit stukje:
{
DELETE QUERY FROM banner
}

Zoiets?

Waarom zorg je er niet voor dat het veld autodel niet in de tabel banner staat, Lijkt me makkelijker.

Omdat die heel veel rijen heeft.
En het over alles moet gaan, dus niet over elke banner appart wel of niet.

Dus is netter alle instellingen in 1 tabel los.

Ik denk dat je dan iets als dit nodig hebt.

PHP Code:

<?php
 
//Haal de instelling autodel op.
$deletebanner = "SELECT autodel FROM banner_instellingen";
$deletebannerq = mysql_query($deletebanner) or die(mysql_error());
$deleterow = mysql_fetch_assoc($deletebannerq);
             
if ($deleterow['autodel'] == 1) 
{   
 $deletesql = "DELETE FROM banner WHERE banner_tot = 0"; // Wel deleten                    
}
else
{
 // Niet deleten  
}
// Einde deleten 
?>

Daarmee gebruik je 2 sql querys waardoor je 2x de database raadpleegt. Niet echt het snelste.

Het kan ook in 1x:

Code:

Delete from banner inner join banner_instellingen on banner.RELATIE = banner_instellingen.RELATIE where banner.banner_tot <= 0 and banner_instellingen.autodel = 1;

(uit mijn hoofd, maak een backup voor het testen ;) )