尽管设置了值，为什么我的变量返回为“未定义”？

Question

javascript 相对较新。似乎无法找出此代码不起作用的原因。

这是Odin Project的javascript练习的一部分，练习12： https://github.com/TheOdinProject/javascript-exercises/tree/main/12_findTheOldest

目前只是在为这个练习做测试的第一部分。 返回变量后，返回“undefined”，测试失败。

这是我的代码。 我知道它很可能远非高效但我想知道为什么这种特定方式，即使不是理想的，在终端测试时也不起作用（除非这种方法在这种特定情况下根本不起作用并且测试）。 当返回值返回“未定义”时，通过 console.log 返回它显示变量确实存储了正确的答案。

const findTheOldest = function(people) {
    withAges = people.map(x => ({
        ...x, age: Number(x.yearOfDeath) - Number(x.yearOfBirth)
      }));
    let max = Math.max(...withAges.map(x => { return x.age; }));
    let maxFind = withAges.find((x => { return Number(x.age) == Number(max); }));
    let maxName = maxFind.name;
    return maxName;
    //return console.log(maxName) returns the correct answer which is the name Ray.
};

module.exports = findTheOldest;

这是测试我的代码结果并返回未定义的 spec.js 代码。

const findTheOldest = require('./findTheOldest')

describe('findTheOldest', () => {
  test('finds the oldest person!', () => {
    const people = [
      {
        name: "Carly",
        yearOfBirth: 1942,
        yearOfDeath: 1970,
      },
      {
        name: "Ray",
        yearOfBirth: 1962,
        yearOfDeath: 2011,
      },
      {
        name: "Jane",
        yearOfBirth: 1912,
        yearOfDeath: 1941,
      },
    ]
    expect(findTheOldest(people).name).toBe('Ray');
  });
});

Answer 1

您返回名称而不是 object。

您实际上正在阅读 'Ray'.name ，它是undefined 。

尝试删除.name ，您将拥有它。

Answer 2

当您在正确的轨道上时，您必须考虑并非数组中的每个 object 都包含yearOfDeath属性的情况。 如果没有，您可以通过多种方式处理它。

一种方法是向每个 object 添加一个年龄属性。这与您的实现类似。 如果当前 object 没有yearOfDeath ，您可以通过使用new Date().getFullYear()获取当前年份并从该人的yearOfBirth中减去它来创建一个。

const findTheOldest = function(people){
  const peopleWithAges = people.map(person => {
        if( !person.hasOwnProperty('yearOfDeath') )
            person.age = new Date().getFullYear() - person.yearOfBirth;
        else
            person.age = person.yearOfDeath - person.yearOfBirth;

        return person; // at every step, you are adding the age property to each person
  });

  const oldestAge = Math.max( ...peopleWithAges.map(person => person.age) );

  // We use find here because using Array.filter would return an array with one object in it.
  return peopleWithAges.find(person => person.age === oldestAge);
}

另一种方法是使用Array.reduce()并在每次减少访问每个元素时更新最旧的元素。 这比多次调用Array.map()和Array.find()更有效。

const findTheOldest = function(people){
   const peopleWithAges = people.map(person => {
        if( !person.hasOwnProperty('yearOfDeath') )
            person.age = new Date().getFullYear() - person.yearOfBirth;
        else
            person.age = person.yearOfDeath - person.yearOfBirth;

        return person; // at every step, you are adding the age property to each person
    });

    return peopleWithAges.reduce((oldestPerson, currentPerson) => {
        if (currentPerson.age > oldestPerson.age)
            oldestPerson = currentPerson;
            
        return oldestPerson;
    });
}

您可以将调用链接到map和reduce ，但这会使代码更难阅读。