alifallah
جمعه 14 فروردین 1394, 01:34 صبح
سلام، من این تابع رو برای لیست کردن موضوعات با قالب بوت استرپ تو ساید بار نوشتم، میخوام ببینم میشه اینو بهتر و خلاصه تر نوشت؟! هم از لحاظ تراکم کد و هم اینکه تو کوئری ها بهینه باشه، ممنون میشم راهنمایی کنید.
function GetSidebarCatsMenu(){
$allcats = GetVar('AllCats');
if(!$allcats) return '<p align="center">هیچ گرو بندی برای کالاها لحاظ نشده است</p>';
$html = '<ul class="nav nav-pills nav-stacked pd_mg_free">';
foreach($allcats as $allcat){
$subcats = Select('*', 'products_cats', 'parent_id="'.$allcat->id.'"', 'id ASC');
if($subcats){
$catsarray = array();
foreach($subcats as $arr){ $catsarray[] = $arr->id; }
if(in_array(GetVar('SelectedCat'), $catsarray)){
$html .= '<li role="presentation" class="dropdown active">';
}else{
$html .= '<li role="presentation" class="dropdown">';
}
$html .= '<a href="'.SiteUrl('search.php?catid='.$allcat->id).'" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">'.$allcat->name.' <span class="caret pull-left" style="margin-top:5px;"></span></a>';
$html .= '<ul class="dropdown-menu" role="menu">';
foreach($subcats as $subcat){
if(GetVar('SelectedCat') == $subcat->id){
$html .= '<li role="presentation" class="active">';
}else{
$html .= '<li role="presentation">';
}
$html .= '<a href="'.SiteUrl('search.php?catid='.$subcat->id).'" role="menuitem" style="text-align: right; font-size: 11px;">'.$subcat->name.'</a>';
$html .= '</li>';
}
$html .= '</ul>';
$html .= '</li>';
}else{
if(GetVar('SelectedCat') == $allcat->id){
$html .= '<li class="active">';
}else{
$html .= '<li>';
}
$html .= '<a href="'.SiteUrl('search.php?catid='.$allcat->id).'">'.$allcat->name.'</a>';
$html .= '</li>';
}
}
$html .= '</ul>';
return $html;
}
function GetSidebarCatsMenu(){
$allcats = GetVar('AllCats');
if(!$allcats) return '<p align="center">هیچ گرو بندی برای کالاها لحاظ نشده است</p>';
$html = '<ul class="nav nav-pills nav-stacked pd_mg_free">';
foreach($allcats as $allcat){
$subcats = Select('*', 'products_cats', 'parent_id="'.$allcat->id.'"', 'id ASC');
if($subcats){
$catsarray = array();
foreach($subcats as $arr){ $catsarray[] = $arr->id; }
if(in_array(GetVar('SelectedCat'), $catsarray)){
$html .= '<li role="presentation" class="dropdown active">';
}else{
$html .= '<li role="presentation" class="dropdown">';
}
$html .= '<a href="'.SiteUrl('search.php?catid='.$allcat->id).'" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">'.$allcat->name.' <span class="caret pull-left" style="margin-top:5px;"></span></a>';
$html .= '<ul class="dropdown-menu" role="menu">';
foreach($subcats as $subcat){
if(GetVar('SelectedCat') == $subcat->id){
$html .= '<li role="presentation" class="active">';
}else{
$html .= '<li role="presentation">';
}
$html .= '<a href="'.SiteUrl('search.php?catid='.$subcat->id).'" role="menuitem" style="text-align: right; font-size: 11px;">'.$subcat->name.'</a>';
$html .= '</li>';
}
$html .= '</ul>';
$html .= '</li>';
}else{
if(GetVar('SelectedCat') == $allcat->id){
$html .= '<li class="active">';
}else{
$html .= '<li>';
}
$html .= '<a href="'.SiteUrl('search.php?catid='.$allcat->id).'">'.$allcat->name.'</a>';
$html .= '</li>';
}
}
$html .= '</ul>';
return $html;
}