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

    +161

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    function GetBasketUserID($bSkipFUserInit = False)
{
		$bSkipFUserInit = ($bSkipFUserInit ? True : False);   //  <----это зачем вообще надо

		if (!array_key_exists("SALE_USER_ID", $_SESSION))
			$_SESSION["SALE_USER_ID"] = 0;

		CSaleBasket::Init(false, $bSkipFUserInit);

		return $_SESSION["SALE_USER_ID"];
}

    Стандартная функция модуля интернет-магазина в битригзе. Наткнулся во время поиска неуловимого бага, из-за которого пришлось лезть в кишки CMS

    Запостил: forkhammer, 01 Декабря 2011

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

    • ava demotivator 01.12.2011 17:25 # +2

      > это зачем вообще надо?
      Это такое хитрое кастование, аналог: $bSkipFUserInit = (bool) $bSkipFUserInit;
      Зачем надо хуй-знает, это ты в кишки лазил вот и скажи нам.
      Ответить

    • ava Vasiliy 01.12.2011 17:31 # +2

      if (!array_key_exists("SALE_USER_ID", $_SESSION))
      $_SESSION["SALE_USER_ID"] = 0;
      Просто убивают подобные конструкции.
      Ответить

      • ava nobody 01.12.2011 17:34 # +1

        Наверное автор не знает такой маневр
        if(!isset($_SESSION['SALE_USER_ID'])) ....
        Что работает мне кажется в разы быстрее и выглядит по приятнее
        Ответить

        • ava rdifb0 01.12.2011 19:57 # +1

          А как на счет такой магии:
          http://ideone.com/D8tnK
          А насчет: ($bSkipFUserInit ? True : False); еще можно вот так написать !!$bSkipFUserInit
          Ответить

          • ava ZX_Spectrum 01.12.2011 22:28 # 0

            насчет магии - годный пример!
            наверное, это чтобы интереснее было дебажить...
            Ответить

          • ava Vasiliy 02.12.2011 08:10 # 0

            Даже не ожидал. Действительно Пых полон магии и волшебства. Ps Может это "пасхальное яйцо"
            Ответить

          • ava nobody 02.12.2011 18:27 # 0

            в этом и фишка, если ключ есть а значения нету, то isset выдаст false, а если это принципиально важно то можно использовать array_exists ...
            Не знаю, как по мне, так что ключ есть но с null что его вообще нету, обычно одно и тоже, но все зависит от ситуации ...
            Для данного именного кода, isset бы подошел лучше чем array_exists ...
            Ответить

    • ava nobody 01.12.2011 17:33 # 0

      $bSkipFUserInit = ($bSkipFUserInit ? True : False); - скорее всего нужна для того чтобы левые данные не подсовывали ...
      Ответить

      • ava forkhammer 01.12.2011 17:41 # 0

        это магическое преобразование используется еще в куче функций этого модуля, причем функции вызывают другие функции и везде эта строка
        Ответить

        • ava nobody 02.12.2011 18:24 # 0

          автор наверное один, с большой извилиной или занозой в голове ...
          Ответить

    • ava istem 01.12.2011 21:17 # 0

      Битрикс... Кишки... visceral divination...
      Ответить

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