Дмитрий

Вот так его можно переписать

// Просто храним все значения left в массиве
const switchActiveStyles = [
  [0, 1000, 2000, 3000, 4000],
  [-1000, 0, 1000, 2000, 3000],
  // и т.д.
]

const activeSwitcherClassName = 'active-switcher'

function activeSwitcherFinder() {
  // Перебор не нужен, активный максимум один
  const active = arr_btn.find(({ classList }) => classList.contains(activeSwitcherClassName)) 
  if (active) active.remove(activeSwitcherClassName)
}

// Найти все можем в цикле
const arr_btn = Array.from([,,,,,], (_, ind) => document.querySelector('.mod-' + ind))
const arr_wraps = Array.from([,,,,,], (_, ind) => document.querySelector('.display-wrap_' + ind))

arr_btn.forEach((btn, ind) => {
  btn.onclick = () => {
    activeSwitcherFinder()
    btn.classList.add('active-switcher')

    // Получаем соответствующий набор значений для left И устанавливаем в цикле
    switchActiveStyles[ind].forEach((style, ind) => {
      arr_wraps[ind].style.left = style + 'px'
    })
  }
})

map это функция, она должна быть вызвана, вы же делаете ей присвоение. К тому же если она ничего не возвращает, использовать ее бессмысленно, используйте вместо нее forEach

strings.forEach(element => {
  console.log(element);

  let sel = element.querySelector('.selector');
  if (sel.value === '0') {
    error [element.name] = sel.name;
  }
});

Большинство программистов, в том числе те, кто будет вас собеседовать - как мартышки, им сказали что правильно так и так, они не задумываясь передают это дальше и бьют по голове каждого, кто осмеливается делать не по шаблону. Вся эта возня с экшинами и типами безусловно избыточна в редаксе в 95% случаев. Я вообще делал надстройку, чтобы напрямую вызывались процедуры, изменяющие стейт. Хотя не рискнул бы выложить это в качестве портфолио, по упомянутым выше причинам.

В самом вашем коде ничего плохого нет, вы можете оставить его, в комментариях пояснив, что вынос экшинов в отдельный файл в данном случае ведет к избыточному бойлерплейту. Единственное все же нагляднее, если mapStateToProps и MapDispatchToProps вынесены на верх, а не вставлены в коннект непосредственно.

Не вижу смысла выделять city ни в ключ, ни в отдельный параметр. В чем смысл? Сделайте плоскую таблицу

this.cities1 = [
        { city: 'Москва', cityCode: 'moskva54', views: '53М', visites: '1,1M', isActive: true  },
        { city: 'Владимир', cityCode: 'vladimir', views: '23М', visites: '1,2M', isActive: false }
];

Либо вообще вместо массива объект:

this.cities1 = {
  'Москва': { city: 'Москва', cityCode: 'moskva54', views: '53М', visites: '1,1M', isActive: true  },
  'Владимир': { city: 'Владимир', cityCode: 'vladimir', views: '23М', visites: '1,2M', isActive: false }
};

В этом случае вы моментально получаете данные по городу или по нескольким, так как названия индексированы.. Если же вам нужен список, вы просто получаете Object.keys(this.cities1) или Object.values(this.cities1) если вместо со всеми данными

Если корректировать ваш вариант, то нужно спредить получаемый из func результат

if (typeof elem == 'object') {
      res.push(...func(elem));
    } else {
      res.push(elem);
    }

но есть еще функция flat у массивов, можно использовать ее

elem.flat(Infinity)

document.querySelectorAll(".m1")
  .forEach(el => el.addEventListener("click",  function(e){ 
    this.style.backgroundColor = "red";
  }));

Потому что принято записывать значения 8-битовыми порциями, просто соглашение

Это поведение и вправду очень странное, но корректируется третьим параметром, выставленным в TRUE

if (in_array($el, $arr, TRUE))

Переименуйте переменную внутреннего цикла а лучше пользуйтесь современным синтаксисом const или let

Я когда учился, делал универсальную таблицу с возможностью группировки, отбора, сортировки строк, подсчетом итоговых значений. Это очень немаленький проект и хорошо прокачивает. Естественно без использования готовых ui компонентов

Потому что каждый раз при нажатии на первую кнопку добавляется listener. Он не перезаписывается. Соответственно при нажатии на вторую кнопку срабатывают все ранее подключенные нажатием на первую кнопку listener-ы. И поскольку переменная a объявлена внутри функции, то увеличиваются все a внутри всех ранее подключенных слушателей.

Анализируя атрибут name

const maxNumber = Math.max(...Array
  .from(document.querySelectorAll('input.form-control.quantity'))
  .map(el => Number(el.name.split(/\[|\]/g)[1] || 0))
)

Очевидно, что для какого-то элемента строчка
const member = guild.members.get(i);

возвращает undefined

Поставьте брейкпойнт с условием в этом месте и посмотрите на каком элементе. Очевидно что где-то у guild.members элементов меньше чем в массиве bot.muted