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

    +155

    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
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    /*фрагмент из компонента*/
$arProps = array(
"NAMES"=>$props["NAMES"]["VALUE"],
"DESCRIPTIONS"=>$props["DESCRIPTIONS"]["VALUE"],
"LINKS"=>$props["LINKS"]["VALUE"],
"IMAGES"=>$props["PICTURES"]["VALUE"],
"URLS" => array(
"0"=>CFile::GetPath($props["PICTURES"]["VALUE"][0]),
"1"=>CFile::GetPath($props["PICTURES"]["VALUE"][1]),
"2"=>CFile::GetPath($props["PICTURES"]["VALUE"][2]),
"3"=>CFile::GetPath($props["PICTURES"]["VALUE"][3]),
"4"=>CFile::GetPath($props["PICTURES"]["VALUE"][4]),
),
);

/*код из шаблона*/
<div class="text-blocks">
<?if($arResult["PROPS"]["DESCRIPTIONS"][0]["TEXT"]):?>
  <div id="banner_text_1" class="text curr" style="left:92px; opacity:0.8; display: block">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][0]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][0]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][1]["TEXT"]):?>
  <div id="banner_text_2" class="text" style="left:156px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][1]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][1]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][2]["TEXT"]):?>
  <div id="banner_text_3" class="text" style="left:220px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][2]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][2]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][3]["TEXT"]):?>
  <div id="banner_text_4" class="text" style="left:284px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][3]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][3]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][4]["TEXT"]):?>
  <div id="banner_text_4" class="text" style="left:284px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][4]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][3]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
</div>

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

    Запостил: akovalenko, 26 Ноября 2010

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

    • ava wmmorgun 26.11.2010 13:48 # 0

      Ууу... Человеку надо рассказать про циклы.
      Ответить

    • ava Анонимус 26.11.2010 14:02 # +1

      очевидно что отсутствие циклов -- не самое страшное преступление этого гавнокода
      Ответить

      • ava Vasiliy 26.11.2010 14:12 # 0

        все остальное классика. Все запостенные ГК в раздел ПХП унылы и однобразны 99% код школоты, из них 90% код людей которые не знают ,что такое шаблоны проектирования остальное лабы, курсачи. Тоска. А где макароны ведь пыха просто рай для написания макароно кода. только жаль goto нету
        Ответить

        • ava Анонимус 26.11.2010 14:16 # 0

          а что , бывают курсачи на php?
          Ответить

          • ava akovalenko 26.11.2010 14:23 # 0

            бывают! я когда учился у меня одногруппник писал курсач, там была работа с БД, вот он его на пыхе и оформил
            Ответить

        • ava wmmorgun 26.11.2010 14:56 # +1

          Vasiliy: только жаль goto нету
          В 5.3 уже есть ;-)

          http://php.net/manual/en/control-structures.goto.php
          Ответить

          • ava Vasiliy 26.11.2010 15:02 # 0

            Ну вот значит скоро появится годный говнокод, а то все одно и тоже.
            Ответить

          • ava Анонимус 26.11.2010 21:15 # +1

            абажаю php! больше новых, хороших возможностей
            Ответить

          • ava Int 26.11.2010 22:21 # 0

            >Firefox can't establish a connection to the server at php.net.
            Говнокодоэффект? о_О
            Ответить

          • ava istem 26.11.2010 23:28 # 0

            Вот так новость...
            Ёперный театр... мой мозжечок забыл как надо говорить в этом случае...
            Ответить

    • ava inkanus-gray 27.11.2010 03:10 # 0

      Странно, что до сих пор никто не сказал про table из одного единственного td. Для завершения картины маслом недотаблица внутри дива, в котором захардкожена координата в пикселях. На неродных шрифтах или неродном дисплее будет красиво.

      Да, ещё в первой записи opacity:0.8; display: block, а в остальных opacity:0; display: none. Это типа меню?

      Кто-нибудь обратил внимание на две последние записи? Надеюсь, что это опечатка...
      Ответить

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