Дайте оценку на сколько этот код ужасен?

Question

[Андрей]

В ходе решения задачи, а именно надо было при прокрутке до элемента подсветить пункт меню(якорь) которому он соответствует.
В ходе обдумывания, пришло в голову такое решение, не обращайте внимание на смешивание, интересно на сколько эта логика правильна.

jQuery('section').each(function () {
    var i, offset=[], offsetId=[];
      offset[i] = jQuery(this).offset().top;
      offsetId[i] = jQuery(this).attr('id');
     var elemSec ={};
     elemSec = {
       id: offsetId[i],
      act: offset[i]
      };
  window.addEventListener('scroll', function() {
       if (pageYOffset > elemSec.act){
         console.log(elemSec.id);
       }
    });
      });

Answer 1

[Programiker]

ужас код.

Answer 2

[necrodeflorator]

Я только проснулся, сейчас утро и возможно я не замечаю чего-то. Но где значение i? Она же undefined, у тебя

Ofset, OfsetId зачем тут? Использовал один раз для присваивания свойства объекта и всё?

Offset пишется правильно так.

Я полагаю ты делаешь что-то подобное?
https://jdevelop.info/articles/html-css-js/577-pla...

Answer 3

[Pashok Doom]

Мешанина. И ofset - очень режет глаз, уж или пиши offset или сразу smeshenie