Будет ли много вопросов или проблем, связанных с производительностью?

Question

У меня есть меню, которое условно отображается в зависимости от языка.

Это выглядит следующим образом:

if (
$page == "page1.htm" || 
$page == "page2.htm" || 
$page == "page3.htm" || 
etc couple 100 of those 
) { 
//output menu 
} 

У меня есть несколько hunderd тех или заявлений в одном случае.

Будет ли это заметно влиять на производительность?

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

Answer 1

Ну, возможно, я бы лучше опубликовать свой комментарий в качестве ответа. Если вы действительно не хотите (или не может) пересмотреть свой код в данный момент:

$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, или все слишком много ../.. бит и болтов ... Никогда не доверяйте сети :)

Answer 2

Всякий раз, когда вы оказываетесь с помощью || много в if заявлении, то ваш, вероятно, лучше всего использовать in_array():

$validPages = Array("page1.htm", "page2,htm", "page3.htm"); 
if(in_array($page, $validPages)){ 
    //Output menu 
} 

Answer 3

Я хотел бы использовать RegExp для этого, вы можете также изменить ваши собственные нужды:

if (preg_match("/^page[0-9]{1,3}\.htm$/", $page)){ 
    //Execute if this is a page. 
} 

на мой взгляд, лучше, чтобы создать своего рода сложных шаблонов/нескольких шаблонов проверить с ним, потому что:

• Вероятно, позволит повысить производительность
• будет держать ваш код чистый (что является наиболее важным)

Answer 4

Как упоминаемый 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 условий.

Answer 5

я предпочитаю это вместо этих строк кода:

$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; 
}