Как ускорить выполнение этого скрипта?

Question

[KOPC1886]

Всем привет!

Подскажите пожалуйста, как можно оптимизировать этот кусок кода?
Выполняется не очень быстро.

foreach($categories as $i => $objCategory)
            {
                $jsonData[$i]['id'] = $objCategory->id;
                $jsonData[$i]['name'] = $objCategory->name;
                $jsonData[$i]['image'] = $objCategory->image;
                $jsonData[$i]['editors'] = $objCategory->editors;
                $jsonData[$i]['can_edit'] = $objCategory->canEdit;

                $cnt = 0;
                $jsonData[$i]['services'] = array();
                foreach($services as $k => $objService)
                {
                    if($objService->mainCategory == $objCategory->id || in_array($objCategory->id, $objService->additionalCategories))
                    {
                        if(!empty($objCategory->sortingServices))
                        {
                            foreach($objCategory->sortingServices as $index => $serviceId)
                            {
                                if($objService->id == $serviceId)
                                {
                                    $cnt = $index;
                                    break;
                                }
                            }
                        }

                        $jsonData[$i]['services'][$cnt]['id'] = $objService->id;
                        $jsonData[$i]['services'][$cnt]['name'] = $objService->name;
                        $jsonData[$i]['services'][$cnt]['favorite'] = $objService->favorite;
                        $jsonData[$i]['services'][$cnt]['main_category'] = $objService->mainCategory;
                        $jsonData[$i]['services'][$cnt]['tags'] = $objService->tags;
                        $jsonData[$i]['services'][$cnt]['can_read'] = $objService->canRead;
                        $jsonData[$i]['services'][$cnt]['can_edit'] = $objService->canEdit;
                        $jsonData[$i]['services'][$cnt]['can_order'] = $objService->canOrder;

                        ++$cnt;
                    }
                }
                ksort($jsonData[$i]['services']);
            }

Comments

[KOPC1886]

Так как всё же ускорить этот скрипт?

Answer 1

[Евгений]

Надо в начало этого кода поместить die()

Answer 2

[Sergey Lerg]

Для начала замените везде $jsonData[$i] на переменную $jsonData_i
$jsonData_i = $jsonData[$i] в начале цикла.

Вместо блока кода с $jsonData[$i]['services'][$cnt] сделайте одну строчку

$services[$cnt] = [
  'id' => $objService->id,
  'name' => $objService->name,
  'favorite' => $objService->favorite,
  'main_category' => $objService->mainCategory,
  'tags' => $objService->tags,
  'can_read' => $objService->canRead,
  'can_edit' => $objService->canEdit,
  'can_order' => $objService->canOrder
];

И после цикла
$jsonData_i['services'] = $services;

Это немного ускорит цикл, но по хорошему нужно прогнать код через profiler, он скажет где именно затыки.
Возможно много времени теряется в функции in_array(), в этом случае additionalCategories нужно сделать ассоциативным массивом.

Просто for цикл вместо foreach тоже должен быть чуть быстрее.

Comments

[KOPC1886]

Sergey Lerg долго выполняется второй foreach

[KOPC1886]

Sergey Lerg у меня также используется шаблон проектирования Итератор.

[Sergey Lerg]

KOPC1886: мне кажется вы можете либо полностью выбросить сортировку, либо сделать её после генерации всего массива. То есть вынести из внутреннего цикла.

[FanatPHP]

Жесть. Хрестоматийный набор бессмысленных "советов" из интернета. Во всем ответе - ровно одна осмысленная строчка. А зато выглядит как солидно - с кодом, с примерами. И все без малейшей пользы. Не стыдно такую туфту гнать?

[Sergey Lerg]

FanatPHP: Ну я-то знаю, что придёт всемогущий FanatPHP и поправит меня где нужно. Go for it!

[FanatPHP]

Пора уже научиться вытирать сопли самостоятельно

Answer 3

[Arman]

1. На каждую категорию вы полностью прогоняете все сервисы, возможно их можно подготовить до категорий в виде ассоц массив. Или хотя бы unset() уже обработанных сервисов, если они не повторяются.
2. Используете свойства объекта, они существуют или доступ идет через магические методы? Может лучше явно создать методы ->get**() и получать данные через них? Магические методы хорошо тратят время/ресурсы.
3. Может результат просто положить в кэш?

Comments

[KOPC1886]

Arik
2 - Использую свойства объекта, без магических методов
3 - Закешировать не могу. По иной причине.

Answer 4

[asd111]

Переходите на hhvm или возьмите php поновее. Либо можно написать модуль для php на С++. С точки зрения логики и кода у вас все норм(на первый взгляд).
Переход с php 5.3 на 5.4 дает сам по себе прирост производительности примерно 15%. А переход на hhvm может дать ещё больший прирост именно на таких задачах - главное в функцию обернуть, тогда это будет скомпилировано и скорость будет соответствующая.

php медленно обрабатывает данные

Comments

[KOPC1886]

asd111 Неужели здесь никак нельзя ускорить работу скрипта? Как то переписать его?