Почему не может прочитать свойство?

Question

[Даниил Игоревич]

var users = [{
  name: "Вася",
  surname: 'Иванов',
  age: 20
}, {
  name: "Петя",
  surname: 'Чапаев',
  age: 25
}, {
  name: "Маша",
  surname: 'Медведева',
  age: 18
}];

function compare(a, b) {
	return function (field){
		return a[field] > b[field] ? 1 : -1;
	}
}
var byField = compare();

users.sort(byField('name'));
users.forEach(function(user) {
  alert( user.age );
});

В консоли показывается - "cannot read property 'name' of undefined". Почему и как исправить?

Answer 1

[Юрий]

вот так

function compare(field){
  return function (a, b) {
    return a[field] > b[field] ? 1 : -1;
  }
}
var byName = compare('name');

users.sort(byName);
users.forEach(function(user) {
  alert( user.age );
});

Answer 2

[Дмитрий Беляев]

Что-то мне подсказывает, что функция compare написана неверно, надо так:

function createCompare(field) {
  return function (a, b){
    if(a[field] > b[field]) return 1;
    if(a[field] < b[field]) return -1;
    return 0;
  }
}

users.sort(createCompare('name'));

И изучите замыкания в js

Comments

[Даниил Игоревич]

Сейчас как раз выполняю задание там, спасибо!

Answer 3

[_ _]

Вы замыкание задом наперед сделали

function compare(field) {
  return function (a, b){
    return a[field] > b[field] ? 1 : -1;
  }
}
var byName = compare('name');

users.sort(byName);
users.forEach(function(user) {
  alert( user.age );
});

Comments

[Даниил Игоревич]

Можете объяснить? И как возможно это замыкание теперь перевернуть в нужную сторону?

Answer 4

[Алексей Струков]

вы когда функцию вызываете, то параметры забываете отправить в нее.
Да и вообще что то не то тут. Функция sort у массива не может сортировать обьекты вроде.

Comments

[Даниил Игоревич]

byField()? В 4-ой строке снизу я же отправил: users.sort(byField('name'));
Или вы не об этой функции?

[Даниил Игоревич]

Может, в функцию sort можно передать в качестве аргумента функцию, у которой два аргумента - a, b, и у них нужно указать имя свойства. Я так представляю.

[Даниил Игоревич]

var users = [{
  name: "Вася",
  surname: 'Иванов',
  age: 20
}, {
  name: "Петя",
  surname: 'Чапаев',
  age: 25
}, {
  name: "Маша",
  surname: 'Медведева',
  age: 18
}];


users.sort(function(a,b){
	return a.age > b.age ? 1 : -1;
});
users.forEach(function(user) {
  alert( user.age );
});

[timfcsm]

Даниил Ораин: в функию compare вы не передаете аргументы... сделайте наоборото
function compare(field) {
return function (a, b){
return a[field] > b[field] ? 1 : -1;
}
}
var byField = compare('name');

[Алексей Струков]

Даниил Ораин: меня вот это смутило - var byField = compare();
По вашему коду тут получается присваивание результата вычисления функции в переменную byField

[Даниил Игоревич]

timfcsm: Я так себе представил и думал, что так и должно быть: в переменную byField записали функцию, которая возвращается функцией compare, то есть:
var byField = function(field){
return a[field] > b[field] ? 1 : -1;
}
Где я тут ошибся?

[Даниил Игоревич]

Алексей Струков: да, а что с этим не так?

[timfcsm]

Даниил Ораин: ну так вы тут и не возвращаете функцию, вы пытаетесь достучаться до полей несуществующих a и b

var byField = function(field){
return function(a, b) {a[field] > b[field] ? 1 : -1;}
}
в общем, ниже написали правильный вариант полностью

[Алексей Струков]

Даниил Ораин: ниже уже написали решение. Я сам немного подзабыл видимо как это работает, раз оказалось что обьекты можно сортировать таким образом