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

    +151.2

    1. 1
    <body onload="blinkAfterSearch();setTimeout('loadDummyImg()', 590000);var ex; try { startUpTasks(); } catch(ex) {}" />

    Недавно отрыл в недрах нашего проекта! Try catch в onload!! :D

    Запостил: jericho, 02 Марта 2010

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

    • ava Johnny 02.03.2010 10:53 # 0

      А в чем, таки, цимес та?)
      Ответить

      • ava turdman 02.03.2010 11:09 # −3

        В onload. Onload, onclick, href="javascript://" и прочее - такой же некошер как и <font>
        Ответить

        • ava Johnny 02.03.2010 15:05 # 0

          Хмм... К чему бы это все? Не вижу ни font ни href://javascript..
          Ответить

          • ava turdman 02.03.2010 15:45 # −2

            Я к тому что код надо писать в <script>, а потом цеплять через DOM/window, а не размазывать как известно-что по странице.
            Ответить

            • ava guest 02.03.2010 20:46 # +1

              Не нгавица - не ешьте, шо ви таки пгицепились?
              Ответить

    • ava jericho 02.03.2010 10:57 # +1

      Мне кажется, что надо было таки включить этот кусок в js файл, а сюда поставить вызов функции. Разделение между представлением и, грубо говоря, контроллером.
      Ответить

      • ava guest 02.03.2010 11:24 # +4

        Есть такие понятия как "дедлайн", "нужно, чтобы работало" и "здесь и сейчас".
        Ответить

        • ava jericho 02.03.2010 14:06 # 0

          Просто горе программисты писали. Как бы скоро не надо было сдавать проект, выделить функцию обработчика событий(в данном случае) в отдельный файл + подключить его к странице критически не скажется на времени выполнения задачи. А вот поддерживать такой код - не самое приятное занятие.
          Ответить

        • ava turdman 02.03.2010 15:02 # 0

          Эти понятия могут оправдать создателей говнокода, но сам код от этого меньшим говном не становится.
          Ответить

      • ava Johnny 02.03.2010 15:07 # +2

        Не знаю, конечно в плане поддержки может и не оч верно, писать все это внутри тэга body. Но синтаксически все верно. Говнокодность спорна...
        Ответить

    • ava murz 02.03.2010 15:38 # +1

      setTimeout('loadDummyImg()', 590000);

      Любопытно, зачем поставлена задержка почти на 10 минут?
      Ответить

      • ava jericho 03.03.2010 09:53 # 0

        В урле содержится параметр dummy=382618136. Это функция обновляет параметр, что бы страница не кешировалась. Я так понял, после этого страница обновляется.
        Ответить

    • ava mrbig66 04.03.2010 16:26 # 0

      Блинк для слоупоков.
      Ответить

      • ava guest 07.03.2010 11:39 # 0

        блинк сразу, это картинка с таймаутом
        Ответить

    • ava guest 04.03.2010 22:10 # −1

      mrbig66, проясни незнающему, что за загадочный слоупок? :)
      Ответить

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