如何解决需要PHP代码性能优化帮助
我有以下代码,它检查元素是否存在,如果存在,则检查名称是否相同,并以递增的数字结尾。
例如,它检查$ test-> elements数组中是否存在键“ test”,如果存在,则检查“ test2”,依此类推,直到键不存在。
>我的原始代码是:
if (isset($this->elements[$desired])) {
$inc = 0;
do {
$inc++;
$new_desired = $desired . $inc;
} while (isset($this->elements[$new_desired]));
$desired = $new_desired;
}
我尝试过:
if (isset($this->elements[$desired])) {
return $this->generateUniqueElement($desired,$postfix);
}
private function generateUniqueElement($desired,$postfix) {
$new_desired = $desired . $postfix;
return isset($this->elements[$new_desired]) ? $this->generateUniqueElement($desired,++$postfix) : $new_desired;
}
但是在我的测试中,速度并没有提高。
有什么想法可以改善代码吗?在所有页面上,此代码被调用超过1万次。有时甚至超过10万次。
期待中的感谢!
解决方法
在不进一步了解如何生成此列表的情况下,这是一个主意:
$highestElementIds = [];
foreach($this->elements as $element) {
preg_match('/(.*?)(\d+)/',$element,$matches);
$text = $matches[1];
$id = (int)$matches[2];
if(!isset($highestElementIds[$text])) {
$highestElementIds[$text] = $id;
} else {
if($id > $highestElementIds[$text]) {
$highestElementIds[$text] = $id;
}
}
}
// find some element by a simple array access
$highestElementIds['test']; // will return 2 in your example
如果您的代码确实被调用了10万次,则只对列表进行一次迭代,然后直接从包含最高编号的数组中直接获得最高ID会更快很多(因为您无需遍历整个序列)再次)。
话虽如此,我仍然想知道首先拥有如此庞大阵列的真正原因是什么...
,典型的唯一ID是随机数(UUID或随机字符)或序列号。后者非常简单,可以使用一个简单的计数器生成:
function generateNewElement($postfix) {
static $i = 0;
return sprintf('%d%s',$i++,$postfix);
}
echo generateNewElement('foo'),PHP_EOL;
echo generateNewElement('foo'),PHP_EOL;
0foo
1foo
2foo
3foo
当然,这只是一个通用解决方案,因此它可能不适合您的特定用例。
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。