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

    • ava Uchkuma 29.04.2011 10:50 # 0

      где тут реализация intval?
      Ответить

      • ava hunterman 29.04.2011 10:52 # 0

        реализации нет. наверное не так прокоментировал :) но суть в том что вместо использования inval() битриксоиды используют эту функцию.
        Ответить

        • ava guest 29.04.2011 17:05 # 0

          Наверно раньше там был еще какой-то код...
          Ответить

    • ava istem 29.04.2011 10:53 # −1

      $section_code, $arFilter - а это с какой целью?
      Ответить

      • ava hunterman 29.04.2011 10:56 # −1

        это для опущеного не интересного кода, в блоке else if. троеточие не туда влезло.

        зы как подредактировать говнокод?
        Ответить

    • ava guest 29.04.2011 11:32 # 0

      А зачем нужен пустой else if? Автор, вы от нас что-то сркываете!
      Ответить

    • ava guest 29.04.2011 11:36 # −2

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

    • ava Vasiliy 30.04.2011 23:10 # 0

      > CIBlockFindTools

      CodeIgniter ???

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

      Вот это судя по названиию ищет блоки в инструментах, сдается мне что есть класс который ищет блоки Не в инструментах.
      Или не блоки .....
      Ответить

    • ava guest 26.08.2011 10:00 # 0

      мда... автору -1
      Ответить

    • ava exbatek 03.08.2015 19:14 # 0

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

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