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

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

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

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

+162

var titleAlt = String(dbField("title").value);
if (titleAlt.toLowerCase().indexOf("<b>") != -1) { titleAlt = titleAlt.replace(/<b>/gi, "").replace(/<\/b>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<i>") != -1) { titleAlt = titleAlt.replace(/<i>/gi, "").replace(/<\/i>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<small>") != -1) { titleAlt = titleAlt.replace(/<small>/gi, "").replace(/<\/small>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<br>") != -1) { titleAlt = titleAlt.replace(/<br>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<p>") != -1) { titleAlt = titleAlt.replace(/<p>/gi, "").replace(/<\/p>/gi, ""); }
f = 0;
while (f != -1) {
    f = titleAlt.indexOf("<img ");
    if (f == -1) { f = titleAlt.indexOf("< img "); }
    if (f == -1) { f = titleAlt.indexOf("<  img "); }
    if (f == -1) { f = titleAlt.indexOf("<IMG "); }
    if (f == -1) { f = titleAlt.indexOf("< IMG "); }
    if (f == -1) { f = titleAlt.indexOf("<  IMG "); }
    if (f != -1) {
        t = -1;
        for (i = f; i < titleAlt.length; i++) {
            if (titleAlt.substr(i, 1) == ">") { t = i; i = titleAlt.length; }
        }
        if (t != -1) {
            ttl = titleAlt;
            titleAlt = ttl.substring(0, f) + ttl.substring(t + 1, titleAlt.length);
        }
    }
}

это не что иное, как попытка очистить строку от html
код как бы говорит нам - "регулярные выражения - зло!"
цикл while - это просто шедевр зодчества эпохи просвещения. он ищет тег img и закрывающую скобку, и если они есть - присваивает результату подстроку от начала строки до первого вхождения img и от последнего вхождения угловой скобки

найдено в серверном JavaScripte на одном из зарубежных сайтов

Запостил:

mozg_raka, 01 Июля 2010

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

Lure Of Chaos 01.07.2010 19:44 # +4

предположение кейса и пробела между уголком и именем тега - это пять...
между прочим <IMg прокатит
Ответить
inkanus-gray 01.07.2010 19:56 # 0

А что там в строках 02-06, жутко похожее на регулярки?
И что будет, если между уголком и IMG три пробела?
Ответить
- Lure Of Chaos 01.07.2010 19:58 # +2
  
  это и есть замена по регулярке. открывающий на закрывающий
  а бровсер поймет эти пробелы?
  Ответить
  - inkanus-gray 01.07.2010 20:00 # +2
    
    Даже один пробел не понимает. Крутой алгоритм. От нас что-то скрывают.
    Ответить
    - Lure Of Chaos 01.07.2010 20:12 # +2
      
      и мне непонятно почему не f = titleAlt.toLowerCase().indexOf("<img ") как выше
      Ответить
Анонимус 01.07.2010 20:10 # +2
Я бы улучшил:
```
if (f == -1) { f = titleAlt.indexOf("< IMG "); }
if (f == -1) { f = titleAlt.indexOf("<  IMG "); }
if (f == -1) { f = titleAlt.indexOf("<   IMG "); }
if (f == -1) { f = titleAlt.indexOf("<    IMG "); }
if (f == -1) { f = titleAlt.indexOf("<     IMG "); }
if (f == -1) { f = titleAlt.indexOf("<      IMG "); }
if (f == -1) { f = titleAlt.indexOf("<       IMG "); }
if (f == -1) { f = titleAlt.indexOf("<        IMG "); }
```
Ответить
- This is obvious 01.07.2010 20:30 # 0
  
  боян, такое уже было :))
  Ответить
  - Lure Of Chaos 01.07.2010 20:34 # 0
    
    все когда то уже было, и это очевидно
    Ответить
    - This is obvious 01.07.2010 20:37 # +2
      
      Напомнило ранее встречавшийся дем: «Та ты что!? А не пиздишь?»
      Ответить
      - Lure Of Chaos 01.07.2010 20:55 # 0
        
        думаю пора начать новую манеру - фотоответов в стиле "фигвам" )
        Ответить
        
        telnet 01.07.2010 21:12 # 0
        
        Ага, попросим у Страйкера аплоад пикч и сажу. И будет у нас неймфажный имиджборд "Говнокодч".
        Ответить
        
        This is obvious 01.07.2010 21:21 # 0
        
        и анонов обратно разрешим, начнётся тут трололо )
        Ответить
        
        inkanus-gray 01.07.2010 21:32 # 0
        
        Говнокод для анонов есть: http://govnokod.com/
        Только почему-то комментариев я там не вижу.
        Ответить
        
        This is obvious 01.07.2010 22:23 # +4
        
        FFFFFFFFFUUUUUUUUUUUU-
        ацтой какой-то ) там даже аноны комментить не хотят )
        Ответить
        
        Анонимус 01.07.2010 22:26 # +1
        
        да нет, есть там и наши: http://govnokod.com/2852
        Ответить
        
        inkanus-gray 01.07.2010 22:56 # 0
        
        Вот это внимательность! Я этого комментария не заметил. Есть гипотезы, кто его мог наложить?
        Ответить
        
        bugmenot 03.07.2010 03:25 # 0
        
        Хех, эти долбоёбы используют govnokod.com в качестве pastebin`а
        Пруф: irc://irc.san.ru/php
        Ответить
        
        Lure Of Chaos 02.07.2010 13:59 # 0
        
        не надо аплоуд пикч, вполне какой нить радикал.ру сойдет = )
        Ответить
        
        This is obvious 02.07.2010 17:06 # +1
        
        и прийдёт какой-то хуй анонимус и запостит тут картинку кота которая растягивается на 100 км. :\
        Ответить
        
        Говногость 02.07.2010 17:30 # 0
        
        Большинство PHP прогеров считают, что пускай постит, а админ забанит.
        Ответить
        
        Lure Of Chaos 02.07.2010 18:36 # +2
        
        в связи с новым законом о СМИ в интернете, комментарии гк.ру сделать премодерируемыми, гы )
        Ответить
        
        inkanus-gray 02.07.2010 20:04 # 0
        
        Сколько ж тогда модераторов потребуется?
        Ответить
        
        This is obvious 02.07.2010 20:32 # 0
        
        На каждого анона 2-3 :))
        Ответить
        
        inkanus-gray 02.07.2010 20:43 # 0
        
        Тогда каждого зарегистрированного нужно принудительно избрать в модераторы, чтобы мы модерировали реплики друг друга.
        Тогда сама Очевидность будет замодерирована и модерировать одновременно!
        Ответить
        
        Lure Of Chaos 03.07.2010 00:39 # +2
        
        если модер один, то кол-во комментов заметно сократится, зато будут 10 раз думать, прежде чем запостить )
        Ответить

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

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

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

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