Говнокод #18890 — PHP — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP / Говнокод #18890

function Show($aMenu, $level=0)
	{
		$scripts = '';

		$bSubmenu = (isset($aMenu["items"]) && is_array($aMenu["items"]) && !empty($aMenu["items"])) || isset($aMenu["dynamic"]) && $aMenu["dynamic"] == true;
		$bSectionActive = isset($aMenu["items_id"]) && (in_array($aMenu["items_id"], array_keys($this->aActiveSections)) || $this->IsSectionActive($aMenu["items_id"]));

		$icon = isset($aMenu["icon"]) && $aMenu["icon"] <> ""
			? '<span class="adm-submenu-item-link-icon '.$aMenu["icon"].'"></span>'
//			: ($level < 1 ? '<span class="adm-submenu-item-link-icon" id="default_menu_icon"></span>' : '');
			: '';
		$id = 'menu_item_'.RandString(10);
		?><div class="adm-sub-submenu-block<?=$level > 0 ? ' adm-submenu-level-'.($level+1) : ''?><?=$bSectionActive && isset($aMenu["items"]) && is_array($aMenu["items"]) && count($aMenu['items']) > 0 ? ' adm-sub-submenu-open' : ''?><?=$aMenu["_active"] ? ' adm-submenu-item-active' : ''?>"><?
		?><div class="adm-submenu-item-name<?=!$bSubmenu ? ' adm-submenu-no-children' : ''?>" id="<?=$id?>" data-type="submenu-item"<?=isset($aMenu['fav_id']) ? ' data-fav-id="'.intval($aMenu['fav_id']).'"' : ''?>><?
		$onclick = '';
		if ($bSubmenu)
		{
			if(isset($aMenu["dynamic"]) && $aMenu["dynamic"] == true && (!$aMenu["items"] || count($aMenu["items"]) <= 0))
			{
				$onclick = "BX.adminMenu.toggleDynSection(".$this->_get_menu_item_width($level).", this.parentNode.parentNode, '".htmlspecialcharsbx(CUtil::JSEscape($aMenu["module_id"]))."', '".htmlspecialcharsbx(CUtil::JSEscape($aMenu["items_id"]))."', '".($level+1)."')";
			}
			elseif(!$aMenu["dynamic"] || !$bSectionActive || $aMenu['dynamic'] && $bSectionActive && isset($aMenu["items"]) && count($aMenu["items"]) > 0)
			{
				$onclick = "BX.adminMenu.toggleSection(this.parentNode.parentNode, '".htmlspecialcharsbx(CUtil::JSEscape($aMenu["items_id"]))."', '".($level+1)."')";
			} //endif;
		}

		?><span class="adm-submenu-item-arrow"<?=$level > 0 ? ' style="width:'.$this->_get_menu_item_width($level).'px;"' : ''?><?=$onclick ? ' onclick="'.$onclick.'"' : ''?>><span class="adm-submenu-item-arrow-icon"></span></span><?

		if(isset($aMenu["url"]) && $aMenu["url"] <> ""):
			?><a class="adm-submenu-item-name-link<?=(isset($aMenu["readonly"]) && $aMenu["readonly"] == true? ' menutext-readonly':'')?>"<?=$level > 0 ? ' style="padding-left:'.$this->_get_menu_item_padding($level).'px;"' : ''?> href="<?=$aMenu["url"]?>"><?=$icon?><span class="adm-submenu-item-name-link-text"><?=$aMenu["text"]?></span></a><?
		elseif ($bSubmenu):
			if(isset($aMenu["dynamic"]) && $aMenu["dynamic"] == true && !$bSectionActive && (!$aMenu["items"] || count($aMenu["items"]) <= 0)):
				?><a class="adm-submenu-item-name-link<?=(isset($aMenu["readonly"]) && $aMenu["readonly"] == true? ' menutext-readonly':'')?>"<?=$level > 0 ? ' style="padding-left:'.$this->_get_menu_item_padding($level).'px;"' : ''?> href="javascript:void(0)" onclick="BX.adminMenu.toggleDynSection(<?=$this->_get_menu_item_width($level)?>, this.parentNode.parentNode, '<?=htmlspecialcharsbx(CUtil::JSEscape($aMenu["module_id"]))?>', '<?=htmlspecialcharsbx(CUtil::JSEscape($aMenu["items_id"]))?>', '<?=$level+1?>')"><?=$icon?><span class="adm-submenu-item-name-link-text"><?=$aMenu["text"]?></span></a><?
			elseif(!$aMenu["dynamic"] || !$bSectionActive || $aMenu['dynamic'] && $bSectionActive && isset($aMenu["items"]) && count($aMenu["items"]) > 0):
				?><a class="adm-submenu-item-name-link<?=(isset($aMenu["readonly"]) && $aMenu["readonly"] == true? ' menutext-readonly':'')?>"<?=$level > 0 ? ' style="padding-left:'.$this->_get_menu_item_padding($level).'px;"' : ''?> href="javascript:void(0)" onclick="BX.adminMenu.toggleSection(this.parentNode.parentNode, '<?=htmlspecialcharsbx(CUtil::JSEscape($aMenu["items_id"]))?>', '<?=$level+1?>')"><?=$icon?><span class="adm-submenu-item-name-link-text"><?=$aMenu["text"]?></span></a><?
			else:
				?><span class="adm-submenu-item-name-link<?=(isset($aMenu["readonly"]) && $aMenu["readonly"] == true? ' menutext-readonly':'')?>"<?=$level > 0 ? ' style="padding-left:'.$this->_get_menu_item_padding($level).'px"' : ''?>><?=$icon?><span class="adm-submenu-item-name-link-text"><?=$aMenu["text"]?></span></span><?
			endif;
		else:
			?><span class="adm-submenu-item-name-link<?=(isset($aMenu["readonly"]) && $aMenu["readonly"] == true? ' menutext-readonly':'')?>"<?=$level > 0 ? ' style="padding-left:'.$this->_get_menu_item_padding($level).'px"' : ''?>><?=$icon?><span class="adm-submenu-item-name-link-text"><?=$aMenu["text"]?></span></span><?
		endif;
		?></div><?

		if(($bSubmenu || (isset($aMenu["dynamic"]) && $aMenu["dynamic"] == true)) && is_array($aMenu["items"]))
		{
			echo  "<div class=\"adm-sub-submenu-block-children\">";
			foreach($aMenu["items"] as $submenu)
			{
				if($submenu)
				{
					$scripts .= $this->Show($submenu, $level+1);
				}
			}
			echo "</div>";
		}

это между прочим метод класса CAdminPage административного интерфейса 1С Битрикс.
Адская смесь всех вэб-технологий, от которой лопаются глаза!

Запостил:

nik757, 20 Октября 2015

Комментарии (23) RSS

Thorax 20.10.2015 13:40 # −2

Мне кажется, что 95% людей на планете - тоже написаны на Битрикс
Ответить
- roman-kashitsyn 20.10.2015 14:35 # +1
  
  Частично на лиспе, но в основном - на perl.
  Ответить
  - guest 20.10.2015 15:53 # 0
    
    Гусары, ни слова про питон!
    Ответить
    - roman-kashitsyn 20.10.2015 16:04 # +1
      
      https://xkcd.com/224/
      Ответить
- CHayT 20.10.2015 14:37 # +1
  
  знаю людей, написанных на brainfuck
  Ответить
gost 20.10.2015 14:54 # +1

Ты так говоришь "между прочим", как-будто 1С Битрикс - это образец идеального кода идеальной компании.
Ответить
- nik757 20.10.2015 17:53 # 0
  
  пффф, это в каком месте такое утверждалось? ппц, %лять, образец идельного кода.
  "Между прочем" - это потому, что весь это 3,14здец находится в методе класса, где ему не место. Не говоря уже про разделение M-V-C а тут млять порноMVC в одном файле
  Ответить
Keeper 20.10.2015 20:56 # 0

Это битрикс детка! Чему ты удивляешься? http://habrahabr.ru/company/bitrix/blog/139721/#comment_4669119
Ответить
- barbarbar 20.11.2015 16:30 # 0
  
  А что там было?
  Ответить
defecate-plusplus 20.10.2015 22:01 # 0

вот вам смешно, а у меня на столе лежит ТЗ от заказчика, где, окромя всего прочего, требуется ему запилить портал на битриксе

вот неужели нет адекватной замены ему? (шарепойнт не предлагать)

а мне теперь субподряд/аутсорс искать
Ответить
- Vasiliy 20.10.2015 22:13 # 0
  
  >вот неужели нет адекватной замены ему? (шарепойнт не предлагать)
  Полно. Как мне рассказывал один разработчик на Битрексе, заказуны сут паром от интеграции с 1С бухгалтерией.
  Хотя проблему сделать тоже самое на другие платформы я не вижу.
  Ответить
  - bormand 20.10.2015 22:35 # 0
    
    > интеграции с 1С бухгалтерией
    Но ведь у битрикса от 1с только название?
    
    И с 1с поди xml'ками общается? Или всё же более тесная интеграция?
    Ответить
    - defecate-plusplus 20.10.2015 22:39 # 0
      
      1с этому заказчику не нужен
      
      корпоративный портал - календари, персоны, чят, новости, вот это всё
      причем, почему-то конкретно на битриксе надо делать
      
      а у меня чо т нет совсем желания расширяться пыхоёбом по такому счастливому случаю
      Ответить
      - Vasiliy 21.10.2015 14:02 # 0
        
        >причем, почему-то конкретно на битриксе надо делать
        маркетинг годный у 1C
        Ответить
      - Stallman 21.10.2015 14:20 # 0
        
        От PHP не зарекайся. (с) борманд
        Ответить
        
        defecate-plusplus 21.10.2015 14:46 # +2
        
        да я и не зарекаюсь
        просто максимально избегаю
        Ответить
        
        Vasiliy 22.10.2015 14:15 # 0
        
        php он везде уже и на десктопе
        Ответить
        
        defecate-plusplus 22.10.2015 15:15 # +1
        
        http://risovach.ru/upload/2015/10/mem/oni-ge-ecshe-tolko-deti_95939001_orig_.jpg
        Ответить
    - Vasiliy 21.10.2015 13:58 # 0
      
      через SOAP может. Хотя может и через xml. И хз ещё какими способами.
      Ответить
- nik757 21.10.2015 08:45 # 0
  
  >> требуется ему запилить портал на битриксе
  аналогичная ситуация. Мои соболезнования...
  Ответить
  - guest 23.10.2015 17:12 # 0
    
    Мои категорические соболезнования! Я прошел через кишечник того, что битрикс назвывает "управление сайтом", но даже после этого их корппортал вызывает у меня смятение!
    Ответить
    - inkanus-gray 23.10.2015 17:41 # 0
      
      Слово «корппортал» в одном предложении со словом «кишечник» у меня сначала прочиталось как «копропортал».
      Ответить
      - kegdan 24.10.2015 22:39 # +1
        
        Не, корппортал - это же портал для трупов
        Ответить

Говнокод: по колено в коде.

PHP / Говнокод #18890

Комментарии (23) RSS

Добавить комментарий