1. PHP / Говнокод #6509

    +160

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    class CIBlockFindTools
    
    ...
    
    function GetSectionID($section_id, $section_code, $arFilter)
    {
    	$section_id = intval($section_id);
    	if($section_id > 0)
    	{
    		return $section_id;
    	} else if(){
                  }
                 return 0;
    ...
    }

    битрикс суров, использует свою реализацию intval(). как мы могли без этого жить?

    Запостил: hunterman, 29 Апреля 2011

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

    • где тут реализация intval?
      Ответить
      • реализации нет. наверное не так прокоментировал :) но суть в том что вместо использования inval() битриксоиды используют эту функцию.
        Ответить
        • Наверно раньше там был еще какой-то код...
          Ответить
    • $section_code, $arFilter - а это с какой целью?
      Ответить
      • это для опущеного не интересного кода, в блоке else if. троеточие не туда влезло.

        зы как подредактировать говнокод?
        Ответить
    • А зачем нужен пустой else if? Автор, вы от нас что-то сркываете!
      Ответить
    • В том смысле, что эта функция легко оптимизируется в следующий код:
      0;
      // example
      // before:
      $foo = GetSectionID(NULL, FALSE, TRUE);
      // after:
      $foo = 0;
      Ответить
    • > CIBlockFindTools

      CodeIgniter ???

      Нравятсяя мне такие классы вот хрен поймешь что он делает и главное на хрена

      Вот это судя по названиию ищет блоки в инструментах, сдается мне что есть класс который ищет блоки Не в инструментах.
      Или не блоки .....
      Ответить
    • мда... автору -1
      Ответить
    • не... этот метод ищет по $section_code(символьный код элемента) SECTION_ID(идентификатор элемента), но если он уже задан то он его обратно возвращает и не парится за поиски, а в компонент эти параметры из настроек приходят
      Ответить

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