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

    +59

    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
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    "IBLOCK_TYPE" => $arParams["IBLOCK_TYPE"],
    "IBLOCK_ID" => $arParams["IBLOCK_ID"],
    "ELEMENT_SORT_FIELD" => $arParams["ELEMENT_SORT_FIELD"],
    "ELEMENT_SORT_ORDER" => $arParams["ELEMENT_SORT_ORDER"],
    "PROPERTY_CODE" => $arParams["LIST_PROPERTY_CODE"],
    "META_KEYWORDS" => $arParams["LIST_META_KEYWORDS"],
    "META_DESCRIPTION" => $arParams["LIST_META_DESCRIPTION"],
    "BROWSER_TITLE" => $arParams["LIST_BROWSER_TITLE"],
    "INCLUDE_SUBSECTIONS" => $arParams["INCLUDE_SUBSECTIONS"],
    "BASKET_URL" => $arParams["BASKET_URL"],
    "ACTION_VARIABLE" => $arParams["ACTION_VARIABLE"],
    "PRODUCT_ID_VARIABLE" => $arParams["PRODUCT_ID_VARIABLE"],
    "SECTION_ID_VARIABLE" => $arParams["SECTION_ID_VARIABLE"],
    "FILTER_NAME" => $arParams["FILTER_NAME"],
    "DISPLAY_PANEL" => $arParams["DISPLAY_PANEL"],
    "CACHE_TYPE" => $arParams["CACHE_TYPE"],
    "CACHE_TIME" => $arParams["CACHE_TIME"],
    "CACHE_FILTER" => $arParams["CACHE_FILTER"],
    "CACHE_GROUPS" => $arParams["CACHE_GROUPS"],
    "SET_TITLE" => $arParams["SET_TITLE"],
    "SET_STATUS_404" => $arParams["SET_STATUS_404"],
    "DISPLAY_COMPARE" => $arParams["USE_COMPARE"],
    "PAGE_ELEMENT_COUNT" => $arParams["PAGE_ELEMENT_COUNT"],
    "LINE_ELEMENT_COUNT" => $arParams["LINE_ELEMENT_COUNT"],
    "PRICE_CODE" => $arParams["PRICE_CODE"],
    "USE_PRICE_COUNT" => $arParams["USE_PRICE_COUNT"],
    "SHOW_PRICE_COUNT" => $arParams["SHOW_PRICE_COUNT"],

    Очень ценные данные. Сразу как-то подозрительно, когда видишь в cms такое. Ну это ж божественная суть битрикса...

    Запостил: gorky, 22 Октября 2012

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

    • иначе было бы слишком просто
      Ответить
      • ага, и слишком дешево
        Ответить
        • Здесь аж 27 символов доллара. 1$ за строку слишком мало, индусы смотрят на лохов из битрикса со смехом, они бы по 10 в строке запихнули.
          Ответить
    • > суть битрикса
      Битрикс это место, где программисты на PHP могут побыть сами собой - ужасными, бесчувственными, безразличными мартышками, которыми они на самом деле и являются.
      Ответить
    • у простых смертных глаза вытекают из глазниц, когда такое видят
      Ответить
      • Простым смертным наверное не стоит вообще в исходники смотреть, любого проекта.
        Ответить
        • Нет. Только проекта на языке с пидерстическим синтаксисом
          Ответить
    • Почему все так любят постить эту тривиальщину и банальщину?
      Блин, такое же я вижу каждый день в своих проэктах. Скушно.
      Толи дело веселее обвинить в своём неосиляторстве библиотеку, компилятор или язык. Сказать что в нем ошибка или проектный просчет.
      С пеной у рта доказывать это пару суток. Утонуть всем топиком и кармой в минуса, перезавести очередного бота без репутации упоротого тролля и выжатым, но довольным уйти дальше быдлокодить свой очередной говнокод.
      Ответить
      • вопрос солидарности и вообще круговая порука.
        ты ниасилил проэкт, патамушта кто-то ниасилил библиатэку, а библиатэк кривой патамушта вендаОСь крива и глючна.
        Ответить
        • >вендаОСь крива и глючна.
          потому что писали ниосилиряторы инудсы.
          Ответить
    • Обычный код. Ничего особенного. Так в Битриксе в шаблоне комплексного компонента вызываются простые компоненты.

      Достаточно в настройках комплексного в одном месте при вызове указать $arParams["IBLOCK_ID"] = 4, а он передаст эту опцию всем внутренним простым компонентам "IBLOCK_ID" => $arParams["IBLOCK_ID"], как приведено это в примере. Довольно удобно.

      Похоже автор впервые установил Битрикс, не разобрался в структуре, открыл на редактирование первый попавшийся шаблон комплексного компонента, обосрался и выложил сюда.
      Ответить
    • Код конечно выглядит страшно... но, имхо, это не говнокод. Вырванный из контекста - ага, страшен.

      Ага, можно было бы конечно сказать for each property of $arParams трам-пам-пам... но...
      Ответить
    • сколько раз выручал именно такой подход!
      Ответить
    • не пойму что вам не нравится в таком подходе, при каждом вызове можно корректироватть отображение, гибко и удобно.
      Ответить
    • Единственная говнокодистость тут только в паре моментов, например, для комплексного компонента параметр назван USE_COMPARE, а для простого компонента - DISPLAY_COMPARE. И все. В остальном же мы видим нормальную передачу параметров от комплексного к простому компоненту.

      Но, конечно, если мозг только под один подход в решении заточился, то лучше сразу обозвать все непривычное говнокодом и запастить сюда.

      Может, не стоит тогда и программировать, это занятие, как-никак, гибкости ума требует при алгоритмировании.
      Ответить
      • Да-да, если мозг вышел из зародешевого состояния и привык к циклам вместо копипасты, то копипасту можно сразу обзывать говнокодом и постить сюда. А если это не одноразовый код на выброс, а встречается в серьезном проекте - то не можно а нужно.
        Ответить
        • Я даже не могу понять, настолько это тонкая ирония или настолько серьезные проблемы с логикой:)
          Ответить

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