[PHP]MySQL setters i getters Opcje

[sadistic_son]

Cześć.
Walczę z kolejnym wymogiem projektu nad którym siedzę.
"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."
Póki co to nie użyłem setter/getter. Pokażcie proszę na przykładzie mojego poniższego kodu jak to powinno być zastosowane.

Klasa Produkt

[PHP] pobierz, plaintext 
class Product
{
 
    public $productName;
    public $sku;
    public $price;
    public $productType;
    public function __construct()
    {
        $this->productName = isset($_POST['name']) ? (string) $_POST['name'] : null;
        $this->sku = isset($_POST['sku']) ?  (string) $_POST['sku'] : null;
        $this->price = isset($_POST['price']) ? (float) $_POST['price'] : null;
        $this->productType = isset($_POST['productType']) ?  (string) $_POST['productType'] : null;
    }
 
    function ensureNoEmpytValues()
    {
        if (empty($this->productName) || empty($this->sku) || empty($this->price) || empty($this->productType))
            return false;
        return true;
    }
}
[PHP] pobierz, plaintext 

Kawałek klasy ProductAdd z metodą wrzucającą do bazy:

[PHP] pobierz, plaintext 
<?php
class ProductAdd extends Product
{
    public function insertNewProduct()
    {
 
        if ($this->ensureNoEmpytValues() === false)
            return false;
 
        $db = new DB();
 
        $sku = $this->sku;
        $productName = $this->productName;
        $price = $this->price;
        $selectedType = $this->productType;
 
 
        $properties = $this->groupFullProperties();
 
        foreach ($properties[$selectedType] as $propertyData) {
            $postIndex = $propertyData['property'];
            if (empty($_POST[$postIndex]))
                return false;
        }
        $queryInsertProduct = "INSERT INTO `product` (`id`, `sku`, `name`, `price`, `type`) VALUES (null, :sku, :productName , :price, :selectedType)";
        $db->query($queryInsertProduct);
        $db->bind(':sku', $sku);
        $db->bind(':productName', $productName);
        $db->bind(':price', $price);
        $db->bind(':selectedType', $selectedType);
 
        $db->execute();
 
        $lastProductId = $db->lastInsertId();
 
        foreach ($properties[$selectedType] as $propertyData) {
            $propertyId = (int) $propertyData['id'];
            $productValue = $_POST[(string) $propertyData['property']];
 
            $queryInsertProductProperty = "INSERT INTO `product_property` (`id`, `product_id`, `property_id`, `value`) VALUES (null,  $lastProductId , $propertyId, :productValue )";
            $db->query($queryInsertProductProperty);
            $db->bind(':productValue', $productValue);
 
            $db->execute();
        }
    }
}
[PHP] pobierz, plaintext 

[nospor]

Teraz na początku ProductList.view.php tworze obiekt $products = new ProductList($db)

$products powinno byc przekazane do ProductList.view.php a nie tworzone w ProductList.view.php

A jak tworzysz/owierasz ProductList.view.php ?

W config natomiast nie mam żadnej klasy, tylko mam podefiniowane stałe do DB oraz include DB.php, Product.php oraz PageDisplay.php

Czemu w config tworzysz/zaciagasz DB i Product i PageDisplay?

No i skoro PageDisplay zarzadza caloscia, to $db masz przekazac do PageDisplay, to wtedy rzeczy w PageDisplay beda widziec $db. No to sa podstawy z Zasiegu Zmiennych
https://www.php.net/manual/en/language.variables.scope.php

[sadistic_son]

Czemu w config tworzysz/zaciagasz DB i Product i PageDisplay?

Tak mi się wydawało rozsądnie na bazie tego co przerobiłem ze wspomnianej nie raz książki, której przerobienia teraz żałuję. A gdzie mam wtedy zaciągać DB i PageDisplay (Product na razie powstało ale nic jeszcze nie robi )? W index.php ?

No i skoro PageDisplay zarzadza caloscia, to $db masz przekazac do PageDisplay, to wtedy rzeczy w PageDisplay beda widziec $db. No to sa podstawy z Zasiegu Zmiennych
https://www.php.net/manual/en/language.variables.scope.php

Zasięg zmiennych to jeszcze ogarniam. Chyba... :| Ale zaraz to się wyjaśnie jak opiszę moją strukturę.

Tak więc w index.php mam niewiele:

[PHP] pobierz, plaintext 
use nameOne\PageDisplay;
use nameOne\DB;
 
session_start();
 
require('pagefiles/config.php');
$displayPage = new PageDisplay($_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'], $db);
$displayPage->decideOnContent();
[PHP] pobierz, plaintext 

w configu definuję stałe do DB (to dobre miejsce na to , tak?) oraz zaciągam DB.php oraz PageDisplay.php.


Teraz PageDisplay wg mojego założenia miała sprawdzać którą z dwóch podstron wybrano (liste produktów lub dodawanie produktu). W index.php ją wywołuję i przekazuję w parametrach $request oraz $db. Jeszcze nie zabrałem się za jej porządkowanie, więc wiem że idzie zwymiotować jak się na nią patrzy. Tak - zaciąga stałe a nie powinna, ma w sobie $_POST a powinna go dostać w parametrze. Wiem to wszystko już, i zamierzam ją oczyścić. A zacząłem od $db.
Wygląda ona tak:

[PHP] pobierz, plaintext 
namespace nameOne;
 
class PageDisplay
{
    private $request;
    private array $recordsIds;
    private $db;
 
    public function __construct($request, $db)
    {
        $this->db = $db;
        $requestString = explode("?", $request);
        $this->request = $requestString[0];
    }
    public function decideOnContent()
    {
 
        if (!$this->request) {
            return;
        }
 
        switch ($this->request) {
            case PRODUCT_LIST_LANDING . '/':
                $title = 'Product List';
                $pageContent = "ProductList";
                include("ProductList.php");
                if (isset($_POST['mass_delete']) && isset($_POST['product_check'])) {
                    $this->massDelete($_POST['product_check']);
                }
                break;
            case PRODUCT_ADD_LANDING:
                $title = 'Product Add';
                $pageContent = "ProductAdd";
                include("ProductAdd.php");
                if (isset($_POST['save']) && isset($_POST['sku'])) {
                    $productObject = new ProductAdd($db);
                    if (!$productObject->checkExistingSKU($_POST['sku'])) {
                        $productObject->insertNewProduct();
                    }
                    header("Location:http://" . PRODUCT_LIST_LANDING);
                }
                break;
            default:
                $title = 'Product List';
                $pageContent = "ProductList";
                include("ProductList.php");
                break;
 
        }
        $this->displayHeader($title);
        require('views/' . $pageContent . '.view.php');
        $this->displayFooter();
    }
 
    public static function displayHeader($title)
    {
        require_once("views/html/header.html");
    }
 
    public static function displayFooter()
    {
        require_once("views/html/footer.html");
 
    }
 
//nie miałem pomysłu gdzie ta metoda ma być, czy jako oddzielna klasa, czy jak...
    private function massDelete(array $recordsIds)
    {
        $idsToDel = implode(", ", array_map('intval', $recordsIds));
 
        $this->db->query("DELETE FROM `product_property` WHERE `product_property`.`product_id` IN ($idsToDel)");
        $this->db->execute();
 
        $this->db->query("DELETE FROM `product` WHERE `product`.`id` IN ($idsToDel)");
        $this->db->execute();
 
    }
 
}
[PHP] pobierz, plaintext 

Teraz jak widać switch decyduje o tym (poprzez include) która strona ma się wyświetlić. Nie wiem czy chcę się tutaj dzielić ProductAdd.php i ProductList.php bo tam to już jest kompletny p1erdolnik i mi wstyd :|


EDIT:
to moje ProductList.php
Chyba nie wygląda jeszcze, aż tak źle, żeby mnie postawić przed ścianą egzekucyjną.

[PHP] pobierz, plaintext 
namespace nameOne;
 
class ProductList
{
    private $db;
 
    public function __construct($db){
        $this->db = $db;
    }
    public function listProducts()
    {
        $products = $this->getProducts();
 
        return $products;
    }
 
    private function getProducts(): array
    {
        // $db = new DB();
        $this->db->query("SELECT product.id prodId, product.sku, product.name, product.price, property.id, property.type, property.property, property.label, product_property.id, product_property.value FROM product_property INNER JOIN property ON product_property.property_id = property.id INNER JOIN product ON product_property.product_id = product.id ORDER BY product.id DESC
        ");
        $results = $this->db->resultSet();
        return $results;
    }
 
    public function listGroupedProducts(): array
    {
        $products = array();
        foreach ($this->listProducts() as $row) {
 
            $pid = $row['sku'];
            if (!isset($products[$pid])) {
                $products[$pid] = array('name' => $row['name'], 'price' => $row['price'], 'id' => $row['prodId'], 'properties' => array());
            }
            if (!empty($row['value'])) {
                $products[$pid]['properties'][] = array('value' => $row['value'], 'label' => $row['label']);
            }
        }
        return $products;
    }
[PHP] pobierz, plaintext 

Ten post edytował sadistic_son 12.01.2023, 13:13:47