1. JavaScript / Говнокод #18499

    +142

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    (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, 21 Июля 2015

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

    • ava 3_14dar 21.07.2015 04:47 # +2 

      }
      });
    });
  });
}));
      Ответить

    • ava guest 21.07.2015 09:54 # −4

      Это не говнокод. Это говноязык.
      Ответить

    • ava 1024-- 21.07.2015 11:46 # 0

      В 15й строке говнокод, а в 8й - говнопринцип.
      Ответить

      • ava wayfarer 21.07.2015 11:47 # 0

        А как иначе то?
        Ответить

        • ava 1024-- 21.07.2015 11:59 # 0

          В 18 строке не показывать показанное.
          В говнопринцип - пересмотреть и найти удобное решение (оставить так, если это не говномода, а действительно удобно; страницы по N элементов; одна страница со всеми показанными элементами; что-то другое)
          Ответить

          • ava wayfarer 21.07.2015 12:15 # 0

            Все элементы изначально скрыты. Мне нужно показывать 10 элементов, а может 5 элементов, а может 15 сразу же. Не понимаю, что тебе не нравится.
            Ответить

            • ava 1024-- 21.07.2015 12:27 # 0

              1. Возьмём за X значение find в строке 13. Когда выполняется строка 15, открыто X элементов и их зачем-то переоткрывают. Не беда, но чуть грустно.
              2. Мне не нравится сама идея надругательства над прокруткой. Ещё с тех пор, как люди начали при прокрутке автоматически что-то подгружать и увеличивать страницу.
              Ответить

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