Cześć.
Mam taką metodę bind w klasie DB:

[PHP] pobierz, plaintext 
namespace nameOne;
 
class DB
{
    private $dbh;
    private $stmt;
 
    public function __construct()
    {
        $this->dbh = new \PDO("mysql:host=" . DB_HOST . ";dbname=" . DB_NAME, DB_USER, DB_PASS);
    }
 
    public function bind($param, $value, $type = null)
    {
        if (is_null($type)) {
            switch (true) {
                case is_int($value):
                    $type = \PDO::PARAM_INT;
                    break;
                case is_bool($value):
                    $type = \PDO::PARAM_BOOL;
                    break;
                case is_null($value):
                    $type = \PDO::PARAM_NULL;
                    break;
                default:
                    $type = \PDO::PARAM_STR;
            }
        }
        $this->stmt->bindValue($param, $value, $type);
    }
}
[PHP] pobierz, plaintext 

W formularzu przesyłam checkboxy, wszystkie z tej samej tablicy, powiedzmy name="boxChecked[]".

I teraz chcę usunąć te rekordy z bazy w których ID mam przesłane w $_POST['boxChecked']

Robię to tak:

[PHP] pobierz, plaintext 
    private function massDelete(array $recordsIds)
    {
        $db = new DB();
        foreach ($recordsIds as $row) {
            $row = (int) $row;
            $db->query("DELETE FROM `product_property` WHERE `product_property`.`product_id` = :id;");
            $db->bind(':id', $row);
            $db->execute();
 
            $db->query("DELETE FROM `product` WHERE `product`.`id` = :id;");
            $db->bind(':id', $row);
            $db->execute();
        }
    }
 
//w innym miejscu wywołanie metody:
                if (isset($_POST['mass_delete_submit']) && isset($_POST['boxChecked'])) {
                    $this->massDelete($_POST['boxChecked']);
                }
[PHP] pobierz, plaintext 

Ten kod działa.
Moje pytanie: Czy to jest po bożemu zrobione? Czegoś brakuje? Z tego co się orientuję to dane z POST z checkboxa to tablica stringów, więc typowanie na int by sie przydało w metodzie massDelete, tak? Czy to >>> $row = (int) $row; wystarczy?

Nie wiem czy masz założone więzy integralności na tych dwóch tabelach, ale gdyby były to wystarczy usuwanie produktu - dzięki więzom integralności zostaną usunięte właściwości bez potrzeby usuwania ich po stronie PHP.
Zamiast pętli for zastosowałbym usuwanie z użyciem klauzuli IN i bez użycia metody bind - id i tak masz integer.

Rozumiem. Nie porobiłem więzów integralności. Ale to moje niedopatrzenie. A gdyby były to wystarczy że wykonam drugie zapytanie (usuwanie z `product`) a zapytanie pierwsze wykona się samo po stronie serwera mysql?


No w sumie racja.

Np tak?O to Ci chodziło? W sumie chyba jest to bardziej wydajne niż oddzielne zapytania per iteracja pętli.

Tak.
Tak. Sprawdź jednak czy każde id jest integer, powinno być, ale nie musi.

Ok, dzięki.
Tylko jeszcze trzeba wtedy te IDki z checkboxa jakoś wyczyścić. Czy to tak może być?

[PHP] pobierz, plaintext 
    private function massDelete(array $recordsIds)
    {
        $db = new DB();
        $idsToDel = '';
        $i = 1;
        foreach ($recordsIds as $row) {
            $row = (int) $row;
            $idsToDel .= $row;
            if (count($recordsIds) != $i) {
                $idsToDel .= ',';
            }
            $i++;
        }
        $query = "DELETE FROM `product_property` WHERE `product_property`.`product_id` IN ($idsToDel)";
        $db->query($query);
        //$db->bind(':id', $row); bindowanie już zbędne, bo mamy INTy, tak?
        $db->execute();
[PHP] pobierz, plaintext 

Dopisałem w poście wcześniejszym (do tego jest array_filter).
Dlaczego foreach? Od tego jest implode.

Tutaj w tym temacie? Wybacz , ale nie dopisałeś

A no racja. Prościej. Dzięki!

EDIT:
Ale chwila. A jak teraz ma wyglądać upewnienei się, że mam same INTy w wartościach do bazy?

[PHP] pobierz, plaintext 
    private function massDelete(array $recordsIds)
    {
        $db = new DB();
        $idsToDel = implode(',',$recordsIds);
 
        $db->query("DELETE FROM `product_property` WHERE `product_property`.`product_id` IN ($idsToDel)");
        $db->execute();
 
        $db->query("DELETE FROM `product` WHERE `product`.`id` IN ($idsToDel)");
        $db->execute();
 
    }
[PHP] pobierz, plaintext 

EDIT2: myślę, że tak będzie dobrze:

[PHP] pobierz, plaintext 
$idsToDel = implode(", ", array_map('intval', $recordsIds));
[PHP] pobierz, plaintext 

Zgadza się?

Dopisałem w poście #4.


A o tym w poście #6.

Ok, wszystko jasne. Ale array_map też robi robotę
Dzięki za pomoc.