Как оптимизировать проверку циклами?

Question

[Space]

Всем привет. Необходимо проверить каждый материал, что он совпадает с моей фильтрацией. Всего фильтрации 3.
Всё до чего додумался это foreach, но не понимаю почему если забиваю все массивы (фильтры) до 10 значений, то работает быстро. А если первые два пустые, а третий полный, то просто бесконечно грузится страница.
Кто что подскажет?

// Обрабатываем материалы
    public function materials($categories, $materials, $tagsMap, $tagsFitlerLevel, $tagsFitlerFood, $tagsFitlerTreatment) {
        $arrayMaterials = array();
        foreach ($materials as $value) {
            if(in_array($value->catid, $categories)){
                foreach ($tagsMap as $tagLevel) {
                   if(in_array($tagLevel->tag_id, $tagsFitlerLevel) && $tagLevel->content_item_id == $value->id || empty($tagsFitlerLevel) || $tagsFitlerLevel[0] == 'null'){
                       
                        foreach ($tagsMap as $tagFood) {
                            if(in_array($tagFood->tag_id, $tagsFitlerFood) && $tagFood->content_item_id == $value->id || empty($tagsFitlerFood) || $tagsFitlerFood[0] == 'null'){

                                foreach ($tagsMap as $tagTreatment) {
                                    if(in_array($tagTreatment->tag_id, $tagsFitlerTreatment) && $tagTreatment->content_item_id == $value->id || empty($tagsFitlerTreatment) || $tagsFitlerTreatment[0] == 'null'){
                                        $arrayMaterial = array();
                                        array_push($arrayMaterial, $value->id);
                                        array_push($arrayMaterial, $value->title);
                                        array_push($arrayMaterial, $value->catid);
                                        array_push($arrayMaterials, $arrayMaterial);
                                        $arrayMaterial = array();
                                        break 3;
                                    }
                               }
                            }
                        }
                    }
                }
            }
        }
        return $arrayMaterials;
    }

Comments

[Ди Ма]

Нужно выбрать из всего списка определенные значения (по настройкам фильтра)?

[latteo]

В 3х вложенных foreach у вас итерируется один и тот же массив $tagsMap?
Почему вы не разбили это на 3 или более функций по смыслу?

[Артем]

Прямая дорога на govnokod.ru с таким кодом

[Space]

Артем: подскажите как сделать по другому?

[Space]

manwe_ru: а каким способом?

[Space]

latteo: но ведь всё равно будет 3 цикла. 3 раза обращаться в функцию. Или что имеете ввиду?

[Space]

Ди Ма: Да.

[Ди Ма]

Cosmos: Почему вы не используйте базу данных?

[latteo]

Cosmos: да, будет 3 цикла, в трёх функциях, но будет понятнее, что происходит. И может даже придёт мысль как это всё оптимизировать и ускорить. Сейчас повтыкав минут 5 в ваш код, несмотря на нормальные названия переменных, я так и не понял что он делает и каким образом. При рефакторинге я такой код, написанный не мной, старался обходить стороной, либо это занимало не один час рабочего времени.

PS Поищите популярные статьи/видео по принципам SOLID

[Space]

Ди Ма: я использую.

[Space]

latteo: спасибо. Поищу. Нашёл способ без 3х циклов. Совсем засиделся...

Answer 1

[Space]

Решение. Не нужно 3 цикла. Зачем я их делал сам не пойму. И проверку вынес на уровень выше.

public function materials($categories, $materials, $tagsMap, $tagsFitlerLevel, $tagsFitlerFood, $tagsFitlerTreatment) {

        $arrayMaterials = array();
        $filterLevel = 0;
        $filterFood = 0;
        $filterTreatment = 0;

        foreach ($materials as $value) {
            if(in_array($value->catid, $categories)){
                foreach ($tagsMap as $tag) {

                    if($tag->content_item_id == $value->id):
                        if(empty($tagsFitlerLevel) || $tagsFitlerLevel[0] == 'null' || in_array($tag->tag_id, $tagsFitlerLevel)){
                            $filterLevel = 1;
                        }
                        if(empty($tagsFitlerFood) || $tagsFitlerFood[0] == 'null' || in_array($tag->tag_id, $tagsFitlerFood)){
                            $filterFood = 1;
                        }
                        if(empty($tagsFitlerTreatment) || $tagsFitlerTreatment[0] == 'null' || in_array($tag->tag_id, $tagsFitlerTreatment)){
                            $filterTreatment = 1;
                        }

                        if($filterLevel == 1 && $filterFood == 1 && $filterTreatment == 1){
                            $arrayMaterial = array();
                            array_push($arrayMaterial, $value->id);
                            array_push($arrayMaterial, $value->title);
                            array_push($arrayMaterial, $value->catid);
                            array_push($arrayMaterials, $arrayMaterial);
                            $arrayMaterial = array();
                        }
                    endif;
                    
                }
                $filterLevel = 0;
                $filterFood = 0;
                $filterTreatment = 0;
            }
        }
        return $arrayMaterials;
    }

Comments

[SharuPoNemnogu]

public function materials(array $categories, array $materials, array $tagsMap, array $tagsFitlerLevel, array $tagsFitlerFood, array $tagsFitlerTreatment) {
    $arrayMaterials = [];
    $categories = array_flip($categories);
    $tagsFitlerLevel = array_flip($tagsFitlerLevel);
    $tagsFitlerFood = array_flip($tagsFitlerFood);
    $tagsFitlerTreatment = array_flip($tagsFitlerTreatment);

    foreach ($materials as $value) {
        if (isset($categories[$value->catid])) {
            foreach ($tagsMap as $tag) {
                if ($tag->content_item_id == $value->id && isset($tagsFitlerLevel[$tag->tag_id]) && isset($tagsFitlerFood[$tag->tag_id]) && isset($tagsFitlerTreatment[$tag->tag_id])) {
                    $arrayMaterials[] = [
                        $value->id,
                        $value->title,
                        $value->catid
                    ];

                    break;
                }
            }
        }
    }

    return $arrayMaterials;
}