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

    +175

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    function showOrHide(id) {
if(jQuery('#'+id).is(':hidden')) {
jQuery('#'+id).show();
} else {
jQuery('#'+id).hide();
}
}

    rutube.ru, jQuery().toggle()

    Запостил: LoserJS, 26 Августа 2011

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

    • ava LoserJS 26.08.2011 00:28 # 0

      И в остальном там столько говнокода
      Ответить

    • ava guest 26.08.2011 03:28 # −2

      А где говнокод-то?
      Ответить

      • ava inkanus-gray 26.08.2011 03:49 # +10

        Наверное в том, что функция эквивалентна одной строке: jQuery('#'+id).toggle()
        Если уж использовать библиотеку, то зачем изобретать велосипед?
        Ответить

    • ava striker 26.08.2011 04:03 # +8

      Пришел из rss плюсануть этот пост
      Ответить

    • ava guest 26.08.2011 05:21 # +3

      Страшен даже не сам велосипед, а двойной вызов поиска элемента по id.
      Ответить

    • ava Eyeless 26.08.2011 05:23 # +1

      Кстати, если id еще и точку будет содержать, оно вообще не сработает.
      Ответить

      • ava striker 26.08.2011 06:07 # 0

        Хм. Почему? Баг jquery?
        Ответить

        • ava Eyeless 26.08.2011 06:11 # +5

          Не баг, а вполне документированная особенность селекторов.
          #name.surname выбирает один элемент с id=name и имеющий класс surname. Чтобы этого не случалось, точку надо экранировать.
          http://api.jquery.com/category/selectors/
          Ответить

      • ava Serabas 15.04.2013 11:38 # +1

        и не только точку
        Ответить

    • ava jQuery 28.08.2011 21:24 # +2

      function showORhide(obj) {
      (obj.is(':hidden') ? obj.show() : obj.hide())
      }

      showORhide($('#xuj'));

      упрощаем говно
      Ответить

    • ava guest 06.09.2011 14:41 # 0

      they are can`t into .toggle()
      Ответить

    • ava Teddy_Brown 09.09.2011 13:55 # +1

      есть мнение что в архитектуре приложения предполагается (в последствии) обрабатывать события показа/сокрытия элемента дополнительными свистелками и перделками.
      Ответить

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