Можно ли использовать доп. обращения к базе во благо красоты кода?

Question

[Слава]

Задаюсь вопросом. Когда пишешь код, в стиле ООП, но при этом обращаешься к базе нужно ли стремиться к сокращению количества обращений к базе? Например, код:

static funtion ProductExists( $name ) {
    return  Product::where('name', $name)->exists();
}

static function blablabla( $data ) {
   if (Self::ProductExitst( $data['name'] ) {
        $id = Product:: where('name', $data['name'])->first()->id;
        ProductComment::where('product_id', $id ) .....
        ProductDocuments::....
   }
}

Или целесообразнее не делать вызов метода ProductExists, а сразу

$product = Product:: where ( 'name', $name)->first();
if ( $product ) {
             ProductComment::where('product_id', $product->id ) .....
}

В первом случае мы сначала делаем запрос к моделе, за которым последуюет запрос к базе данных.
потом мы по сути делаем снова запрос к базе данных чтоб получить нужные данные и уже использовать их в коде.

Во втором случае мы сразу получаем данные (если они есть) и если они есть, то используем их.

Так нужен читаемый стиль ООП для подобных случаев? он улучшает читаемость, но увеличивает число обращений к базе.

Ну это самый простой может не доконца раскрывающий вопрос случай. Я хочу узнать как православнее делать.

Подскажите что да как

Comments

[Barmunk]

Как насчет firstOrFail() плюс with() ?

[Слава]

Barmunk, ну это уже laravel style. я же без тэга Laravel хотел посоветоваться. суть в том делать лишнее обращение к базе ради проверки создав отдельный метод для этого или нет.

[Слава]

Barmunk, просто в ООП принято все проверки реализовывать через методы. а это +1 обращение к базе. вот я и задался вопросом можно так делать или нет? как к этому отнесется нанимающий на работу программист.

типа

if ( !$this->hasPrice( $product_id ) {
    $this->refreshPrice ( $product_id )
}

[Barmunk]

Слава, к ООП вопрос имеет мало смысла. Тут больше вопрос к оптимизации sql.

Чтобы тебе было легче понять разницу между первым и вторым вариантом, напиши для себя эти запросы в сыром формате, к примеру увидишь что exists отличается от остальных.

Далее получение продукта по айди, само по себе уже проверка на наличие продукта и тогда зачем нужен целый метод exists, который вроде бы делает тоже самое, но не совсем

[Слава]

Barmunk, ну он влюбом случае увеличивает число операций. а в ООП код должен быть красивым и состоять только из сущностей. Или я ошибаюсь?

[Barmunk]

Слава, увеличивает, а чем меньше запросов к базе тем лучше.

Answer 1

[Александр]

А при чем тут вообще ООП? ООП не говорит нам как часто нужно делать запросы к базе, какие запросы писать. ООП предлагает работать с объектами. При чем тут читаемость вообще? Я могу прочитать как несколько вызовов методов обращения к бд, так и один.
Сокращение колл-ва обращений к базе - не ООПешный подход. Это подход оптимизации производительности. Если у вас есть 1 запрос, ваш код будет работать быстрее, чем если бы запросов было 2. Читаемость кода улучшает грамотное проектирование методов - разделение ответственности, правильные имена и многое другое.
В данном случае нет вообще никакого смысла делать несколько одинаковых запросов. В вашем вы скорее всего просто получите ошибку, если элемент в базе не найден. Если нет, то проверки
if ( $product ) ...
хватит сполна.

Answer 2

[Александр Скуснов]

Способы повышения скорости:
1) Хранимые процедуры. (не помню, если такой метод как включение параметров - уже давно не прикасался к БД). Число процедур можно сделать любым.
2) Последовательность объединения таблиц (Join), когда сначала надо применять самые эффективные фильтры (сокращающие выборки)