[PHP]Bindowanie - czy dobrze to robię? Opcje

[sadistic_son]

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?

[trueblue]

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.

[sadistic_son]

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.

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?

bez użycia metody bind - id i tak masz integer.

No w sumie racja.

Zamiast pętli for zastosowałbym usuwanie z użyciem klauzuli IN

Np tak?

Kod

delete from t
where id in (1, 4, 6, 7)

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

Ten post edytował sadistic_son 10.01.2023, 12:15:58

[trueblue]

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

Ten post edytował trueblue 10.01.2023, 12:18:08

[sadistic_son]

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 

[trueblue]

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

[sadistic_son]

Dopisałem w poście wcześniejszym (do tego jest array_filter).

Tutaj w tym temacie? Wybacz , ale nie dopisałeś (IMG:style_emoticons/default/wink.gif)

Dlaczego foreach? Od tego jest implode.

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ę?

Ten post edytował sadistic_son 10.01.2023, 12:55:53

[trueblue]

Tutaj w tym temacie? Wybacz , ale nie dopisałeś (IMG:style_emoticons/default/wink.gif)

Dopisałem w poście #4.

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

A o tym w poście #6.

[sadistic_son]

Ok, wszystko jasne. Ale array_map też robi robotę (IMG:style_emoticons/default/smile.gif)
Dzięki za pomoc.