Как протестировать производительность функций начинающему php-разработчику?

Question

[AgentSmith72]

Здравствуйте, недавно при показе своего MVC фреймворка опытному разработчику, он сказал, что я зря гоняю переменную в аргументах, а лучше обращаться к ней, как к свойству класса.

Возникла дилемма, как лучше создавать архитектуру MVC:

Получать один раз эти значение и далее, без дублирования кода передавать их аргументами в места, где они требуются?

или

Не передавать эти значения, а обращаться к свойству класса из функций.

Каким способом самому протестировать архитектуру, чтобы точно видеть разницу в использовании системных ресурсов, как в первом случае, так и во втором?

Благодарю за уделённое время.

Comments

[Дмитрий]

https://m.youtube.com/playlist?list=PLE20id3DjfFni...

[AgentSmith72]

Дмитрий, Спасибо, PSR фреймворк на основе стандартов, обязательно гляну..

Answer 1

[AgentSmith72]

Скрипт для тестирования времени и затрат памяти во...

Answer 2

[ThunderCat]

Как абсолютно верно заметил Роман Юрьевич Ипатьев, проблема не в вызове аргументов, а в абсолютно нелогичном подходе к архитектуре, которая тут просто швах...

Просто на будущее - архитектура в разы важнее скорости исполнения, так как ресурсов всегда можно накинуть, а с кривой архитектурой ни поддержка, ни нормальная работа невозможна. Вы экономите 2 байта на каких-то мелких манипуляциях через Ж, при этом в коде совершенно невозможно разобраться, а любая модификация функционала может уронить весь проект или заставить менять код в 40 местах.

Файл RouteController.php обрабатывает URL

уже плохо, контроллер не должен знать что-то про урл и прочие переменные извне. Для этого есть роутер и реквест.

и делает вывод о типе контроллера (гость, авторизованный пользователь или администратор).

Как тип контроллера соотносится с ролью пользователя?

Создаёт объект этого контроллера и отправляет в него аргументами (str)имя контроллера и (str) полученный URL.

У вас же уже вызван контроллер, RouteController.php, либо он не контроллер, либо зачем тогда снова контроллер создавать? И зачем ему урл?

Эти аргументы попадают в родительский контроллер файла Controller.php и оттуда далее используются в условном ветвлении и передаются в соответствующие функции, где дальше из этих функций опять передаются в нужные функции.

Сложнааа, слоожжнаа (с) Карина. Очень запутанно и очень странно работает ваша творческая мысль.

На самом деле все проще и сложнее одновременно. Понимаю что на основе разрозненных "слышал звон" вы попытались создать что-то свое, в надежде создать лучшее светлое будущее для всех, но, короче, ничего не получилось.

Про то как оно примерно должно работать рассказывать наверное не стоит, чтобы не смутить ваш творческий поиск готовыми решениями. Если будет интересно - Дмитрий скинул вам замечательный курс от Елисеева, прям рекомендую.

Comments

[AgentSmith72]

уже плохо, контроллер не должен знать что-то про урл и прочие переменные извне. Для этого есть роутер и реквест.

Вообще-то это правило работает только касательно Модели и Вида, где они не знают друг о друге.

Как тип контроллера соотносится с ролью пользователя?

Так что создаётся объект к примеру AdminController, а аргументы попадают в родительский контроллер типового контроллера. Типовой контроллер формирует имя метода у типовой модели AdminModel.

У вас же уже вызван контроллер, RouteController.php, либо он не контроллер, либо зачем тогда снова контроллер создавать? И зачем ему урл?

($controller == 'main')? $i = 0 : $i = 2;
$check = 'get' . ucfirst(explode('/', $url)[$i]);  // URL нужен чтобы не хранить инфо в свойствах класса

if(method_exists($this->model, $check))
else // 404 error

Вы удивитесь но мой MVC фреймворк создан и прекрасно работает., он намного логичнее старых версий, и вызывает гнев у опытных разработчиков, так как требует меньше задействованной памяти.

А URL в RouteController.php нужен, так как автобусы едут прямо по карте (с) Яндекс.

[ThunderCat]

AgentSmith72,

Вы удивитесь но мой MVC фреймворк создан и прекрасно работает., он намного логичнее старых версий, и вызывает гнев у опытных разработчиков, так как требует меньше задействованной памяти.

О как... Ясненко, понятненько... Я думаю он вызывает у них икоту и дергающийся глаз, и вовсе не из-за задействованной памяти. Кстати, советую как-либо сохранить сей тред, дабы лет этак через 3-5 посмотреть на него, и ощутить ностальгию и жесткий приступ шейма, переходящего в дикий ржачь. Ну, в случае если вы продолжите заниматься разработкой, и попутно вас не придушит какой-нибудь особо нервный коллега, уличив вас в жестком издевательстве над здравым смыслом...

[ThunderCat]

AgentSmith72,

Вообще-то это правило работает только касательно Модели и Вида, где они не знают друг о друге.

Вас кто-то нае... жестоко обманул, так как инкапсуляция работает не так...

Так что создаётся объект к примеру AdminController, а аргументы попадают в родительский контроллер типового контроллера. Типовой контроллер формирует имя метода у типовой модели AdminModel.

Это все равно что в такси привозить машины на машине, потому что мы не знаем сколько будет пассажиров, и когда приедем - ага, тут 1 чел, сгружаем ниссан микра, будет норм, и едем на микре, а сзади едет автовоз, чтобы в конце маршрута погрузить микру... Короче гениально... Причем (совпадение? Не думаю...) в нормальных фреймворках есть такая штука - диспетчер, которого вы в своем "таксопарке" уволили, и теперь возите весь парк с собой.

Все остальное - примерно в том же духе.

[AgentSmith72]

ThunderCat, URL формируется в зависимости от типа пользователя, если это авторизованный пользователь, то URL принимает вид site.ru/user/Nikolai-Drozdov/office

Таким образом, если мы здесь не находим типовой контроллер:
($controller == 'main')? $i = 0 : $i = 2;
то искать метод типовой модели следует после имени авторизованного пользователя.

Что вас смущает конкретно?

[AgentSmith72]

ThunderCat,

Это все равно что в такси привозить машины на машине, потому что мы не знаем сколько будет пассажиров,

ошибаетесь, мы сразу из RouteController попадаем в типовой контроллер, где формируется имя нужной машины, и она вызывается объектом, созданным в конструкторе родительского контроллера типовых контроллеров.

$method собирается как конкретная машина для конкретного типа запроса

return $content = [
      'posts' => $this->model->$method($url)
];

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

[ThunderCat]

AgentSmith72,

Что вас смущает конкретно?

вообще все. Начиная с того что урл парсится каким-то куском кода с непонятной логикой, подходящей только под текущий вариант использования, и заканчивая тем что все работает только собранное в кучу, слепленное скотчем и прибитое гвоздями к потолку. Если я захочу изменить параметры, или использовать другую последовательность аргументов, или у меня появятся языковые версии - все, код можно выкинуть в помойку и писать новый движок.

А так- вопрос вообще о чем? Как экономить байты? Просто не пишите код, во первых сразу ахрененная экономия памяти, примерно 0 байт будет занимать, во вторых экономим тонны невыпитой валерианы и несъеденного корвалола от ваших коллег, ну и прям по заветам Доктора Дью - "как сделать очень полезное ни**чего"...

Если серьезно - понимание что правильная структура важнее экономии придет, не сразу, но придет, а пока - успехов в учебе.

[AgentSmith72]

ThunderCat, Какая другая последовательность может быть, если в адресе
site.ru/user/Nikolai-Drozdov/edit/3/Sobaka
при парсинге URL второй индекс либо существующий метод либо ошибка 404?

все работает только собранное в кучу, слепленное скотчем и прибитое гвоздями к потолку.

Это называется полиморфизм, и занимает сиё несколько строк, но обрабатывает все возможные варианты страниц.

Если я захочу изменить параметры

Парсинг URL проверяет наличие контроллера, и имя метода которое идёт либо сразу после домена, либо после имени авторизованного пользователя. Другие параметры URL не связаны с логикой и могут быть любыми.

Если вам и надо будет что-то менять, то это не отобразится на логике, так как она безупречна.

[AgentSmith72]

ThunderCat, Вопрос был какие инструменты опытные разработчики используют, при проектировании архитектуры, чтобы эффективно расходовать системные ресурсы.

Пока чувствовал себя Галилеем перед стариками из Ватикана, обвинявшими меня в ереси, нашёл функцию getrusage(), которая показывает мне количество памяти и потраченное на обработку логики время.

[ThunderCat]

AgentSmith72, Ваш RouteController подозрительно похож на смесь бульдога с носорогом, то есть роутера и диспетчера, который почему то у вас все равно контроллер.

Смотрите, если его разделить на 2 сущности, например роутер, который будет сопоставлять паттерн урл с соответствующими контроллерами и экшенами, а её мы передадим другой сущности, которая по нему уже создаст нужный Контроллер и вызовет нужный экшн... Хмм, похоже на нормальный мвц...

[AgentSmith72]

ThunderCat, ваша бурная фантазия рисует вам некого мутанта

Дверь в здание:

namespace application\core;

class RouteController {

    public function __construct() {

        $url = trim($_SERVER['REQUEST_URI'], '/');
        $check = explode('/', $url)[0];

        $controller = 'main';

		    if($check == 'admin' or $check == 'user') $controller = $check;

		    $path = '\application\controllers\\' . ucfirst($controller) . 'Controller';

		    $object = new $path($controller, $url);
    }
}

И по моей логике, типовой контроллер - дверь в конкретное помещение.

[ThunderCat]

AgentSmith72, ладно, товарищ Галилео, я вам сдам секркты инквизиции, и будет вам щасие...

шоб вражины не спалили...

memory_get_usage()
memory_get_peak_usage()
Тока никому, а то враги повсюду...

Answer 3

[Ипатьев]

Никак.
Никакой разницы в использовании системных ресурсов для разных способов вызова функций нет.

Оценивать тот или иной подход следует с точки зрения архитектуры, а не воображаемой разницы в использовании системных ресурсов.
Но для этого надо задать другой вопрос, или по крайней мере отредактировать этот, убрав из него все фантазии про системные ресурсы.

Comments

[AgentSmith72]

Пример простой, но речь идёт про разницу между аргументами и обращением к свойствам:

В первом случае, после окончания работы функции, значения переменных удаляются из памяти.
Во втором случае, мы постоянно тратим память на хранение значений.

[Ипатьев]

Оооо, да тут всё серьёзно.
то есть мы тут на тропе войны, и ищем аргумент для спора с опытным разработчиком
чтобы кинуть ему все 300 байт разницы в его самодовольную ухмылку

[AgentSmith72]

Роман Юрьевич Ипатьев, Мой вопрос подразумевает: как тестировать функциональную реализацию, чтобы в зависимости от затрат системных ресурсов использовать либо вызов чистильщика значений переменных-аргументов либо обращаться к памяти.

Так как я планирую выстраивать архитектуру отталкиваясь не от удобочитаемости кода, а от эффективности использования системных ресурсов.

[Ипатьев]

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

[AgentSmith72]

Роман Юрьевич Ипатьев, опытный разработчик мне и сказал, что так как программисты народ ленивый, то ему проще обращаться к памяти, легко узнавая, что там находится, чем прослеживать роутинг переменных с начала их пути.

По мне так это уже не KISS, а лёгкая эротика.

[Ипатьев]

AgentSmith72, если вас интересует производительность на уровне регистров процессора и отдельных байт памяти, то РНР - не ваш выбор.
Типичное приложение РНР работает с хранилищами данных к которым обычно ходит по сети. Память сразу выделяет большими кусками. Живет от 1 сотой до единиц секунд. таких обращений к функциям - и в самом РНР, и в хранилищах, к которым он обращается, и в веб сервере - тысячи.
Разница с вашими страданиями по обращению к памяти в несколько порядков. Если заменять разницу в обработке запроса целиком, то увидеть её будет невозможно. В лучшем случае - на специально придуманном тестовом стенде, который измеряет вызовы сферически функций в вакууме

[AgentSmith72]

Роман Юрьевич Ипатьев, Директива в файле php.ini

memory_limit = 128M // Задаёт максимальный объем памяти в байтах, который разрешается использовать скрипту.

Нашёл функцию getrusage(), которая делает всё что требуется.

$data = getrusage();

echo $data["ru_oublock"]; // количество операций вывода блока
echo $data["ru_inblock"]; // количество операций приёма блока
echo $data["ru_msgsnd"]; // количество отправленных сообщений IPC
echo $data["ru_msgrcv"]; // количество принятых сообщений IPC
echo $data["ru_maxrss"]; // наибольший размер установленной резидентной памяти
echo $data["ru_ixrss"]; // суммарное значение размера разделяемой памяти
echo $data["ru_idrss"]; // суммарное значение размера неразделяемых данных
echo $data["ru_minflt"]; // кол. исправленных страниц (лёгкая ошибка страницы)
echo $data["ru_majflt"]; // кол.ошибочных страниц (тяжёлая ошибка страницы)
echo $data["ru_nsignals"]; // количество полученных сигналов
echo $data["ru_nvcsw"]; // количество согласованных переключений контекста
echo $data["ru_nivcsw"]; // количество несогласованных переключений контекста
echo $data["ru_nswap"]; // количество свопов
echo $data["ru_utime.tv_usec"]; // время на задачи пользователя (user time) (микросекунды)
echo $data["ru_utime.tv_sec"]; // время на задачи пользователя (user time) (секунды)
echo $data["ru_stime.tv_usec"]; // время на системные задачи (system time) (микросекунды)

Answer 4

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

https://xdebug.org/docs/trace

Comments

[AgentSmith72]

Спасибо, xDebug может показать узкие места.

попробуйте функцию getrusage(), которая показывает время отработки скрипта в МС, а также затраты памяти.

$data = getrusage();

echo $data["ru_oublock"]; // количество операций вывода блока
echo $data["ru_inblock"]; // количество операций приёма блока
echo $data["ru_msgsnd"]; // количество отправленных сообщений IPC
echo $data["ru_msgrcv"]; // количество принятых сообщений IPC
echo $data["ru_maxrss"]; // наибольший размер установленной резидентной памяти
echo $data["ru_ixrss"]; // суммарное значение размера разделяемой памяти
echo $data["ru_idrss"]; // суммарное значение размера неразделяемых данных
echo $data["ru_minflt"]; // кол. исправленных страниц (лёгкая ошибка страницы)
echo $data["ru_majflt"]; // кол.ошибочных страниц (тяжёлая ошибка страницы)
echo $data["ru_nsignals"]; // количество полученных сигналов
echo $data["ru_nvcsw"]; // количество согласованных переключений контекста
echo $data["ru_nivcsw"]; // количество несогласованных переключений контекста
echo $data["ru_nswap"]; // количество свопов
echo $data["ru_utime.tv_usec"]; // время на задачи пользователя (user time) (микросекунды)
echo $data["ru_utime.tv_sec"]; // время на задачи пользователя (user time) (секунды)
echo $data["ru_stime.tv_usec"]; // время на системные задачи (system time) (микросекунды)

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

AgentSmith72, эээ.. судя по коду я бы на твоем месте вначале php доучил. без обид

Answer 5

[Vitsliputsli]

Этот разработчик имел ввиду, что вы "гоняете через аргументы" является свойством класса, и таковым и надо его сделать. Нет смысла это задавать извне, если это неизменяемая часть класса.
А потому, скорее всего никакой речи о "борьбе за ресурсы" не было и в помине.
Если вы предлагаете передачу объекта через аргумент функции заменить на создание еще одного объекта, значит вы плохо представляете, как и на что выделяется память. Оставьте это пока, разберитесь с зависимостями в коде.
Вообще странная вещь для разработчика описывать код словами, там все так плохо, что из кода вообще ничего не понятно?

Comments

[AgentSmith72]

Вообще странная вещь для разработчика описывать код словами

..причём тут код, вопрос о разнице в подходах, измерении скорости скрипта и затрат памяти.

Считаю крайне неэтичной, привычку использовать память, чтобы хранить там всякую дичь и постоянно использовать её, хотя можно обойтись без этого.

Если вы предлагаете передачу объекта через аргумент функции заменить на создание еще одного объекта, значит вы плохо представляете, как и на что выделяется память.

Хотите оценить мой код, изучите мои комментарии с ThunderCat, там внизу присутствует код RouteController.

..написали бы как и для чего выделяется память, было бы полезней намного..

[Vitsliputsli]

AgentSmith72,

..причём тут код, вопрос о разнице в подходах, измерении скорости скрипта и затрат памяти.

Когда обсуждаем код, он всегда причем. Какие еще подходы в измерении скорости и затрат памяти? Кроме как засечь время и посмотреть расход памяти.

Считаю крайне неэтичной, привычку использовать память, чтобы хранить там всякую дичь и постоянно использовать её, хотя можно обойтись без этого.

Логично, если все работает без дичи, то она и не нужна.

Хотите оценить мой код, изучите мои комментарии с ThunderCat, там внизу присутствует код RouteController.

Речь не про оценку, а про то что по коду гораздо проще понять что вы имеете ввиду, чем по словесному описанию. И уж тем более не проводить розыск кода.

..написали бы как и для чего выделяется память, было бы полезней намного..

Да не вопрос. Конкретно в этом случае, передача аргументом функции не будет практически ничего стоить, т.к. передача идет по ссылке, а объект в любом случае должен быть создан. А вот создание дополнительных объектов рефлексии - это затраты как памяти, так и времени на создание объектов.
Хотя эти затраты достаточно малы, и никому не интересны, если только вы не пишите высокопроизводительный код, а проблемный кусок повторяется бесконечное число раз.

[AgentSmith72]

Vitsliputsli,

Знакома вам функция?

$data = getrusage();

echo $data["ru_oublock"]; // количество операций вывода блока
echo $data["ru_inblock"]; // количество операций приёма блока
echo $data["ru_msgsnd"]; // количество отправленных сообщений IPC
echo $data["ru_msgrcv"]; // количество принятых сообщений IPC
echo $data["ru_maxrss"]; // наибольший размер установленной резидентной памяти
echo $data["ru_ixrss"]; // суммарное значение размера разделяемой памяти
echo $data["ru_idrss"]; // суммарное значение размера неразделяемых данных
echo $data["ru_minflt"]; // кол. исправленных страниц (лёгкая ошибка страницы)
echo $data["ru_majflt"]; // кол.ошибочных страниц (тяжёлая ошибка страницы)
echo $data["ru_nsignals"]; // количество полученных сигналов
echo $data["ru_nvcsw"]; // количество согласованных переключений контекста
echo $data["ru_nivcsw"]; // количество несогласованных переключений контекста
echo $data["ru_nswap"]; // количество свопов
echo $data["ru_utime.tv_usec"]; // время на задачи пользователя (user time) (микросекунды)
echo $data["ru_utime.tv_sec"]; // время на задачи пользователя (user time) (секунды)
echo $data["ru_stime.tv_usec"]; // время на системные задачи (system time) (микросекунды)

А вот создание дополнительных объектов рефлексии - это затраты как памяти, так и времени на создание объектов.

Я так и написал..

Речь не про оценку, а про то что по коду гораздо проще понять что вы имеете ввиду,

Да я уже понял, что вас отвлекает код от сути вопроса.

Хотя там всё просто: Маршрутизатор анализирует URL, на основе этого создаёт нужный контроллер и отправляет маркер в родительский конструктор созданного контроллера.

Где маркер нужен в дальнейшем для проверок условного ветвления, без необходимости хранить информацию маршрутизатора в свойствах класса.

А вот создание дополнительных объектов рефлексии - это затраты как памяти, так и времени на создание объектов.

Дело не в объекте рефлексии, сама рефлексия подразумевает проанализировать свой код.
Разумеется так никто писать не будет. Это просто метафора на подход к работе с информацией из памяти.

[Vitsliputsli]

AgentSmith72,

Знакома вам функция?

$data = getrusage();

Вы опрос проводите?

Я так и написал..

Неа, вы предлагали с точностью наоборот.

Да я уже понял, что вас отвлекает код от сути вопроса.

Ага, код в разработке - это отвлекающий фактор, философия на первом месте.

Дело не в объекте рефлексии, сама рефлексия подразумевает проанализировать свой код.
Разумеется так никто писать не будет. Это просто метафора на подход к работе с информацией из памяти.

Именно, дело не в рефлексии, а в объекте.
Да, я уже понял, что вы к разработке подходите только с точки зрения пофилософствовать.

[AgentSmith72]

Vitsliputsli, ты ничего не понял.

Тебя что головой роняли в детстве? Я спросил о подходе, а именно как люди делают, гоняют переменные или хранят в свойствах, а вы имбецилы мне тут дичь втираете про мой идеальный код.

[Vitsliputsli]

AgentSmith72,

Vitsliputsli, ты ничего не понял.

Тебя что головой роняли в детстве? Я спросил о подходе, а именно как люди делают, гоняют переменные или хранят в свойствах, а вы имбецилы мне тут дичь втираете про мой идеальный код.

А ты перечитай самое первое предложение в ответе. Вся остальная дичь связана не с "твоим идеальным кодом", а с кодом который будет использоваться в том или ином "подходе". Потому как твои словесные изливания нисколько не говорят о коде, лишь о твоих поверхностных представлениях.
Я не предлагаю тебе строить какую-то "чистую" архитектуру, я отвечаю на твой вопрос, но не прямо и тупо, потому что ты пытаешься себе отстрелить ноги, а поясняя, что ты не понял того, кто проводил ревью. Как не понимаешь меня, и походу всех окружающих. Поэтому самое время спросить себя, точно ли дело в окружающих?

[AgentSmith72]

Vitsliputsli, дело в том что вы просто нарисовались здесь, не принеся ни байта полезной информации. Во мне ли дело или в вас пустословах?

[Vitsliputsli]

AgentSmith72,

дело в том что вы просто нарисовались здесь не принеся ни байта полезной информации. Во мне ли дело или в вас пустословах?

так ты не меня риторически спрашивай, спроси себя. Представь говорит тебе какая-нибудь персона: "Все кругом имбицилы, которых головой роняли в детстве, и один я охрененно классный парень". Какие ты сделаешь предварительные выводы о нем?
Тебе могут принести диск с терабайтами полезной информации, но если ты их не сможешь прочитать, они действительно бесполезны. И ладно, что не сможешь, всякое бывает, но обвинять других, если что-то не осилил, тупо.