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

    +151

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    <div id="stat" class="list_menu" onclick="
                        if ($('#stat_sub').is(':hidden')){
                            $('#stat_sub').slideDown('slow');
                        } else {
                            $('#stat_sub').slideUp('slow');
                        }">Статистика</div>

    Тот же автор, что и #4015

    Запостил: scalar4eblo4no, 17 Августа 2010

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

    • ava temotor 17.08.2010 17:13 # −1

      Тут какие-то проблемы с jQuery?
      Ответить

    • ava Eugene 17.08.2010 17:15 # 0

      <div id="stat" class="list_menu" onclick="$('#stat_sub').slideToggle('slo w');">Статистика</div>
      Ответить

      • ava bildja 17.08.2010 17:37 # +1

        <div id="stat" class="list_menu">Статистика</div>

        $('#stat').click(function() {
        $('#stat_sub').slideToggle('slow');
        });
        Ответить

        • ava Eugene 17.08.2010 17:43 # −2 

          <div id="stat" class="list_menu">Статистика</div>

$(function() {
    $('#stat').click(function() {
        $('#stat_sub').slideToggle('slow');
    });
});
          Ответить

          • ava bildja 17.08.2010 17:45 # 0

            А у меня чё-то тег [code] не сработал..
            Я имел в виду, что так события аттачить лучше, чем инлайном.. Но не совсем понял, если честно, зачем $(function() {}), это типа когда дом сформируется?
            А то я пишу $(document).ready(function() {}), может, я не так делаю?
            Ответить

          • ava cheef 18.08.2010 11:13 # −2

            зачем тут ждать document ready?
            Ответить

            • ava Lure Of Chaos 18.08.2010 11:20 # +1

              что бы документ загрузился
              Ответить

              • ava cheef 18.08.2010 11:48 # −2

                спасибо КО ) но я о том, что этот код тут лишний. Поскольку див уже в доме до вызова самого js
                Ответить

                • ava Lure Of Chaos 18.08.2010 14:17 # 0

                  необязательно. Лучше не устраивать гонки, а перестраховаться
                  Ответить

    • ava scalar4eblo4no 17.08.2010 17:17 # +1

      Так мешать JS и верстку - некошерно!
      Ответить

      • ava temotor 17.08.2010 20:28 # 0

        Это бесспорно, конечно. Но хз так уж говнокод ли.

        Интересно, что на Том Самом Сайте [url]http://www.w3.org/TR/REC-html40/interact/scripts.html#events[/url
        экраном ниже есть примерно такой же if в onclick-е. Что, конечно, не оправдывает кучу кода в HTML, просто факт.
        Ответить

    • ava Lure Of Chaos 17.08.2010 21:28 # 0

      вот оно, макака освоила жхуери
      Ответить

    • ava sectus 18.08.2010 03:00 # +4

      Товарищи, примеры такого кода можно выкладывать километрами в режиме реального времени... только стоит ли? Интересен ли этот код с точки зрения посетителя говнокод.ру? Мне нет.
      Ответить

      • ava Lure Of Chaos 18.08.2010 14:20 # 0

        так уже километры этого и есть. Думаю, нужно выкладывать только интересное
        Ответить

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