Говнокод

(jQuery.noConflict()(function ($) {
  "use strict";
  $(document).on('ready', function () {
    $('.sisea-results-list').each(function () {
      var elements = $(this).find('.sisea-result'),
          size     = elements.size(),
          find     = 10;
      for (var i = 0; i < find; i++) elements.eq(i).show(300);
      $(window).on('scroll', function() {
        var scrollTop = window.pageYOffset || document.documentElement.scrollTop,
            offset    = $('#scroll').offset().top;

        if (scrollTop > offset - 800) {
          find < size ? find += 10 : find = size;
          for (var i = 0; i < find; i++) elements.eq(i).show(300);
        }
      });
    });
  });
}));

При скролле элементы появляются на странице. Говнокод ли?

Запостил:

wayfarer, 20.07.2015 (Updated 29.03.2018)

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

3_14dar 11 лет ago # 0
```
}
      });
    });
  });
}));
```
Ответить
guest 11 лет ago # 0

Это не говнокод. Это говноязык.
Ответить
- kegdan 11 лет ago # 0 ⇈
  
  Это говногость
  Ответить
1024-- 11 лет ago # 0

В 15й строке говнокод, а в 8й - говнопринцип.
Ответить
- wayfarer 11 лет ago # 0 ⇈
  
  А как иначе то?
  Ответить
  - 1024-- 11 лет ago # 0 ⇈
    
    В 18 строке не показывать показанное.
    В говнопринцип - пересмотреть и найти удобное решение (оставить так, если это не говномода, а действительно удобно; страницы по N элементов; одна страница со всеми показанными элементами; что-то другое)
    Ответить
    - wayfarer 11 лет ago # 0 ⇈
      
      Все элементы изначально скрыты. Мне нужно показывать 10 элементов, а может 5 элементов, а может 15 сразу же. Не понимаю, что тебе не нравится.
      Ответить
      - 1024-- 11 лет ago # 0 ⇈
        
        1. Возьмём за X значение find в строке 13. Когда выполняется строка 15, открыто X элементов и их зачем-то переоткрывают. Не беда, но чуть грустно.
        2. Мне не нравится сама идея надругательства над прокруткой. Ещё с тех пор, как люди начали при прокрутке автоматически что-то подгружать и увеличивать страницу.
        Ответить
        
        wayfarer 11 лет ago # 0 ⇈
        
        Согласен.
        Ответить