[PHP]Metoda z parametrami POST czy bez Opcje

[sadistic_son]

Cześć,
Jak lepiej napisać metodę?
Opcja 1:

[PHP] pobierz, plaintext 
public function insertNewProduct(string $sku, string $name, string $price, string $productType)
 {
        $db->query("INSERT INTO `product` (`id`, `name`) VALUES (null, '$name')";);
        $db->bind(':name', $name);
        $db->execute();
//ciąg dalszy tutaj jest zbędny
 }
 
//później wywołanie metody:
$productObject->insertNewProduct($_POST['sku'], $_POST['name'], $_POST['price'], strtolower($_POST['productType']));
[PHP] pobierz, plaintext 

Opcja 2:

[PHP] pobierz, plaintext 
public function insertNewProduct()
 {
        $name = (string) $_POST['name'];
        $selectedType =  (string) $_POST['productType'];
        $sku =  (string) $_POST['sku'];
        $price =  (string) $_POST[price];
        $db->query("INSERT INTO `product` (`id`, `name`) VALUES (null, '$name')";);
        $db->bind(':name', $name);
        $db->execute();
//ciąg dalszy tutaj jest zbędny
 }
 
//później wywołanie metody:
$productObject->insertNewProduct();
[PHP] pobierz, plaintext 

Obstawiam opcję 1 za lepszą. Mam rację?


Halo, ktoś, odpowie?

[viking]

[PHP] pobierz, plaintext 
public function insertNewProduct(Product $product)
i Product zawiera właściwości produktu
[PHP] pobierz, plaintext 

[sadistic_son]

Czy chodzi Ci o to żeby produkt był oddzielna klasą (w tym wypadku abstrakcyjną?) i zawierał pola produktu? I potem odwołuję się do klasy Product? Nie wiem czy dobrze rozumuję.

Ten post edytował sadistic_son 10.01.2023, 13:45:59

[viking]

https://draganatanasov.com/2022/11/12/data-...objects-in-php/

[sadistic_son]

No to mnie ukochałeś. Dwa razy to przeczytałem (język ang. to nie problem) i nie łapię

[viking]

Co ci mogę powiedzieć. Czytaj. To forum nie powinno być źródłem jedynej wiedzy. W 5s tego nie poznasz ale rzucam tematy które mogą się przydać.
I nie, dlaczego Product miałby być klasą abstrakcyjną?

[sadistic_son]

Ok, dzięki za zapodanie tematu, ale to jeszcze nie mój poziom.

Dlaczego klasa Product powinna być abstrakcyjna? No cóż... mam to narzucone:
"Z OOP musisz zademonstrować konstrukcję kodu w sensownych klasach, które rozszerzają (dziedziczą) jedna drugą, więc chcielibyśmy zobaczyć klasę abstrakcyjną dla głównej logiki produktu. Zainteresuj się polimorfizmem"

[com]

Te zadanie jest bez sensu, tak sie nie powinno towrzyc kodu jak narzucili

[vokiel]

Skoro chcą tak utrudniać to pogrupuj produkty w jakieś mniejsze rodzaje i zrób abstrakcyjna klasę główną, np:

[PHP] pobierz, plaintext 
abstract class Product
{
    protected string $sku;
 
    public function getSku(): string
    {
        return $this->sku;
    }
 
    public function setSku(string $sku): self
    {
        $this->sku = $sku;
 
        return $this;
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
class namedProduct extends Product
{
    protected string $name;
 
    public function getName(): string
    {
        return $this->name;
    }
 
    public function setName(string $name): self
    {
        $this->name = $name;
 
        return $this;
    }
}
[PHP] pobierz, plaintext 

Potem jakieś repozytorium dla produktów

[PHP] pobierz, plaintext 
class ProductRepository
{
    public function add(Product $product)
    {
        // Jakiś zapis do bazy
    }
}
 
[PHP] pobierz, plaintext