[ Поиск ] - [ Пользователи ] - [ Календарь ]
Полная Версия: Не могу определиться какой вариант лучше
Страницы: 1, 2
McLotos
Добрый вечер. Пытаюсь оптимизировать код, есть 2 варианта, не могу выбрать какой лучше.
условия внутри цикла
for($i=0;$i<count($pages);$i++)
{
$ArrURL = explode('.',$router -> route);
if(USE_EXTENSION)
{
if((strtolower($pages[$i]['name']) == strtolower($ArrURL[0])) && (empty($pages[$i]['module'])))
{
$file = is_file($tpl -> skin_dir . $tpl -> skin . $type . '.' . $ArrURL[0] . $tpl -> ext) ? $type . '.' . $ArrURL[0] : $type;
$pos = $i;
}
}

else
{
$ArrURL = explode('/',$router -> route);
if((trim($pages[$i]['address'],'/') == trim($ArrURL[0],'/')) && (empty($pages[$i]['module'])))
{
$file = is_file($tpl -> skin_dir . $tpl -> skin . $type . '.' . $ArrURL[count($ArrURL)-1] . $tpl -> ext) ? $type . '.' .$ArrURL[count($ArrURL)-1] : $type;
$pos = $i;
}
}
}

или
циклы внутри условий
 if(USE_EXTENSION)
{ # адреса страниц формата site.ru/contacts.wp
$ArrURL = explode('.',$router -> route);
for($i=0;$i<count($pages);$i++)
{
if((strtolower($pages[$i]['name']) == strtolower($ArrURL[0])) && (empty($pages[$i]['module'])))
{
$file = is_file($tpl -> skin_dir . $tpl -> skin . $type . '.' . $ArrURL[0] . $tpl -> ext) ? $type . '.' . $ArrURL[0] : $type;
$pos = $i;
}
}
}

else
{ # адреса страниц формата site.ru/about/contacts
$ArrURL = explode('/',$router -> route);
for($i=0;$i<count($pages);$i++)
{
if((trim($pages[$i]['address'],'/') == trim($ArrURL[0],'/')) && (empty($pages[$i]['module'])))
{
$file = is_file($tpl -> skin_dir . $tpl -> skin . $type . '.' . $ArrURL[count($ArrURL)-1] . $tpl -> ext) ? $type . '.' .$ArrURL[count($ArrURL)-1] : $type;
$pos = $i;
}
}
}


_____________
программирование - инструмент для решения конкретных задач, любая попытка спроектировать что-то универсальное приведет к провалу.©paul85
В любом случае тебе прийдётся пройти путь изобретения велосипеда, который прошли другие, только причиной твоего изобретения будет непонимание принципов работы велосипеда изобретённого другими людьми.©SlavaFr
jQuery это попытка использовать АН-225 для перевозки зубочистки
Быстрый ответ:

 Графические смайлики |  Показывать подпись
Здесь расположена полная версия этой страницы.
Invision Power Board © 2001-2024 Invision Power Services, Inc.