[kostyanet]

old = this; в этой строке храним нажатый

Да потому что поддались соблазну искорежить flow в угоду ЧСВ.

говнокод где надо думать что к чему, хотя думать тут не о чем:

old && old.classList.remove("active");
old = this;
old.classList.add("active");

код, где думать не о чем, поскольку все самоочевидно:

this.classList.add("active");
if(old)
  old.classList.remove("active");
old=this;

все потому что по-нашему нельзя чтобы все было просто и понятно - нация не оценит; по-нашему надо все простое и понятное превратить в замудреное гавно, чтоб оно плющило и таращило, чтобы такие как ТС задавали "глупые" вопросы что такое и откуда old.

Поэтому у вас дохуя очков, а у меня нихуя - вы протагонист национальной традиции.