[php/mysql] Delete functie werkt niet goed.

[Basbarbrbarbr]

Ik ben aan het proberen om een deletefunctie toe te voegen aan mijn webpagina, helaas werkt mijn script niet zoals ik wil

iig ik wil uit verschillende tabellen verwijderden waar het killid bijv 100 is, nu gaat dat in 2 van de 3 tabellen goed behalve waar het killid primary key is. iig hier me script:

pagina 1:

code:

<form method="post" action="deletemail2.php">
Killmail id: <input type="text" name="kmid"><br />
<input type="submit" name="Submit" value="Submit">
</form>

pagina2:

code:

$query1 = "DELETE FROM Kill WHERE KillID = '$kmid'";
mysql_query($query1) or die(mysql_error());
$query2 = "DELETE FROM Destroyed WHERE KillID = '$kmid'";
mysql_query($query2) or die(mysql_error());;
$query3 = "DELETE FROM Involved WHERE KillID = '$kmid'";
mysql_query($query3) or die(mysql_error());;

2 en 3 doet hji zonder problemen leeg maken, maar bij 1 wil het me niet lukken

[NMe]

Allereerst komt het nogal vreemd op me over dat je werkwoorden/voltooid deelwoorden als tabelnaam kiest. Entiteiten ("tabellen") hebben doorgaans zelfstandige naamwoorden als naam, en als je een tabel alleen maar kan benoemen met een werkwoord, dan duidt dat vaak op slecht ontwerp. Is de database goed genormaliseerd?

Dat heeft verder niets met je probleem te maken overigens. Hoe zit het met referentiële integriteit? Als je binnen MySQL InnoDB gebruikt en de KillID velden in de 2e en 3e tabel zijn als foreign key gedefiniëerd, dan moet je eerst 2 en 3 uitvoeren, en dan pas 1.

[Sybr_E-N]

Je moet even meer informatie geven, want dit is wel erg karig. Wat gaat er dan precies mis? Welke error informatie krijg je dan? Toevallig InnoDB als type? Van wat voor type is KillID, een integer of string. Want je zet nu kwootjes om een integer getal, dat is nergens voor nodig, tenzij het een string is dan.

[ Voor 4% gewijzigd door Sybr_E-N op 18-08-2006 09:01 ]

[KabouterSuper]

Wellicht zijn je database tabellen case sensitive? Je schrijft de tabel 'kill' met kleine letters, terwijl de andere twee tabellen beginnen met een hoofdletter.

[Basbarbrbarbr]

@ -NMe-
geprobeert, maar krijg telkens devolgende error:

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 'Kill WHERE KillID = 100' at line 1

@Sybr_E-N :
het is een int(11)


@KabouterSuper
nee klopt zo

ben niet zo thuis in Mysql, heb geprobeert op te zoeken wat InnoDB inhoud maar kan het niet zo direct vinden


overigens mijn nieuwe code:

code:

$query1 = "DELETE FROM Involved WHERE KillID = $kmid";
$query2 = "DELETE FROM Destroyed WHERE KillID = $kmid";
$query3 = "DELETE FROM Kill WHERE KillID = $kmid";
mysql_query($query1) or die(mysql_error());
mysql_query($query2) or die(mysql_error());
mysql_query($query3) or die(mysql_error());

[ Voor 60% gewijzigd door Basbarbrbarbr op 18-08-2006 09:16 ]

[KabouterSuper]

De foutmelding ziet er standaard uit: ongeldig statement.

Hoe ziet de tabel eruit? Wellicht heb je een extra underscore in de kolomnaam gestopt.....Kill_ID in plaats van KillID.

[NMe]

KILL is een reserved word in MySQL 5.0 en kun je dus niet als tabelnaam gebruiken zonder er backticks ( ` ) omheen te zetten in je queries. Kies liever een andere naam of zet er dus backticks omheen.

Als je niet weet wat InnoDB is, dan is er een héél kleine kans dat je het gebruikt. Niet druk over maken.

[moto-moi]

SQL:

delete from kill where killid = blaat;

Oftewel: kill is een keyword in sql, dit zal wel werken:

SQL:

delete from `kill` where killid = blaat

[ Voor 0% gewijzigd door moto-moi op 18-08-2006 09:17 . Reden: stomme -NMe- :'( ]

[Basbarbrbarbr]

jaaaa

dat doet het, heb ik weer

bedankt mensen

[MBV]

ehm, zeg me alsjeblieft dat je daar wat checks omheen hebt staan, en dat je register_globals niet aan hebt staan

Als je geen checks uitvoert kan ik namelijk zo een DROP TABLE of erger uitvoeren...