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

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

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

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

+158

if (whole_text.toLowerCase().search(new RegExp(search_phrase, 'i')) < 0) {
    $(this).css('display', 'none');
} else {
    $(this).css('display', 'block');
}

Запостил:

striker, 19 Октября 2011

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

Teddy_Brown 19.10.2011 09:52 # −2

и что вас смущает?
Ответить
- roman-kashitsyn 19.10.2011 10:15 # +6
  
  сначала всё к lower case, а потом регэксп с кейс игнором
  Ответить
  - Teddy_Brown 19.10.2011 11:35 # −3
    
    эм... перестраховка - это нормально.
    к тому же search_phrase не лоукейсится.
    Ответить
    - roman-kashitsyn 19.10.2011 12:14 # −1
      
      > перестраховка - это нормально
      А вот лишний расход памяти - это не нормально
      Ответить
      - Teddy_Brown 19.10.2011 12:37 # −1
        
        перерасход, конечно, плохо. особенно в этой ситуации. ведь лучше избежать перерасхода памяти любыми средствами. даже если это приведет к возможной нестабильности. :):
        Ответить
        
        AnimeGovno-_- 19.10.2011 12:42 # −4
        
        :):
        WTF?
        Ответить
        
        Teddy_Brown 19.10.2011 13:33 # +5
        
        смайл Шредингера
        Ответить
        
        roman-kashitsyn 19.10.2011 15:35 # 0
        
        Если whole_text - пара страниц, то разница существенна.
        Ответить
        
        Teddy_Brown 19.10.2011 15:39 # +1
        
        если whole_text - пара страниц и выполнение этого места может серьезно сказаться на производительности -- у вас проблема с архитектурой в целом :(:
        Ответить
        
        roman-kashitsyn 19.10.2011 15:49 # 0
        
        Видимо, в слово "архитектура" вы вкладываете какой-то известный только вам смысл
        Ответить
        
        Teddy_Brown 19.10.2011 16:13 # 0
        
        я про архитектуру приложения, которрая допускает такие узкие места.
        Ответить
AnimeGovno-_- 19.10.2011 10:38 # −9

показать все, что скрытов бан!
Ответить
- kindofbear 19.10.2011 11:51 # 0
  
  компрессинг
  Ответить
- Lure Of Chaos 19.10.2011 18:32 # +1
  
  все срочно в баню!
  Ответить
wvxvw 19.10.2011 15:52 # +1
Мне так кажется, что таким образом кто-то пытался добавить удобства при чтении здешних комментариев. Хотя, конечно, сделано в духе времени.
```
$(this).css('display', 
    new RegExp(search_phrase, 'i').test(whole_text) ? 
    'block' : 'none');
```
если что.
Ответить
- Teddy_Brown 19.10.2011 16:14 # 0
  
  прозреваете говноверсию куклоскрипта?
  Ответить
guest8 09.04.2019 11:46 # −999

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

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

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

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

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