Говнокод #7990 — JavaScript

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

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

JavaScript / Говнокод #7990

+159

// ==UserScript==
// @name	govnokod.ru new comments mark
// @namespace	('__'X)
// @description	govnokod.ru new comments mark for Ctrl+F
// @include	http://*.govnokod.ru/*
// @include	http://govnokod.ru/*
// ==/UserScript==

function addNewMark()
{
var	i, d = document.getElementsByTagName('div');
	for (i in d) if (d[i].className == 'entry-comment-wrapper new')
	{
		d[i].getElementsByTagName('p')[0].getElementsByTagName('a')[1].innerHTML += 'new';
	}
}

if (location.hostname.match(/govnokod\.ru$/))
{
	if (document.addEventListener)
	{
		document.addEventListener('DOMContentLoaded', addNewMark(), false);
	} else {
		window.addEventListener('load', addNewMark(), false);
	}
}
//* OR
//* 1) Copypaste into browser @field: javascript:var i,d=document.getElementsByTagName('div');for(i in d)if(d[i].className=='entry-comment-wrapper new')d[i].getElementsByTagName('p')[0].getElementsByTagName('a')[1].innerHTML+='new';void(0);
//* 2) Ctrl+F for '#new'

Накидал только что. Помечает новые посты меткой "#new", можно контролэфить.
Работает только для страниц отдельных ГК, в том числе из мусорки.
Треды до полсотни комментов можно и так оглядеть, если больше - открываю отдельно.
Но всё равно стоит добавить фичу на стороне сервера.

Запостил:

ej, 26 Сентября 2011

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

alexoy 26.09.2011 22:21 # −4

Не нужно. Минус
Ответить
- Lure Of Chaos 26.09.2011 22:46 # 0
  а что тебе нужно? Бледину?
  (вот контекст, а то не поймете)
  
  http://www.youtube.com/watch?v=vbrzk2PmUFs&feature=player_detailpage#t=25s
  Ответить
  - TheHamstertamer 27.09.2011 13:48 # +1
    
    Я сразу понял)
    Ответить
istem 26.09.2011 23:25 # 0

Нужно ещё пару рубрик:
PHP.Даты; и
Говнокод для Говнокода.
Ответить
ZX_Spectrum 27.09.2011 01:28 # 0

насколько я понял, это будет работать только для отдельных страниц с говнокодами, а в общей ленте - нет?
Ответить
bugmenot 27.09.2011 01:51 # +3

бля, опять for ... in
Ответить
- Nemoden 27.09.2011 10:15 # −1
  
  и что с for...in'ом не так?
  Ответить
  - bugmenot 27.09.2011 13:10 # +3
    
    бля, опять спрашивают, что не так с for ... in :-D
    энумерация пропертей же!
    Ответить
    - ej 27.09.2011 19:09 # 0
      
      Нагуглил http://stackoverflow.com/questions/242841/javascript-foreach-vs-for, переписал так:
      
      var d = document.getElementsByTagName('div'), i = d.length; for (; i--;) //... document.addEventListener ? document.addEventListener('DOMContentLoaded', addNewMark(), false) : window.addEventListener('load', addNewMark(), false);
      Ответить
      - ej 27.09.2011 19:30 # 0
        
        Поправка: while (i--)
        Ответить
        
        SmackMyBitchUp 27.09.2011 20:04 # 0
        
        Да, даже если и for, то for(;;i--)
        Ответить
        
        ej 27.09.2011 20:14 # 0
        
        Без условия?
        Ответить
        
        istem 27.09.2011 20:24 # +1
        
        Слушай, а почему
        
        window.addEventListener('load', addNewMark(), false);
        
        конструкция вида addNewMark() не вернёт ли свой результат?
        (а так как она ничё не возвращает - не получим ли мы фигу?)
        --
        (упс, там ниже уже написали... недоскроллил...)
        Ответить
      - bugmenot 27.09.2011 20:39 # 0
        
        в общем-то там есть и forEach есть, непринципиально как итерировать NodeList
        
        хотя тоже проще бы инъекцию стиля сделать и сделать класс new более заметным (я уже страйкера уговорил и он сделал чуть менее пастельно :)
        Ответить
Teddy_Brown 27.09.2011 11:28 # −1

Какое ужасное (на мой взгляд) оформление кода.
Ответить
- ej 27.09.2011 16:12 # 0
  
  А я плюсану. Я вообще не программист. Оно работает и пригодилось, но если кому не лень, поправляйте.
  Ответить
wvxvw 27.09.2011 14:23 # +2

А как оно так вообще работает? addEventListener'у нужна ссылка на функцию, а вы туда undefined передали.
Ответить
- ej 27.09.2011 16:19 # 0
  
  Как юзерскрипт в Опере. Ошибок в логе нет. Лучше без скобок в отличие от function(){}?
  Ответить
  - ej 27.09.2011 18:47 # 0
    
    Нет, без скобок не работает.
    Ответить
  - wvxvw 28.09.2011 03:20 # 0
    Очевидно в Опере очень шаловливыми руками писали интерпретатор...
    
    window.addEventListener("click", alert, false); window.addEventListener("click", alert("whatever"), false);
    
    С какой стати у вас работает - я не знаю.
    Ответить
    - bugmenot 28.09.2011 06:03 # 0
      
      в опере тоже не работает
      и типа не должно
      во второй строка eval забыли
      Ответить
    - ej 28.09.2011 14:17 # 0
      
      Opera v11.10.2092 (последующие - fffuuu).
      
      window.addEventListener("click", alert, false); //* при каждом клике выводит текст "[object MouseEvent]" window.addEventListener("click", alert("whatever"), false); //* вывело "whatever" при вводе скрипта в адрес и всё
      
      Сабж без скобок таки работает, если оставить только
      
      window.addEventListener('load', addNewMark, false);
      
      безо всяких условий и документов, которые ничему не помогают, насколько я вижу.
      Ответить
Teddy_Brown 27.09.2011 14:48 # −4

... и почему не JQuery? <img src='fat_green.jpg'>

upd 201190271549 :
кому-то режет глаз мой камент по поводу оформления кода? ОЛОЛО!
Ответить
- SmackMyBitchUp 27.09.2011 15:16 # +1
  
  По-моему, в большей степени вам режет глаз оценка комментария. Кармадрочите?
  Ответить
  - Teddy_Brown 27.09.2011 15:17 # −1
    
    А то!
    Ответить
SmackMyBitchUp 28.09.2011 15:31 # 0

Хотел бы я подсветку новых комментариев на полосе прокрутки. Как поиск на странице в хроме. Было бы удобнее. Длинные треды приходится просто листать =(
Ответить
guest8 09.04.2019 11:36 # −999

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

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

JavaScript / Говнокод #7990

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

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