Forum PHP.pl

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?

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

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

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

No to mnie ukochałeś. Dwa razy to przeczytałem (język ang. to nie problem) i nie łapię (IMG:style_emoticons/default/sad.gif)

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

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"

Te zadanie jest bez sensu, tak sie nie powinno towrzyc kodu jak narzucili (IMG:style_emoticons/default/wink.gif)

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