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

    +166

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    $('#protesters-open').click(function() {
	if ($('#protesters').hasClass('hidden'))
	{
		$('#protesters').removeClass('hidden');
	}
	else
	{
		$('#protesters').addClass('hidden');
	}
});

    http://nsk.2gis.ru/media/js/main.js
    вроде серьезная компания, а такого говна там навалом
    раз уж используется jQuery, то можно и toggleClass использовать

    Запостил: Back, 31 Мая 2011

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

    • ava Vasiliy 31.05.2011 10:38 # +2

      Я так подозреваю в css .hidden { dispaly:none;} можно просто toggle за юзать.
      Ответить

      • ava Back 31.05.2011 10:54 # 0

        да, совершенно верно. там где-то есть более сложные примеры у них в коде, поэтому я обобщил :)
        Ответить

    • ava Lure Of Chaos 31.05.2011 12:49 # +3

      они даже jQuery в достаточной мере не могут осилить.
      Ответить

    • ava bugmenot 31.05.2011 13:08 # 0 

      $('*[id=protesters]')
      Ответить

      • ava Back 31.05.2011 13:10 # +2

        надеюсь, это шутка :)
        Ответить

        • ava bugmenot 31.05.2011 17:02 # 0

          http://govnokod.ru/5690
          Ответить

          • ava Back 31.05.2011 17:28 # 0

            как я понимаю, $(...) юзает нативный querySelectorAll, что очень быстро. но насколько он необходим в запросе такого вида? не подразумевается ли использование выборки по id наличие всего лишь одного элемента с таким id? тогда $("#protesters") должен быть быстрее. или я что-то не так понял?
            Ответить

            • ava bugmenot 01.06.2011 15:12 # 0

              хуй его знает,
              мне не настолько интересно, чтобы в сырцы быдложкуери лазить
              (хотя говна там навалом, гарантирую)
              Ответить

    • ava jQuery 01.06.2011 14:32 # +1

      // Интерактивный список
      $('.int-li .name span').click(function() {

      $('.int-li .item').removeClass('current');

      if ($(this).parent().parent().hasClass('cur rent'))
      {
      $(this).parent().parent().removeClass('c urrent');
      }
      else
      {
      $(this).parent().parent().addClass('curr ent');
      }
      });


      нашел там же вариант по жестче
      Ответить

      • ava bugmenot 01.06.2011 15:13 # 0

        use [code], luke
        Ответить

      • ava Back 01.06.2011 17:06 # 0

        да, я уже не стал третий раз постить :) но оно там все безусловно есть. причем, упорно не хотят исправлять
        Ответить

        • ava jQuery 02.06.2011 01:18 # 0

          принцип русского программиста - не трогай, пока работает :D
          Ответить

          • ava Back 02.06.2011 03:38 # 0

            ага, "обожаю" этот принцип. особенно когда проект принимаешь :)
            Ответить

    • ava guest8 09.04.2019 17:59 # −999

      показать все, что скрытоvanished
      Ответить

    • ava guest8 09.04.2019 18:01 # −999

      показать все, что скрытоvanished
      Ответить

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