[sql/asp] DELETE error

[Mischa_NL]

Hey tweakers,

Bij een forum moet je natuurlijk topics (met inhoud) kunnen deleten.
Zo ook bij mijn forum. Daarom had ik deze query in elkaar geprutst.

code:

strSQL = "DELETE tblTopics.*, tblPosts.* FROM tblTopics, TblPosts "
strSQL = strSQL & "WHERE tblTopics.topic_ID = tblPosts.Topic_ID AND tblTopics.topic_id=" & request.QueryString("t")

De query geeft een nogal rare fout als hij hem moet uitvoeren met ASP:

code:

Microsoft JET Database Engine error '80004005'

Record is deleted.

Wat is er in godsnaam fout aan mijn query?! Wie kan me helpen?

[NMe]

Kun je wel uit 2 tabellen tegelijk verwijderen? En zo ja, geeft dat in dit geval dan geen problemen met eventuele referentiële sleutels van de posts tabel naar de topics tabel?

[Mischa_NL]

ik doe het wel 1 voor 1, wat maakt het ook uit

EDIT/ Dat werkt dus ook niet. het probleem is dat ik MEERDERE posts moet deleten meestal.

topic met 1 post (de ts post) werkt WEL
een topic met replies kan ik niet wegooien

[ Voor 65% gewijzigd door Mischa_NL op 20-07-2004 20:22 ]

[pistole]

Mischa_NL schreef op 20 juli 2004 @ 20:07:

code:

strSQL = "DELETE tblTopics.*, tblPosts.* FROM tblTopics, TblPosts "
strSQL = strSQL & "WHERE tblTopics.topic_ID = tblPosts.Topic_ID AND tblTopics.topic_id=" & request.QueryString("t")

sinds wanneer geeft je veldnamen op als je records gaat verwijderen?

[Verwijderd]

Delete from blabla Where blabla is toch de normale syntax?
Raar asterix-gedoe

[Mischa_NL]

code:

strSQL = "DELETE FROM tblTopics, tblPosts "
strSQL = strSQL & "WHERE tblposts.topic_ID =" & request.QueryString("t") & " AND tblTopics.topic_id=" & request.QueryString("t")

adoCon.execute(strSQL)

Dit werkt dus OOK niet, nu de fout:

code:

Specify the table containing the records you want to delete.

[SuperRembo]

De simpelste oplossing lijkt mij om het met 2 queries te doen. Eerst de posts bij het topic verwijderen, dan het topic zelf.

SQL:

DELETE FROM tblPosts WHERE Topic_ID = @topicID
DELETE FROM tblTopics WHERE Topic_ID = @topicID

Je zou ook een cascade delete kunnen instellen op de posts-topics foreign key. Dan worden de posts bij een topics automatisch verwijderd als je een topic verwijderd.

[Verwijderd]

De syntax is Delete Table1, Table2 FROM Table1, Table2 Where ...
Het ligt dus aan die asterixjes

(heb het zelf moeten opzoeken, had nog nooit van multiple delete gehoord)

[gorgi_19]

Vraagje; waarom leg je geen relaties met cascading delete?
* gorgi_19 gaat trouwens geen opmerkingen maken over SQL Injection.

[ Voor 34% gewijzigd door gorgi_19 op 20-07-2004 22:34 ]

[Verwijderd]

gorgi_19 schreef op 20 juli 2004 @ 22:34:
Vraagje; waarom leg je geen relaties met cascading delete?
* gorgi_19 gaat trouwens geen opmerkingen maken over SQL Injection.

Dat wordt zijn volgende topic: waar zijn al mijn topics heen

Nee, ik denk dat sql injection geen groot gevaar gaat doen ontstaan op deze pagina, want ze is zowiezo - hoop ik - niet toegankelijk voor gewone gebruikers.

[gorgi_19]

Verwijderd schreef op 21 juli 2004 @ 11:37:
[...]
Dat wordt zijn volgende topic: waar zijn al mijn topics heen

Dan moet je 'softdeletes' gaan maken. En die cascading zut zorgt er alleen voor dat de relaties die je hebt gelegd afgedwongen worden.

Nee, ik denk dat sql injection geen groot gevaar gaat doen ontstaan op deze pagina, want ze is zowiezo - hoop ik - niet toegankelijk voor gewone gebruikers.

Never trust user input. Met een beetje pech wordt er gebruik gemaakt van SQL Server; een boze moderator geeft zichzelf dan een paar extra roles.