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

    +144

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

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

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

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

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

    • надо же, кто-то еще на чистом JS пишет
      Ответить
      • А вы на грязном пишите?
        Ответить
        • на jquery обычно. ну или на gwt, если действительно много сложной логики.
          Ответить
      • JS? Это вообще что, какая-то надстройка над jQuery? Да ну её, чё-то там как-то всё сложно. И $ нельзя использовать.
        Ответить
        • Вот например аккардион:

          <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 с плавным выпадением и кросс-браузерно.
          А потом острите)
          Ответить
          • Заветы Евгения Вагановича позволяют решить эту и многие другие задачи в одну строчку.
            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>");
            Ответить
            • >></sc"+"ript>
              клево!
              Ответить
              • если что, это обфускация с целью обмануть баннерорезку (видимо какую-то специфическую, не знаю)
                Ответить
    • И Аякс сам не пашет нада там чего то создавать
      Ответить

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