-
Notifications
You must be signed in to change notification settings - Fork 27
переменная нужна для memory_get_usage, чистится после выполнения функции #10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Подожди.
Ты хочешь, чтобы в столбце MEMORY выводилось сколько памяти было запрошено при выполнении теста?
Можно тогда просто убрать очистки от переменных.
Или сохранять значение по памяти в переменную, а тестовые переменные очищать.
Ты хочешь, чтобы в столбце MEMORY выводилось сколько памяти было запрошено при выполнении теста?
да, для этого и добавил этот столбец :)
Или сохранять значение по памяти в переменную, а тестовые переменные очищать.
php автоматом очищает переменные после выполнения функций, потому в столбце memory только напротив concat большое значение
Тогда надо везде unset убрать, во всех тестах перед return
.
Нашел unset в test_13_Array_Fill и test_14_Array_Unset
В array_fill имеет смысл убрать, а array_unset тестит unset, здесь это нужно.
Кстати, в array_unset тайминг некорректен т.к. учитывается еще range. По идее сначала надо наполнить массив, а потом определять $time_start, могу заодно это переписать.
Еще обратил внимание, что $s определяется в цикле и коммит 4156024 не нужен
добавил отдельно тест наполнения массива, разница между ним и array_unset показывает скорость работы unset
так же эти тесты показали, что php7 экономнее использует память для массивов
bench.php
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Должно быть $i++ в цикле
bench.php
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Должно быть $arrayDimensionLimit * $arrayTestLoopLimit
, т.к. считается кол-во операций в php.
bench.php
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$i++
bench.php
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$j++
Добавил комментарии к коду
сразу хочу уточнить: во всех функциях где есть for(...;++$) заменить ++$ на $++, или только в прокомментированных?
на самом деле без разницы ++$ или $++ в for(...), внутри цикла значение не меняется
Только в прокомментированных
...mment-304811598
Ок
это нужно для столбца MEMORY