JavaScript如何高效遍历对象数组校验参数并即时返回错误

原实现存在的问题 #

你之前写的逻辑有三个明显缺陷：

• 校验逻辑写反：频道ID的判断逻辑完全错误，只要数组里存在任意一个ID和传入值不相等的频道，就会把错误标记设为false，会导致合法ID、非法ID都出现误判
• 性能浪费：无论是否找到匹配项，都会完整遍历整个用户、频道数组，无法提前终止
• 风格冗余：手动维护错误标记的写法不符合JavaScript的惯用编码风格

简洁实现方案 #

JavaScript数组内置的some()方法天生适配这个校验场景：它在遍历数组时，只要回调函数返回true就会立刻终止遍历，整体返回true，刚好满足「找到匹配项就立即停止、返回结果」的需求。
基础实现代码如下：

// 校验传入ID是否存在对应记录
const isChannelValid = data.channels.some(channel => channel.channelId === channelId);
const isUserValid = data.users.some(user => user.uId === authUserId);

// 任意一项校验不通过，直接返回错误终止函数执行
if (!isChannelValid || !isUserValid) {
  return { error: 'error' };
}

如果追求极简的单行写法，可以把校验和判断逻辑合并，不需要声明中间变量：

if (!data.channels.some(c => c.channelId === channelId) || !data.users.some(u => u.uId === authUserId)) return { error: 'error' };

提示：如果你需要频繁做这类ID校验，可以在数据初始化时把合法ID预存为Set结构，后续校验的时间复杂度可以从O(n)降到O(1)，性能更高：

// 数据初始化时执行一次即可
const validUids = new Set(data.users.map(u => u.uId));
const validChannelIds = new Set(data.channels.map(c => c.channelId));

// 后续校验无需遍历数组
if (!validChannelIds.has(channelId) || !validUids.has(authUserId)) {
  return { error: 'error' };
}

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