Хороша ли архитектура моего кода? Информация цепляется по API?

Question

[Pupochkin123]

подскажите пожалуйста, насколько данный код хорошо организован по принципам инкапсуляции и SOLID? Как его можно улучшить?

<?php

Class Product
{
    public int $product_id;

	public function __construct($product_id)
	{
		$this->product_id = $product_id;
	}

	public function getDetail()
	{
		// Здесь запрос к API который возвращает JSON с информацией о товаре.
	}

	public function getTitle()
	{
		$detail = $this->getDetail();

		return $detail['data']['title'];
	}

	public function getPrice()
	{
		$detail = $this->getDetail();

		return $detail['data']['price'];
	}
}

Comments

[Dmitry Bay]

Прикалываешься?

1) У тебя в каждом геттере запрашивается апи метод. надо добавить $_detail, и если не пуста, то дергать ее.
2)

данный код хорошо организован по принципам инкапсуляции и SOLID

Тут нет ни одного, ни другого.

[Дмитрий]

Как бы тут нет особой архитектуры. Один класс это один класс. Но сильное подозрение что у вас будет апи дергаться каждый раз когда кто нибудь захочет получить price или title

[Pupochkin123]

Спасибо за ответы, можете пожалуйста в прикрепить пример как стоит правильно его организовать? Спасибо ещё раз!

Answer 1

[Дмитрий]

Первое, вы при каждом получении свойства будете лезть в апи, нужно делать подобие кэша

private function getDetail()
 {
    if (!$this->detailCache) { 
        // Здесь запрос к API который возвращает JSON с информацией о товаре.
       $this->detailCache = $data;
    }
 
    return $this->detailCache;
 }

Второе, все свойства должны быть приватными или защищенными
Третье, метод который лезет в апи не должен быть публичным

Answer 2

[Vitsliputsli]

Все public, все доступно извне, нет сокрытия.
Прям сам объект куда-то лезет по api, забирает данные, которые сам и предоставляет, обрабатывает ответ, явное нарушение SRP.

Answer 3

[Rsa97]

Я бы сделал как-то так. Но не претендую на истину в последней инстанции.

spoiler

class API
{
    public function getProduct(int $productId): Product
    {
        $product = $this->request(...); // Получение данных из API
        return Product::from($product)
    {
}

class Product
{
    private int $id;
    private string $title;
    // прочие свойства

    public function getId(): int
    {
        return $this->id;
    }
    
    public function getTitle(): int
    {
        return $this->title;
    }

    // прочие геттеры

    // сеттеры, если нужны
 
    public static function from(object $source): self
    {
        $product = new static();
        $product->id = $source->id;
        $product->title = $source->title;
        // заполнение свойств
        return $product;
    }
}

...
$api = new API($login, $password);
$product = $api->getProduct(325);

Хотя, для современных версий PHP можно описать преобразование из json с помощью атрибутов и сделать трейт с функцией from, которая будет парсить полученный из API объект согласно этим атрибутам. Плюс аналогичный трейт, реализующий JsonSerializable::jsonSerialize по атрибутам.