JavaScript数组排序问题：map计算value时返回undefined

看起来你是想先给数组里的每个对象计算出一个value值，再基于这个值排序，但遇到了map返回undefined的问题对吧？我来帮你梳理一下问题出在哪，以及怎么解决。

首先，先看你代码里的几个潜在问题 #

• 对象赋值的不规范写法：你在数组里用person_1 = { ... }这种方式，会在全局作用域创建不必要的变量，更规范的写法是直接在数组里定义对象字面量。
• map方法缺少return：这是导致你得到undefined的核心原因！map方法需要回调函数返回新数组的元素，如果你的回调里没有显式写return计算后的value，那每个位置都会是undefined。

两种可行的解决方案 #

方案一：先给对象添加value属性，再排序

如果需要保留计算后的value在对象里，可以先遍历数组给每个对象赋值value，再用sort排序：

function comparisonGame() {
  // 规范创建对象数组，不用额外变量
  const peopleArr = [
    { name: 'Nikita', age: 26, height: 182 },
    { name: 'Pasha', age: 28, height: 193 }
    // 其他对象...
  ];

  // 先给每个对象计算并添加value属性（这里假设你的公式是age * height，你可以替换成自己的）
  peopleArr.forEach(person => {
    person.value = person.age * person.height; // 这里替换成你的实际公式
  });

  // 根据value排序，升序的话a.value - b.value，降序就反过来
  const sortedArr = peopleArr.sort((a, b) => a.value - b.value);
  
  console.log(sortedArr);
}

方案二：直接在排序的比较函数里计算value（不用提前map）

如果不需要保留value属性，直接在sort的比较函数里计算每个对象的目标值，这样更高效，还能避免额外的map遍历：

function comparisonGame() {
  const peopleArr = [
    { name: 'Nikita', age: 26, height: 182 },
    { name: 'Pasha', age: 28, height: 193 }
    // 其他对象...
  ];

  // 定义计算value的函数，方便复用
  const calculateValue = (person) => {
    return person.age * person.height; // 替换成你的实际公式
  };

  // 直接在排序时计算两个对象的value进行比较
  const sortedArr = peopleArr.sort((a, b) => {
    const valueA = calculateValue(a);
    const valueB = calculateValue(b);
    return valueA - valueB; // 升序，降序用valueB - valueA
  });
  
  console.log(sortedArr);
}

为什么你之前的map会返回undefined？ #

举个反例，如果你写了这样的map：

const values = peopleArr.map(person => {
  // 这里没有return，所以每个元素都是undefined
  person.age * person.height;
});

这时候values数组里全是undefined，自然没法用来排序。解决办法就是在map回调里加上return，或者用箭头函数的简写（去掉大括号，自动return）：

const values = peopleArr.map(person => person.age * person.height);

不过如果你的目的是排序原数组，上面的两种方案会比单独用map更直接哦。

内容的提问来源于stack exchange，提问作者Nikita Kornienko