为何无法为每个数组元素调用函数？控制台返回undefined问题

嘿，我看了你的代码片段，发现几个关键问题导致console.log返回undefined，咱们一步步梳理修复：

问题分析 #

1. 代码未完成引发语法错误：你的array2的map回调里，value:后面的Math.random()缺失了，这会直接触发JavaScript语法错误，让脚本中断执行，newArray2根本没机会被正确赋值，自然console.log它会返回undefined。
2. 返回结果是数组而非单个元素：你用slice(0,1)获取的是包含单个元素的数组（比如['I want to']），如果你的预期是单个字符串，直接输出这个数组虽然不会是undefined，但可能不是你想要的结果；如果后续代码依赖单个值，也会引发其他问题。

修正后的完整代码 #

先把逻辑补全，同时调整为直接获取单个随机元素：

const array1 = ['I want to','You want to'];
// 随机排序后取第一个元素，用[0]直接提取单个值，替代slice(0,1)返回的数组
let newArray1 = array1
  .map(item => ({value: Math.random(), item: item}))
  .sort((a,b)=> a.value - b.value)
  .map(item => item.item)[0];

const array2 = [' eat',' sleep '];
// 补全缺失的Math.random()，完成随机排序逻辑
let newArray2 = array2
  .map(item => ({value: Math.random(), item: item}))
  .sort((a,b)=> a.value - b.value)
  .map(item => item.item)[0];

// 现在输出就能得到预期的随机组合啦
console.log(newArray1 + newArray2);
console.log(newArray1);
console.log(newArray2);

更简洁的随机取元素方法 #

如果你的需求只是从数组里随机选一个元素，其实不用这么复杂的排序逻辑，直接通过随机索引获取效率更高：

// 封装一个通用的随机取元素函数
const getRandomItem = (arr) => arr[Math.floor(Math.random() * arr.length)];

const array1 = ['I want to','You want to'];
const array2 = [' eat',' sleep '];

const randomItem1 = getRandomItem(array1);
const randomItem2 = getRandomItem(array2);

console.log(randomItem1 + randomItem2);

这个方法的时间复杂度是O(1)，比排序的O(n log n)更高效，逻辑也更清晰易懂。

内容的提问来源于stack exchange，提问作者KT-mongo