"PHP" / Говнокод #7841 Ссылка на оригинал

0

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

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

Запостил: franzose franzose, (Updated )

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

      • ava alex322 # Ссылка на оригинал 0 ⇈

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

          • ava alex322 # Ссылка на оригинал 0 ⇈

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

            • ava TarasB # Ссылка на оригинал 0 ⇈

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

        • ava yasenhui # Ссылка на оригинал 0 ⇈

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

  • ava roman-kashitsyn # Ссылка на оригинал 0

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

  • ava cephuo # Ссылка на оригинал 0 

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓