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

Question

[Sacred133]

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

Пример кода

if (!isset($_GET['byName']) && !isset($_GET['byPrice']))
{

 
    $arVert = array();
    $arSquare = array();
    $arHor = array();
    $arQwadrat = array();
    $arWithPhoto = array();
    $arNoPhoto = array();
    foreach ($arResult['elements'] as $key => $value) 
        $arFile = array();
        if (isset($value["DEPTH_LEVEL"])) {
            if (count($value['UF_CROPED']) > 0) {
                $arFile = CFile::GetFileArray($value['UF_CROPED'][0]);
                $arWithPhoto[$key] = $value;
            } else {
                $arNoPhoto[$key] = $value;
            }
        } else {
            if (!empty($value["DETAIL_PICTURE"])) {
                $arWithPhoto[$key] = $value;
                $arFile = CFile::GetFileArray($value["DETAIL_PICTURE"]);
            } else {
                $arNoPhoto[$key] = $value;
            }
        }
        
        if (!empty($arFile['WIDTH']) && !empty($arFile['HEIGHT']))
        {
            $k = floatval(intval($arFile['WIDTH']) / intval($arFile['HEIGHT']));
             if ($k > 1.2)
            {
                $arHor[$key] = $value;
            }
           
            elseif ($k < 0.8) {
                $arVert[$key] = $value;
           }
            else
             {
                $arSquare[$key] = $value;
             }
        }
    }
    $arResult['elements'] = array_merge($arVert, $arSquare, $arHor, $arWithPhoto, $arNoPhoto);
}

Answer 1

[Алексей Уколов]

Зачем вам массив $arWithPhoto? В него дублируется то, что попадает в $arHor, $arVert и $arSquare.

Answer 2

[smilingcheater]

Потому что вы сначала складываете $value в $arWithPhoto
$arWithPhoto[$key] = $value;
Потом его же в $arHor/$arVert/$arSquare

а ещё вы забыли открывающуюся фигурную скобку после foreach ($arResult['elements'] as $key => $value)

Comments

[Sacred133]

Спасибо! Подтолкните к правильной последовательности, не доходит, пока что....

[smilingcheater]

Sacred133, как в соседнем ответе написал Алексей Уколов - а зачем вам этот массив вообще? Если в элементе есть хоть какое-то фото - то он в любом случае попадёт в какой-то из трёх $arHor/$arVert/$arSquare.
Так что просто уберите его заполнение, и в принципе его не используйте.

if (isset($value["DEPTH_LEVEL"])) {
            if (count($value['UF_CROPED']) > 0) {
                $arFile = CFile::GetFileArray($value['UF_CROPED'][0]);
            } else {
                $arNoPhoto[$key] = $value;
            }
        } else {
            if (!empty($value["DETAIL_PICTURE"])) {
                $arFile = CFile::GetFileArray($value["DETAIL_PICTURE"]);
            } else {
                $arNoPhoto[$key] = $value;
            }
        }

Или у вас может быть ситуация, что картинка заполнена, но почему-то не получилось узнать её размеры? Тогда перенесите добавление в массив в else-блок:

if (!empty($arFile['WIDTH']) && !empty($arFile['HEIGHT']))
        {
             ....
        } else {
             $arWithPhoto[$key] = $value;
        }

[Алексей Уколов]

Там может быть случай, когда CFile::GetFileArray не нашёл файл, его тоже нужно обработать, но писать, явно, нужно будет не в $arWithPhoto, а в $arNoPhoto.