delete multiple rows using php array

Question

$ids = $_POST['ids'];

// sky***earth***sea***sun***...

$ids = explode("***", $ids);

foreach ($ids as $id) {
    $st = $db->query("delete from tags where id = " . $id);
}

Is there a more elegant way to delete multiple rows, especially regarding peformances in case of huge array? Something like:

$st = $db->query("delete from tags where id in " . $ids);

Any suggestion?

This is a duplicate of stackoverflow.com/questions/17657760/…. On top of that, your code is vulnerable to SQL Injections, please learn about them here: stackoverflow.com/questions/60174/… — Paul
– Paul, Commented May 6, 2018 at 17:42
@Paul, on client side there is no any input, just user click event. Is the code still vulnerable to sql injeciton? — qadenza
– qadenza, Commented May 6, 2018 at 17:45
Possible duplicate of Better method to delete multiple rows in a MySQL database with PHP? — Pradeep
– Pradeep, Commented May 6, 2018 at 17:45
@bonaca you are using the superglobal $_POST. There is no need for any form in the frontend. I can just send a POST request to that specific file and my payload in ids will be passed to the query without any filtering, escaping or something related. — Paul
– Paul, Commented May 6, 2018 at 17:48
If your id is a text column i.e. sky, earth etc Then the $id needs to be wrapped in quotes like $st = $db->query("delete from tags where id = '$id' "); But do pay heed to the SQL Injection Attack Even if you are escaping inputs, its not safe! Use prepared parameterized statements in either the MYSQLI_ or PDO API's — RiggsFolly
– RiggsFolly, Commented May 6, 2018 at 17:56

Kearl · Accepted Answer · 2018-05-06 18:48:39Z

1

Just replace the exploded *** with a comma(,)

$st = $db->query("delete from tags where id in (" . implode(",", explode("***", $ids)) .")");

Source: https://stackoverflow.com/a/17657893/5837918

answered May 6, 2018 at 18:48

Kearl

865 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

sumit saini · Accepted Answer · 2018-05-06 18:16:28Z

1

$kk= ''; foreach ($ids as $id) { $kk.=$id. ','; } $kk = rtrim($kk, ','); $st = $db->query("delete from tags where id In($kk))";

answered May 6, 2018 at 18:16

sumit saini

323 bronze badges

Collectives™ on Stack Overflow

delete multiple rows using php array

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related