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

    +167.7

    1. 1
    $('div[id="myid"]')

    Запостил: fuckyounoob, 14 Ноября 2009

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

    • Тоже неплохо )
      Ответить
    • даааа, скоро для таких придется в фреймворки оптимизирующий парсер селекторов писать %)
      Ответить
      • Ну SQL же оптимизирует запросы к БД, так почему бы не оптимизировать и запросы к DOM? ;)
        Ответить
    • может он просто забыл поставить ^ после "id"
      Ответить
    • лол
      Ответить
    • блен! плюсанул!
      Ответить
    • Идиотизм. Просто человек не знает нормально css-селекторы, так сразу говнокод?
      Ответить
      • А вот похоже и автор говнокода ;)
        Ответить
      • > Просто человек не знает нормально css-селекторы, так сразу говнокод?
        Просто человек не знает язык, так сразу говнокод?
        Просто человек не умеет проектировать, так сразу говнокод?

        Идиотизм - это ваша фраза, а не то, что код по праву назван говнокодом.
        Ответить
      • Здесь не принято судить о людях... Настоящим гикам интересней обсуждать код, а не существ, его испражнивших.
        Ответить
        • Принято. Здешних тян и олежку охотно обсуждют.
          Ответить
          • Я имею в виду, что здесь не принято обвинять в чём-то конкретных людей, написавших код в топике. Особенно порицается явное наличие имени автора кода. Особенно, если это лаба.
            Все учатся, у всех случаются ошибки.

            А то, что некоторые люди здесь обсуждают друг друга - это их проблемы. Я этому не особо рад.
            Ответить
    • ну если допустить что там было что-то вроде $('div[id="myid" class="test"]')
      то ещё понятно. сократил и забыл... ну а так, хуле, бывает ))
      Ответить
      • А $('#myid.test') чем не устраивает?
        Ответить
        • адепты jquery по религиозным соображениям, отрицают данный селектор :)
          Ответить
      • тут не нужен ни div, ни проверка атрибутов
        нужно только $( '#myid' ), ибо айдишники должны быть уникальны, а если они уникальны, то проверять что-то еще бессмысленно
        Ответить
    • я знаю чем этот код отличается от обычного - он достает все элементы с id, а обычный только один.
      Ответить
      • а как бы по стандарту - id в документе должны быть уникальны
        Ответить
        • нужно быть ко всему готовым
          Ответить
        • Один скрипт может обрабатывать несколько страниц. Допустим, на одной будет див с id=myid, а на другой span или другой тег, но обрабатывать нужно только див. Да, говно, потому что айдишник можно было раньше продумать. Но что делать, если уже́ так сверстали? Отдельный js для каждой страницы может оказаться нерациональным.

          Хотя с тем, что можно было бы и сократить до $('div#myid'), согласен.
          Ответить
          • Если id не уникален, то
            $('div#myid') - не работает в старых IE. Даже несмотря на jQuery там будет только первый элемент :D
            $('div[id="myid"]') - а так - все :)
            Ответить
            • А про неуникальность id я и не говорил. По идее страниц, содержащих неуникальные id, быть не должно... но увы, случается, если по какой-то причине элемент страницы размножили, поэтому за замечание спасибо. Нужно проверить.
              Ответить
            • > Если id не уникален, то...
              ... верстальщик мудак.
              Ответить
    • Кэпы, скажите, чем это хуже div#myid, кроме вида?
      Ответить
      • Имхо, если верстал не мудак, достаточно просто #myid без div. Айдишки же уникальны.

        > чем это хуже div#myid, кроме вида?
        Видимо этим: http://govnokod.ru/2141#comment195180
        Ответить

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