我有一个菜单,根据语言有条件地显示。
它看起来像这样:
if (
$page == "page1.htm" ||
$page == "page2.htm" ||
$page == "page3.htm" ||
etc couple 100 of those
) {
//output menu
}
我在一个if中有几百个这样的or语句。
这会对性能产生可测量的影响吗?
我对可维护性不感兴趣,而只对性能感兴趣。我想知道它是否值得优化。
当你发现自己在if
语句中大量使用||
时,那么你可能最好使用in_array()
:
$validPages = Array("page1.htm", "page2,htm", "page3.htm");
if(in_array($page, $validPages)){
//Output menu
}
我会使用RegExp,您也可以根据自己的需要编辑:
if (preg_match("/^page[0-9]{1,3}'.htm$/", $page)){
//Execute if this is a page.
}
在我看来,最好创建某种复杂模式/几个模式来检查,因为它:
- 可能会提高性能
- 将保持你的代码干净(这是最重要的)
好吧,也许我最好把我的评论作为一个答案。如果你真的不想(或不能)重新考虑你的代码:
$pages = array_fill_keys(array("page1.htm", "page2,htm", "page3.htm"), true);
if (array_key_exists($page, $pages))
{
//do whatever
}
然后,如果您要检查客户端是否请求现有页面,您可以考虑使用:
if (file_exists($page))
{
require_once($page);
}
或者类似的东西。当然,检查$page
变量的值,如/etc/httpd/httpd.conf
,或所有太多的../..
位和螺栓…永远不要相信网络:)
正如F4r-20所提到的,使用in_array()
是一个好主意。此外,如果您确定页面将被命名为page1.htm
、page2.htm
等,则可以使用for loop
构造来执行此任务;这样的:
for($i=0; $i<$no_of_pages; $i++){
$pages[$i] = 'page'.($i+1).'.htm';
}
if(in_array($page, $pages)){
//your condition here
}
还可以将页面存储在表的列中,以便在需要时执行sql查询。
关于||
运算符,它不一定检查所有条件(例如100)。只要满足任何条件,它就停在那里并进入if语句体。所以,最好的情况是1条件,最坏的情况是100条件。
我更喜欢这样,而不是那些行代码:
$page="page10.htm"; //Example data
$match=array();
preg_match('/page('d*)'.htm/', $page,$match);
$match=$match[1];
switch($x)
{
case $x>1 && $x<40:
echo "uder 50";
break;
case $x>40:
echo "upper 50";
break;
}