[PHP]MySQL setters i getters Opcje

[nospor]

[PHP] pobierz, plaintext 
 
        public function listGroupedProducts(): array
        {
            $products = array();
            foreach ($this->getProducts() as $row) {
     
     
                if (!isset($products[$row['prodId']])) {
                    $product = new Product();
                    $product->setProductName($row['name']);
                    $product->setSku($row['sku']);
                    $product->setPrice($row['price']);
                    $product->setProductId($row['sku']);
                    $product->setProductId($row['prodId']);
 
                    $products[$row['prodId']] = $product;
     
                }
                if (!empty($row['value'])) {
                    $products[$row['prodId']]->addProperty($row['value'], $row['label']);
                }
 
            }
            return $products;
        }
[PHP] pobierz, plaintext 

I w klasie Product dopisz metode
addProperty(string $value, string $label)

ktora bedzie dodawala kolekne property do tablicy properties produktu

Powód edycji: [nospor]:

[sadistic_son]

ktora bedzie dodawala kolekne property do tablicy properties produktu

Wybacz, ale nie rozumiem jak ta metoda ma wyglądać. O tej godzinie to w ogóle już mało rozumiem

To masz na myśli?

[PHP] pobierz, plaintext 
    public function addProperty(string $value, string $label)
    {
        $this->setProperties(['value' => $value, 'label' => $label]);
    }
[PHP] pobierz, plaintext 

Ten post edytował sadistic_son 13.01.2023, 16:10:35

[nospor]

masz ddoac a nie nadpisac.


....
private array $properties = [];

...


public function addProperty(string $value, string $label): void {
$this->properties[] = [
'value' => $value,
'label' => $label
];
}

[sadistic_son]

No tak. Oczywiste...
Ech, czas na dłuższą przerwę bo już nie myslę...
Dzięki za pomoc nospor. Liczę, że dziś wieczorem wyślę projekt (lub jutro rano, bo cała noc przede mną).


Hej, pytanie stricte z typu "co autor miał na myśli" I tutaj chciałbym się dowiedzieć jak ty nospor uważasz - czy biorąc pod uwagę poniższe:
""Logika MySQL powinna być obsługiwana przez obiekty z właściwościami, zamiast bezpośrednich wartości kolumn. Proszę użyć setters i getters aby to osiągnąć oraz nie zapomnieć użyć ich do zapisu i wyświetlania logiki."
to można uznać, że spełniam warunek (poniżej) posługiwania sie getter/setter przy zapisie?

[PHP] pobierz, plaintext 
                if (isset($_POST['save']) && isset($_POST['sku']) && isset($_POST['name']) && isset($_POST['price']) && isset($_POST['productType'])) {
                    $product = new Product();
                    $product->setProductName($_POST['name']);
                    $product->setSku($_POST['sku']);
                    $product->setPrice($_POST['price']);
                    $product->setProductType($_POST['productType']);
 
                    $productObject = new ProductAdd($this->db);
                    if (!$productObject->checkExistingSKU($product->getSku())) {
                        $productObject->insertNewProduct($product);
                    }
 
/*
oraz w innym pliku metoda insertNewProduct, a przynajmniej jej kawałek, bez większości bebechów
*/
 
 public function insertNewProduct(Product $product)
    {
        if ($this->ensureNoEmpytValues($product) === false)
            return false;
 
        $sku = $product->getSku();
        $productName = $product->getProductName();
        $price = $product->getPrice();
        $selectedType = $product->getProductType();
 
// tutaj pętal forach - nie ważne
 
        $queryInsertProduct = "INSERT INTO `product` (`id`, `sku`, `name`, `price`, `type`) VALUES (null, :sku, :productName , :price, :selectedType)";
        $this->db->query($queryInsertProduct);
        $this->db->bind(':sku', $sku);
        $this->db->bind(':productName', $productName);
        $this->db->bind(':price', $price);
        $this->db->bind(':selectedType', $selectedType);
        $this->db->execute();
 
        $lastProductId = $this->db->lastInsertId();
[PHP] pobierz, plaintext 

Przed wywołaniem metody do zapisu są settery. W ciele metody do zapisu są gettery. No to chyba spełniam to wymaganie przy zapisie, co? Jeśli też tak to rozumiesz to (chyba) skończyłem...


Ten post edytował sadistic_son 13.01.2023, 17:52:31

[nospor]

no chyba tak

Tylko ne tworz takich cudow
$sku = $product->getSku();

przeciez mozsesz od razu:
$this->db->bind(':sku', $product->getSku());

[sadistic_son]

Tylko ne tworz takich cudow
$sku = $product->getSku();

przeciez mozsesz od razu:
$this->db->bind(':sku', $product->getSku());

Tak, teraz będe się właśnie takich rzeczy pozbywać, wywalać moje komentarze, wszelkie echo 'dupa' i var dumpy wywalać, itp, itd. Ale to już można po kolacji przy piwku I wysyłam. Dzięki wielkie za nieoceniona pomoc!


Hej, tak jeszcze jedna rzecz mi nie daje spokoju w moim kodzie. Metoda powinna dostać dane z POST w parametrze, a u mnie w jednym przypadku tak nie jest i metoda dostaje $_POST w ciele funkcji. Dzieje się to tutaj: if (empty($_POST[$propertyData['property']])) oraz tutaj: $this->db->bind(':productValue', $_POST[(string) $propertyData['property']]);
Nie wiem jak przekazać do parametru tą wartość POST, bo nie znam jej indeksu - jest pobierany z bazy na podstawie wypełnionych pól i wybranej wartości z SELECT. Czy powienienm poniższy fragmencik wkomponować w wywołanie metody?

[PHP] pobierz, plaintext 
        $properties = $this->groupFullProperties();
        foreach ($properties[$product->getProductType()] as $propertyData) {
            if (empty($_POST[$propertyData['property']]))
                return false;
        }
[PHP] pobierz, plaintext 

Metodę wywołuję tak:

[PHP] pobierz, plaintext 
                if (isset($_POST['save']) && isset($_POST['sku']) && isset($_POST['name']) && isset($_POST['price']) && isset($_POST['productType'])) {
                    $product = new Product();
                    $product->setProductName($_POST['name']);
                    $product->setSku($_POST['sku']);
                    $product->setPrice($_POST['price']);
                    $product->setProductType($_POST['productType']);
                    $productObject = new ProductAdd($this->db);
                    if (!$productObject->checkExistingSKU($product->getSku())) {
                        $productObject->insertNewProduct($product);
                    }
                    header("Location:http://" . PRODUCT_LIST_LANDING);
                }
[PHP] pobierz, plaintext 

Tutaj kawałek klasy z omawianą metodą:

[PHP] pobierz, plaintext 
    public function insertNewProduct(Product $product)
    {
        if ($this->ensureNoEmpytValues($product) === false)
            return false;
        $properties = $this->groupFullProperties();
        foreach ($properties[$product->getProductType()] as $propertyData) {
            if (empty($_POST[$propertyData['property']]))
                return false;
        }
        $this->db->query("INSERT INTO `product` (`id`, `sku`, `name`, `price`, `type`) VALUES (null, :sku, :productName, :price, :selectedType)");
        $this->db->bind(':sku', $product->getSku());
        $this->db->bind(':productName', $product->getProductName());
        $this->db->bind(':price', $product->getPrice());
        $this->db->bind(':selectedType', $product->getProductType());
        $this->db->execute();
        $lastProductId = (int) $this->db->lastInsertId();
        foreach ($properties[$product->getProductType()] as $propertyData) {
            $propertyId = $propertyData['id'];
            $this->db->query("INSERT INTO `product_property` (`id`, `product_id`, `property_id`, `value`) VALUES (null, $lastProductId, $propertyId, :productValue )");
            $this->db->bind(':productValue', $_POST[(string) $propertyData['property']]);
            $this->db->execute();
        }
 
[PHP] pobierz, plaintext 

[nospor]

Oj chyba sie zgubilem. O co dokladnie pytasz?

[sadistic_son]

Chodzi mi o to, że:

W formularzu mam pole select z którego wybieram rodzaj produktu. Jak jest to np mebel to pojawiają się inputy dla wysokości, szerokości i długości. Dla dvd pojawia się rozmiar (MB), itd. Teraz wywołując metodę $productObject->insertNewProduct($product); podaję jej jako parametr obiekt $product w którym to za pomocą getterów mam ustawione wartości pozostałych inputów, ale bez tych dodawanych dynamicznie (szerokośc, wysokość, rozmiar itd - $_POST['width'] itd) bo w momencie wywoływania tej metody nie wiem jeszcze które z inputów zostały poprzez select dodane do formularza. Dowiaduję się tego dopiero wewnątrz metody, poprzez odwołanie bezpośrednio do $_POST[$propertyData['property']] tworzonego w forach (spójrz na kod powyżej). I traz moje pytanie to - jak mam przekazać w parametrze to $_POST['width'] (albo $_POST['size'] albo dowolne inne wysłane) przy wywołaniu insertNewProduct($product) zamiast wrzucać je w ciele metody?

Czy powinienem poniższy kawałek (który obecnie jest tylko w ciele medoty insertNewProduct) równierz wykonywać przy sprawdzeniu czy w ogóle wywowłać insertNewProduct?

[PHP] pobierz, plaintext 
        $properties = $this->groupFullProperties();
        foreach ($properties[$product->getProductType()] as $propertyData) {
            if (!empty($_POST[$propertyData['property']])) //wywołaj insertNewProduct
[PHP] pobierz, plaintext 

Oczywiście trzeba ten kawałek zmodyfikować by dodawał do obiektu $product też tablicę zawierającą $_POST[$propertyData['property']].

Ten post edytował sadistic_son 23.01.2023, 15:42:24

[nospor]

Poprostu do metody przekaz caly $_POST i po sprawie.
Normalnie bys przekazal obiekt Request jak cie prosilem tyle razy, ale ze nie napisales to przekazuj caly $_POST

[sadistic_son]

No ok, ale co w tej klasie Request miałoby być? Nie napisałem jej, bo kiedy dopytywałem o czyszczenie POST to mi doradziłeś, że wystarczy bindowanie do bazy oraz htmlspecialchars do wyświetlania POST wystarczy. Oraz pisałeś też że czyszczeni POST powinno być w oddzielnej klasie właściwie. Więc nie mam pomysłu jak ta klasa w ogóle ma wyglądać i po co być.

Coś w TYM stylu? Wiem, że to sprzed 11 lat...

Ten post edytował sadistic_son 23.01.2023, 17:17:07

[viking]

Dawałem ci wcześniej linki do klas na github. Nie możesz zajrzeć i zobaczyć co w nich jest? Na tym też polega programowanie. Patrzeć jak piszą lepsi.

[nospor]

Nie napisałem jej, bo kiedy dopytywałem o czyszczenie POST to mi doradziłeś, że wystarczy bindowanie do bazy oraz htmlspecialchars do wyświetlania POST wystarczy. Oraz pisałeś też że czyszczeni POST powinno być w oddzielnej klasie właściwie.

No ale to nie moja wina ze sie zafiksowales na tym czyszczeniu i nie czytales co sie do ciebie innego pisze.

Pisalem ci wyraznie, ze w Request na poczatek masz mie getPost() getGet() getByKey() getUri
takie podstawowe metody ktore zwracaja rzeczy ktorych uzywasz z request.

Inni, jak juz viking wspomnial, tez podawali ci linki do przykladowych klas. Takze nie, nikt ci tutaj nie doradzal bys rezygnowal z klasy Request.