1. JavaScript / Говнокод #4412

    +144

    1. 1
    if ((text == '' || text == '\n') && document.getElementById('log').innerHTML == '...')

    Вот такое говнецо я сделал. Надо исправить.
    Следуя лозунгу говнокода "Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!", выкладываю.

    КЭП: document.getElementById('log').innerHTML == '...' - Фэйл, не надо завязываться на содержимое.

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

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

    • ava Анонимус 22.10.2010 03:08 # 0

      надо же, кто-то еще на чистом JS пишет
      Ответить

      • ava fddpro 22.10.2010 05:36 # 0

        А вы на грязном пишите?
        Ответить

        • ava Анонимус 22.10.2010 13:33 # 0

          на jquery обычно. ну или на gwt, если действительно много сложной логики.
          Ответить

      • ava telnet 22.10.2010 06:24 # +5

        JS? Это вообще что, какая-то надстройка над jQuery? Да ну её, чё-то там как-то всё сложно. И $ нельзя использовать.
        Ответить

        • ava Анонимус 22.10.2010 13:37 # 0

          Вот например аккардион:

          <div>
<div class="accTitle">title1</div>
<div class="accBody">body1<br/>body1</div>
</div>

<div>
<div class="accTitle">title2</div>
<div class="accBody">body2<br/>body2</div>
</div>

<div>
<div class="accTitle">title3</div>
<div class="accBody">body3<br/>body3</div>
</div>


          а вот код на jquery (посчитайте строчки)
          $(document).ready(function() {
    $(".accBody").slideUp(0);
    $(".accBody:first").slideDown(0);
    $(".accTitle").click(function() {
        $(".accBody").slideUp();
        $(this).parent().find(".accBody").slideDown();
    });
});


          А теперь изобразите тоже самое на javascript с плавным выпадением и кросс-браузерно.
          А потом острите)
          Ответить

          • ava telnet 22.10.2010 13:56 # 0

            Заветы Евгения Вагановича позволяют решить эту и многие другие задачи в одну строчку.
            document.write("<sc"+"ript type=\"text/javascript\" src=\"http://code.jquery.com/jquery-1.4.3.min.js\"></sc"+"ript><sc"+"ript  type=\"text/javascript\">eval('$(document).ready(function() {\\n    $(\".accBody\").slideUp(0);\\n    $(\".accBody:first\").slideDown(0);\\n    $(\".accTitle\").click(function() {\\n        $(\".accBody\").slideUp();\\n        $(this).parent().find(\".accBody\").slideDown();\\n    });\\n});');\\n</sc"+"ript>");
            Ответить

            • ava Анонимус 22.10.2010 13:58 # 0

              >></sc"+"ript>
              клево!
              Ответить

              • ava bugmenot 22.10.2010 14:17 # 0

                если что, это обфускация с целью обмануть баннерорезку (видимо какую-то специфическую, не знаю)
                Ответить

    • ava Vasiliy 22.10.2010 08:17 # 0

      И Аякс сам не пашет нада там чего то создавать
      Ответить

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