-
Notifications
You must be signed in to change notification settings - Fork 67
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
Id and signature in scan log #99
base: master
Are you sure you want to change the base?
Conversation
$signatureDetect->id = $attributes->getNamedItem('id')->nodeValue; | ||
$attrFormat = $attributes->getNamedItem('format')->nodeValue; | ||
$attrChildId = $attributes->getNamedItem('child_id')->nodeValue; | ||
$signatureDetect->severity = $attributes->getNamedItem('sever')->nodeValue; |
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.
Я в предыдущем коммите делал конвертацию в массив. При поиске по массиву будет работать быстрее, требует меньше памяти, решение получается более гибкое (можно, например, кэшировать его сериализацией на момент сканирования). Может стоит сюда это перенести?
Еще был фикс ошибки с конструктором Auth - я переименовывал метод auth(), чтобы не было Constructor is already defined на некоторых версиях PHP.
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.
Фикс перенесем обязательно. Про массив - надо померить, кажется. Если действительно с классом будет значительное увеличение накладных ресурсов, то да, надо сделать массив.
Дай день-другой на сравнение.
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.
Есть подозрение, что если и есть увеличение, то незначительное, а часть с формированием очереди так удобно вынести, как мне кажется.
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.
http://stackoverflow.com/questions/2193049/php-objects-vs-arrays
arrays: 1.8451430797577
memory: 460416
Array
(
[aaa] => aaa
[bbb] => bbb
[ccc] => aaabbb
)
arrays: 1.8294548988342
memory: 275696
SomeClass Object
(
[aaa] => aaa
[bbb] => bbb
[ccc] => aaabbb
)
No description provided.