[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 

[sadistic_son]

O, i teraz się zrozumieliśmy. Dzięki za odpowiedź (IMG:style_emoticons/default/smile.gif)

Nastepne pytanie, jeśli mogę.

Zabrałem się za naprawę klasy Product oraz usunięcie jej dziedziczenia np z ProductAdd.
W Product pozbyłem się konstruktora aby posługiwać się setterami i getterami. Wygląda ona teraz tak (kawałek):

[PHP] pobierz, plaintext 
class Product
{
 
    private string $productName;
    private string $sku;
    private float $price;
    private string $productType;
    // public function __construct()
    // {
    //     $this->productName = isset($_POST['name']) ? (string) $_POST['name'] : '';
    //     $this->sku = isset($_POST['sku']) ? (string) $_POST['sku'] : '';
    //     $this->price = isset($_POST['price']) ? (float) $_POST['price'] : 0;
    //     $this->productType = isset($_POST['productType']) ? (string) $_POST['productType'] : '';
    // }
 
    public function setProductName(string $productName): void
    {
        $this->productName = $productName;
    }
 
    public function getProductName(): string
    {
        return $this->productName;
    }
 
//itd dla pozostałych zmiennych
[PHP] pobierz, plaintext 

Teraz chcę się posłużyć tą klasą. Gdy miałem konstruktor to było jasne dla mnie:

[PHP] pobierz, plaintext 
$product = new Product();
$productObject->insertNewProduct($product);
 
 
// gdzie indziej tworzenie metody insertNewProduct($product):
    public function insertNewProduct(Product $product)
    {
        $sku = $product->sku;
        $productName = $product->productName;
        $price = $product->price;
        $selectedType = $product->productType;
[PHP] pobierz, plaintext 

To działało. Teraz natomiast pozbyłem się konstruktora i porobiłem settery i gettery i zastanawiam się jak powinno wyglądac wywołanie metody insertNewProduct przy użyciu Product.

[PHP] pobierz, plaintext 
                        $product = new Product();
                        $product->setProductName($_POST['name']);
                        $product->setSku($_POST['sku']);
                        $product->setPrice($_POST['price']);
                        $product->setProductType($_POST['productType']);
                        $productObject->insertNewProduct($product);
//czy tak?
//dobrze kombinuję?
[PHP] pobierz, plaintext 

No i deklaracja metody:

[PHP] pobierz, plaintext 
    public function insertNewProduct(Product $product)
    {
        $sku = $product->getSku();
[PHP] pobierz, plaintext 

EDIT: Tzn, że to działa to wiem. Moje pytanie to - czy robię to zgodnie ze sztuką, lub chociaż niedaleko jej.

Ten post edytował sadistic_son 13.01.2023, 11:07:11