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

    +164

    1. 1
    $arParams["FILES_COUNT"] = intVal(intVal($arParams["FILES_COUNT"]) > 0 ? $arParams["FILES_COUNT"] : 5);

    Битрикс. Количество, так сказать, файлов.

    Запостил: franzose, 12 Сентября 2011

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

    • ava Lure Of Chaos 12.09.2011 12:50 # 0

      5 в уме?
      Ответить

      • ava franzose 12.09.2011 12:52 # +1

        Меня больше позабавили intval'ы...
        Ответить

        • ava alex322 12.09.2011 13:23 # 0

          Выглядит конечно смешно, но предложите более красивый и краткий способ?
          Ответить

          • ava TarasB 12.09.2011 13:37 # 0

            if (intVal($arParams["FILES_COUNT"]<=0)) {$arParams["FILES_COUNT"] = intVal(5)};
            Ответить

            • ava alex322 12.09.2011 13:47 # 0

              в таком случае $arParams["FILES_COUNT"]='123test' пройдет мимо вашего условия
              Ответить

              • ava TarasB 12.09.2011 14:09 # +1

                А, точно, intval делается для любого результата.
                Тогда так:
                $arParams["FILES_COUNT"] = intVal($arParams["FILES_COUNT"]);
                if ($arParams["FILES_COUNT"]<=0) {$arParams["FILES_COUNT"] = 5};
                Ответить

          • ava yasenhui 12.09.2011 14:28 # +1

            ($arParams["FILES_COUNT"] = (int) $arParams["FILES_COUNT"]) > 0 or $arParams["FILES_COUNT"] = 5;
            Ответить

    • ava roman-kashitsyn 12.09.2011 15:44 # +1

      Меня терзает только один вопрос: почему 5, а не 3, 7, 42, <моё любимое число>?
      Ответить

      • ava Lure Of Chaos 12.09.2011 16:12 # +2

        потому что это не ваше любимое число
        Ответить

      • ava i-- 12.09.2011 17:14 # −1

        На логичный вопрос "А почему 5?" есть еще более логичный ответ "А почему бы и не 5...".
        Ответить

    • ava cephuo 13.09.2011 12:35 # 0 

      $arParams["FILES_COUNT"] = (int) intVal(intVal($arParams["FILES_COUNT"]) > 0 ?
intVal($arParams["FILES_COUNT"]) :
(int) 5);
if (is_int($arParams["FILES_COUNT"])) { ... } // Чтоб наверняка
      Ответить

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