Почему не работает map в JavaScript?

Question

[rapidum_alder]

Хочу сделать проверку формы до отправки.
(Слышал есть встроенный метод, но он не подходит т.к. у меня есть кнопка промежуточное сохранение и постоянная)
Когда пользователь нажимает на постоянной сохранение, надо проверить все селекторы в форме на значение на равное нулю.
Но когда я map'ом прохожу по всем элементом он ни чего не возвращает.

document.addEventListener("DOMContentLoaded", () => {
    let form = document.getElementById('MainForm');
    let btn = document.getElementById('btnPass');
    if (form) {
        let error = [];
        btn.addEventListener('click', event => {
            event.preventDefault();
            let strings = Array.from(form.querySelectorAll('.table_item'));
            console.log(strings);

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

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

В итоге console.log(strings) у меня отображает массив из строк таблицы.
Array(3) [ tr.table_item, tr.table_item, tr.table_item ]
а console.log(element); у меня пустота. т.е. даже не вызывается.

Пробовал вместо .map .forEach и менял стрелочную функцию на обычную.
strings.map = (function(element) {

Comments

[RAX7]

strings.map = ((element) => {
strings.map((element) => {

[Andrew]

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

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

??????????????????? точно так нужно .map исользовать?
у вас у стрингов есть свойство "map"?

[rapidum_alder]

RAX7, Спасибо, работает

[rapidum_alder]

Andrew, Название переменных это то "искусство" в котором я крайне не силен. Я имел ввиду строки когда называл

Answer 1

[Дима Паутов]

Вы написали его неверно

// не
strings.map = function () {

}
//a
strings.map(function(){

})

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

Answer 2

[Дмитрий]

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

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

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