Как отменить загрузку полей, если клиенту не нужно их получать?

Question

[ant123455432143]

Допустим есть метод, где мы получаем определенные поля, по типу

...
$arProduct['REVIEWS'] = Review::getProductReviews($arProduct['ID']);
$arProduct['QUESTIONS'] = Question::getProductQuestions($arProduct['ID']);
...

Нужно сделать так, чтобы клиент мог указать какие поля не нужно загружать из бд.
То есть, если $fieldsToLoad = [REVIEWS, QUESTIONS], значит их не нужно передавать клиенту. Как это возможно сделать? Есть такой вариант:

$fieldsToLoad = [
        'REVIEWS' => function () use ($arProduct, $reviewsSize) {
            $arProduct['REVIEWS] = Review::getProductReviews($arProduct['ID']);
        },
        'QUESTIONS' => function () use ($arProduct, $questionsSize) {
            $arProduct['QUESTIONS'] = Question::getProductQuestions($arProduct['ID']);
        }
]

    foreach ($fieldsToLoad as $fieldName => $fieldLoader) {
        if (!in_array($fieldName, $notLoadFields)) {
            $fieldLoader();
        }
    }

Но проблема в том, что возможно нужно будет добавить общее условие для каких то полей, как пример:

...
if ($block == 60) {
        $arProduct['REVIEWS'] = Review::getProductReviews($arProduct['ID']);
        $arProduct['QUESTIONS'] = Question::getProductQuestions($arProduct['ID']);
}
...

И тогда получается не очень, так как для каждого элемента массива $fieldsToLoad, которое попадает под это условие, придется писать отдельное условие. Есть ли более лучшее решение вопроса?

Comments

[ThunderCat]

Это шо, битрикс? Или вас битириксоид покусал? Ключи массивов капсом, $fieldsToLoad - поля которые не нужны...

[John Didact]

Я не понимаю, что это за подход вообще, может так и делают, хз… я вообще не делаю нихрена. Но как мне кажется, если делать так, то лучше заранее удалить с $fieldsToLoad элементы, которые клиент использовать не хочет и не париться больше.

[ant123455432143]

ThunderCat, да, это битрикс

[ant123455432143]

John Didact, клиенты разные, каждому свое

[John Didact]

John Didact, … или вообще формировать $fieldsToLoad опираясь на нужность поля клиенту, типа

$shits = array_flip($notLoadFields);
$fieldsToLoad = array();
isset($shits['REVIEWS']) or $fieldToLoad['REVIEWS'] = function (){};
isset($shits['QUESTIONS']) or $fieldToLoad['QUESTIONS'] = function (){};
// …

[John Didact]

John Didact, а вообще, там функции одно и то же делают, значения лишь разные, поэтому можно это всё в отдельную функцию оформить, ну или вообще в цикл

[ant123455432143]

John Didact, выглядит классно, спасибо. Но немного необычно (как и в моем варианте). Хотелось бы еще варианты увидеть, может кто еще предложит

[ant123455432143]

а вообще, там функции одно и то же делают, значения лишь разные, поэтому можно это всё в отдельную функцию оформить, ну или вообще в цикл

John Didact, не, это в примере так, а вообще там разные действия могут происходить, (хотя суть одна)

Answer 1

[Павел Виноградов]

function addAdditionalOption( $var, $var2, $callback )
[
if ( $var == $var2 ) {
$callback();
}
]

foreach ($fieldsToLoad as $fieldName => $fieldLoader) {
        if (!in_array($fieldName, $notLoadFields)) {
           addAdditionalOption( $var, $var2, $fieldLoader );
        }
    }

Comments

[Павел Виноградов]

В том же духе возможны вариации

Answer 2

[Антон Шаманов]

а что тебе мешает создать класс-билдер(или классы) который формирует список требуемых полей и потом вызывает соответствующие методы для получения данных?