Как сделать лимит выполнения цикла for?

Question

[crack_user]

Задача - если юзер отправил 5 или больше фоток, мы грузим лишь первые 4 и отправляем также первые 4.
Мой код -

if ($data->object->message->attachments[$i]->type == 'photo' && $colls >= 5){ // Если массив равен type = photo, а кол-во photo больше или равно 5
        $colls ++;
        for($i = 0; $i != $count; $i++) { // При каждом выполнение - к $i добавляем +1
            if($i >= 5) { // При достижение выполнений больше 4 - выполняем код и останавливаем
                $type = $data->object->message->attachments[$i]->photo->id;
                $sizes = $data->object->message->attachments[$i]->photo->sizes;
                $count_size = count($sizes);
                for($is = 0; $is != $count_size; $is ++){// Код на получение максимального размера с фотки
                    if($data->object->message->attachments[$i]->photo->sizes[$is]->height >= $h){
                        $h = $data->object->message->attachments[$i]->photo->sizes[$is]->height;
                        $m = $is;
                    }
                    if($data->object->message->attachments[$i]->photo->sizes[$is]->width >= $w){
                        $w = $data->object->message->attachments[$i]->photo->sizes[$is]->width;
                        $m = $is;
                    }
                }
                break;
            }
        }
    }

Comments

[Slava Rozhnev]

В чем проблема?

[FanatPHP]

Я думаю что в лишней вложенности.

Но вообще у меня дикий когнитивный диссонанс, когда я виду все эти цепочки method chaingng-a написанные человеком, который не в состоянии решить алгоритмическую задачу в одно действие

Да какую блин алгоритмическую - арифметическую, на материале 2 класса начальной школы!

[v3shin]

define('MAXIMUM_PHOTOGRAFIY', 4);
...
$count = min($count, MAXIMUM_PHOTOGRAFIY);
for($i = 0; $i < $count; $i++) {}

Answer 1

[Adamos]

Прописать в модели Message метод limited_attachments и использовать его во всех подобных случаях.

Comments

[crack_user]

Извиняюсь, я Вас чутка не понял, в методе message.send прописать limited_attachments? Или где?

[Adamos]

crack_user, нет, в самом классе Message, в котором реализованы все эти методы.

Answer 2

[Сергей delphinpro]

for($i = 0;      // выполнять от нуля, до тех пор
    $i < $count, // пока $i меньше общего кол-ва
    $i < 5;      // или меньше пяти
    $i++) {