Почему undefined?

Question

[JackShcherbakov]

Возникла проблема. Почему первое слвоо в алерете undefined? Вызвано ведь в правильном контексте через call:
https://jsfiddle.net/CitizenOne/ha49evee/20/
Но с bind работает:
https://jsfiddle.net/CitizenOne/ha49evee/21/

Answer 1

[Kovalsky]

Функция-обработчик, которую вы передаёте в forEach, выполняется в контексте текущего элемента массива.

Если в метод forEach() был передан параметр thisArg, при вызове callback он будет использоваться в качестве значения this. В противном случае, в качестве значения this будет использоваться значение undefined. В конечном итоге, значение this, наблюдаемое из функции callback, определяется согласно обычным правилам определения this, видимого из функции.

Comments

[JackShcherbakov]

Вот цитата из справочника

Если в метод forEach() был передан параметр thisArg, при вызове callback он будет использоваться в качестве значения this. В противном случае, в качестве значения this будет использоваться значение undefined. В конечном итоге, значение this, наблюдаемое из функции callback, определяется согласно обычным правилам определения this, видимого из функции.

То есть контекст не текущий элемент.

[Kovalsky]

JackShcherbakov, да, значит я тупой, и функция-обработчик устанавливает своё значение this, то есть window. Варианты решения: использовать стрелочные функции, сохранять контекст в переменную, передавать контекст в forEach с помощью thisArg. Стрелочные функции будут самым простым вариантом:

this.students.forEach(student => alert(this.title + ': ' + student))

Answer 2

[Владимир Алхименко]

//'use strict';

let group = {
  title: "Наш курс",
  students: ["Вася", "Петя", "Даша"],

  showList: function() {
    this.students.forEach(function(student) {
      alert(group.title + ': ' + student); // будет ошибка
    })
  }
}


group.showList.call(group);

Comments

[Владимир Алхименко]

=))).Почитай контекст this

[JackShcherbakov]

AngryProgrammer001, Ну так вы прсото взяли и this убрали. В вашем коде не надо никакого call. Без него сработает.

Answer 3

[rinatoptimus]

Пользуйтесь console.log();
В вашем случае, если прописать console.log(this) сразу после showList: function() { ...
, в консоли выведется объект. Соответственно, попробуйте так:

group.students.forEach(function(student) {
      alert(group.title + ': ' + student); 
    })

Comments

[JackShcherbakov]

Ну так вы прсото взяли и this убрали. В вашем коде не надо никакого call. Без него сработает. А мне надо с this. Странность в том, что с bind все рабоатет.