Говнокод #4645 — Куча — Говнокод.ру

Куча / Говнокод #4645

+135
1. 1
2. 2
3. 3
4. 4
5. 5
6. 6
7. 7
```
<p id="entrance">
            Нашли или ...
</p>

<p id="entrance" style="background: #fff; padding: 8px; -webkit-border-radius: 8px; -moz-border-radius: 8px; border-radius: 8px; -webkit-box-shadow: 0 0 5px #aaa">
	Дорогие пользователи
</p>
```
Решил я написать скрипт для GreaseMonkey, который убирает сообщение "Дорогие пользователи, "
И вот что обнаружил: в коде есть два одинаковых айдишника id="entrance".

Используется стиль p#entrance.

Если надо использовать стили в нескольких элементах, используй классы.

А разные айдишники оставь для джаваскрипта,
а то не выбрать определенный элемент с помощью document.getElementById.

В крайнем случае используй атрибут name. Для него есть метод document.getElementsByName

Запостил: Oleg_quadro, 16 Ноября 2010

Tweet
Комментарии (42) RSS
- Oleg_quadro 16.11.2010 13:21 # 0
  
  В стилях специально не копался, это Firebug выдал.
  Ответить
- HyperGeek 16.11.2010 14:54 # −2
  
  А где -moz-box-shadow? Дискриминация =)
  Ответить
- Анонимус 16.11.2010 15:14 # 0
  
  Кэп, нету name у P.
  Ответить
  - Oleg_quadro 16.11.2010 15:27 # −1
    
    а что мешает его добавить?
    Ответить
    - Анонимус 16.11.2010 16:18 # 0
      
      w3c
      Ответить
      
      Oleg_quadro 16.11.2010 16:29 # 0
      
      что тогда вместо него?
      Ответить
      
      Анонимус 16.11.2010 16:35 # 0
      
      атрибут name есть только у полей формы и ссылки. А в xhtml 1.1 его и вообще нет:)
      если хочется обозначить группу P -- используйте класс. На него можно повесится селектором и в css и в jquery.
      Ответить
      
      Oleg_quadro 16.11.2010 16:38 # 0
      
      спасибо. ждём браузерной реализации getElementsByClass
      Ответить
      
      Анонимус 16.11.2010 16:43 # −1
      
      зачем?
      чем плох $("boo.class") ?
      Ответить
      
      Oleg_quadro 16.11.2010 16:51 # 0
      
      просто думаю браузерная реализация будет быстрее.
      Ответить
      
      Анонимус 16.11.2010 16:52 # −1
      
      будет. Но только я не верю, что поиск элемента по классу в современном браузере может быть боттл-неком
      Ответить
      
      Oleg_quadro 16.11.2010 16:58 # 0
      
      дык, на простой страничке не будет.
      А если там какой-нить календарь: 3 года назад, 3 года вперёд, то там всё и выплывает.
      Ответить
      
      Анонимус 16.11.2010 23:13 # −1
      
      если у Вас 999 тегов на странице -- то да
      может
      Ответить
      
      bugmenot 16.11.2010 17:49 # −1
      
      быдло-жкуерями плох
      nodelist = element.getElementByClassName(string) /* же! */
      Ответить
      
      Анонимус 16.11.2010 23:15 # −2
      
      >>же!
      IE не поймет
      Ответить
      
      Oleg_quadro 16.11.2010 23:51 # +1
      
      не только он. тут getElementsByClassName записана без буквы s,
      после Element
      Ответить
      
      istem 17.11.2010 00:06 # +1
      
      Зато там есть /* же! */ - это все поймут
      Ответить
      
      andremacareno 17.11.2010 10:55 # 0
      
      буква "е" лишняя, "опа" не хватает.
      Ответить
      
      bugmenot 17.11.2010 11:06 # 0
      
      глазастый!
      Ответить
      
      bugmenot 17.11.2010 11:08 # 0
      
      IE и прочую эзотерику надо окостылировать
      Ответить
      
      Анонимус 17.11.2010 11:09 # 0
      
      ...что и делает "быдложуквери". )
      
      на жабьем скрипте и так писать противно, еще не хватает вручную обходить все баги браузеров)
      Ответить
      
      bugmenot 17.11.2010 11:13 # 0
      
      нет, быдло-жкуери идут своим особым путём, который с виду похож на querySelectorAll
      Ответить
      
      Анонимус 17.11.2010 11:18 # 0
      
      блин, ну нельзя же убить всех пользователей с ослом?!
      Ответить
      
      bugmenot 17.11.2010 11:31 # +1
      
      ослоцид это возможно и хорошо, но причем тут это?
      Ответить
      
      Анонимус 17.11.2010 11:34 # 0
      
      Потому что querySelectorAll не работал, по крайней мере в семрке.
      
      зы: осел -- это IE. Так-то!
      Ответить
      
      bugmenot 17.11.2010 11:39 # 0
      
      причем здесь замена драфтного getElementsByClassName на не менее драфтное расширение?
      Ответить
      
      Анонимус 17.11.2010 11:44 # 0
      
      Я видимо Вас не понимаю)
      жэквери хорош тем, что в нем я могу получить все элементы по самому извращенному селектору не парясь о своместимости.
      Мой код будет работать в любом, выпущенном за последние 5 лет, браузере. Мне достаточно только подключить 5 килобайт кода.
      
      Конечно было бы здорово, если б такая возможность была у любого HTMLDom, но ее, увы, нет. Приходится юзать костыль (как это часто бывает в веб-разработках).
      Ответить
      
      bugmenot 17.11.2010 11:53 # 0
      
      > все элементы по самому извращенному селектору
      даже если нужен селектор 'body'? :)
      
      > 5 килобайт кода
      ORLY? таки 5? не 4,9?
      Ответить
      
      Анонимус 17.11.2010 11:56 # 0
      
      >>даже если нужен селектор 'body'
      Это врядли)
      
      Я не предлагаю везде и всегда юзать JQuery (не поверите -- я умею писать на чистом JS), но иногда JQuery правда помогает.
      
      Это конечно не оправдывает типовой jquery говнокод, где 400 раз создается одна и та же функция, потому что ее не вынесли в переменную, а анонимно навешивают на click итд..
      
      >>ORLY? таки 5? не 4,9?
      Это Вы за размер сайта боритесь?)
      Ответить
      
      bugmenot 17.11.2010 12:13 # −1
      
      боретесь, и такая же грубая ошибка в размере говножкуерь
      Ответить
      
      Анонимус 17.11.2010 12:18 # +3
      
      Вы нази
      Ответить
      
      Анонимус 17.11.2010 11:21 # 0
      
      OFFTOP: Вы мне чем-то веб-килла напоминаете)
      
      толи аватарой толи стилем беседы
      Ответить
      
      bugmenot 17.11.2010 11:26 # 0
      
      спасибо, попробуйте смазать скипидаром =)
      Ответить
      
      Анонимус 17.11.2010 11:33 # 0
      
      Вы так говорите, как будто это что-то плохое
      Ответить
      
      bugmenot 17.11.2010 11:34 # 0
      
      "Ну разумеется" :-)
      Ответить
- Oleg_quadro 16.11.2010 16:30 # 0
  
  Впрочем, щас подумал:
  Если не обращаться с помощью джаваскрипта к определенным элементам, может и id вместо класса покатит.
  Ответить
  - bugmenot 17.11.2010 11:14 # +1
    
    http://validator.w3.org/check?uri=http%3A%2F%2Fgovnokod.ru%2F464 5&charset=%28detect+automatically%29&doc type=Inline&group=0
    Ответить
    - Oleg_quadro 17.11.2010 12:50 # 0
      
      спасибо.
      Ответить
- Oleg_quadro 16.11.2010 16:52 # 0
  
  Ладно, в этом конкретном случае не говнокод)). Если не обращаемся к элементам с помощью javascript.
  Ответить
  - Arigato 16.11.2010 17:29 # +1
    
    Вообще-то id должен быть уникален, чего нет в примере.
    Ответить
    - Oleg_quadro 16.11.2010 17:34 # 0
      
      Дык а смысл, если мы не обращаемся с помощью javascript?
      или это не по w3c?
      Ответить
      
      Arigato 16.11.2010 17:45 # 0
      
      Ну так на то он и id
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Где здесь C++, guest?!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

Куча / Говнокод #4645

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

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