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

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    # массив вывода BB-кодов
  $BBcode = array(
  "SIZE" => "<select class='nc_bbcode_bar_size' onChange=\"insert_bbcode('".$winID."','".$formID."','".$textareaID."', '[SIZE=' + this.value + ']', '[/SIZE]'); this.selectedIndex=0;\"  name='bb_fontsize' title='".NETCAT_BBCODE_SIZE."' ".($help ? "onMouseOver=\"show_bbcode_tips('".$winID."','".$formID."','".$textareaID."','".NETCAT_BBCODE_HELP_SIZE."');\"" : "").">\r\n<option value=''>-- ".NETCAT_BBCODE_SIZE_DEF." --\r\n<option value='8'>8px\r\n<option value='10'>10px\r\n<option value='12'>12px\r\n<option value='14'>14px\r\n<option value='16'>16px\r\n<option value='18'>18px\r\n<option value='20'>20px\r\n<option value='22'>22px\r\n<option value='24'>24px\r\n</select>\r\n",
  "COLOR" => "<a href='#' onClick=\"show_color_buttons('".$textareaID."'); return false;\" id='nc_bbcode_color_button_".$textareaID."' ".($help ? "onMouseOver=\"show_bbcode_tips('".$winID."','".$formID."','".$textareaID."','".NETCAT_BBCODE_HELP_COLOR."');\"" : "")."><img src='".$SUB_FOLDER."/images/i_color.gif' alt='".NETCAT_BBCODE_COLOR."' class='nc_bbcode_wicon'></a>\r\n",
  "SMILE" => "<a href='#' onClick=\"show_smile_buttons('".$textareaID."'); return false;\" id='nc_bbcode_smile_button_".$textareaID."' ".($help ? "onMouseOver=\"show_bbcode_tips('".$winID."','".$formID."','".$textareaID."','".NETCAT_BBCODE_HELP_SMILE."');\"" : "")."><img src='".$SUB_FOLDER."/images/i_smile.gif' alt='".NETCAT_BBCODE_SMILE."' class='nc_bbcode_wicon'></a>\r\n",
  "B" => "<a href='#' onClick=\"insert_bbcode('".$winID."','".$formID."','".$textareaID."','[B]','[/B]'); return false;\" ".($help ? "onMouseOver=\"show_bbcode_tips('".$winID."','".$formID."','".$textareaID."','".NETCAT_BBCODE_HELP_B."');\"" : "")."><img src='".$SUB_FOLDER."/images/i_bold.gif' alt='".NETCAT_BBCODE_B."' class='nc_bbcode_icon'></a>\r\n",
  // ... ещё 8 подобных строк ...
  "CUT" => "<a href='#' onClick=\"insert_bbcode('".$winID."','".$formID."','".$textareaID."','[CUT=\'".NETCAT_BBCODE_CUT_MORE."\']','[/CUT]'); return false;\" ".($help ? "onMouseOver=\"show_bbcode_tips('".$winID."','".$formID."','".$textareaID."','".NETCAT_BBCODE_HELP_CUT."');\"" : "")."><img src='".$SUB_FOLDER."/images/i_cut.gif' alt='".NETCAT_BBCODE_CUT."' class='nc_bbcode_icon'></a>\r\n");

    Нет, это не Битрикс, это хуже. Зовётся сие чудо NetCat. Моё личное впечатление от CMSки - ТЗ составлял кто-то весьма неглупый, потому что пользоваться довольно удобно и в концепцию заложены прикольные идеи, но вот код писать посадили младшекурсников за экзамен автоматом. Без переписывания половины кода будущего у неё нет. Уже на PHP 5.3 или на MySQL старше 5.0 она отказывается работать. И тем не менее всё больше хостеров гордо пишут на своих сайтах, что поддерживают её.

    Запостил: telnet, 16 Мая 2010

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

    • ava Мистер Хэнки 16.05.2010 20:11 # 0

      ой, да вроде всё в порядке
      Ответить

    • ava Lure Of Chaos 16.05.2010 21:28 # 0

      > Нет, это не Битрикс

      а то бы еще ББкод был бы по-русски ))
      Ответить

    • ava guest 17.05.2010 14:48 # −1

      У вас есть вариант лучше?
      Ответить

      • ava telnet 18.05.2010 09:04 # 0

        Не понял, что Вы имеете в виду. Реализацию списка кнопок BB-кодов?
        Ответить

        • ava guest 21.05.2010 11:07 # 0

          Как бы Вы написали аналогичный кусок кода?
          Ответить

          • ava telnet 21.05.2010 17:59 # 0

            Во-первых, поскольку без JavaScript всё это великолепие всё равно не работает, я бы перенёс генерацию панели BB-кодов целиком на клиентскую сторону. С сервера приходил бы только вызов JS-функции, выводящей панель, и помещённая в тег <noscript> надпись "У вас не включён JS, вот вам <a>справка</a> по BB-кодам". Во-вторых, конечно же, я бы использовал циклы вместо копипасты. Ну и плюс мелкие доработки крупным напильником, чтобы не так говнисто было.
            З.Ы. Оригинал функции (OH SHI~, copyrighted content!): http://paste.org/pastebin/view/18782
            З.З.Ы. paste.org ещё немного изговнякал код. В оригинале каждый "rn" выглядит как "\r\n".
            Ответить

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