数组为空导致forEach无法执行,filter未返回有效值需如何调整?
问题排查与修复方案
核心问题定位
第一行_.filter执行后返回空数组,通常是以下几种原因导致:
- 源数组
grantedValuesFinal本身为空 - 数组所有元素的
numberOfShareOptionAwardGranted字段值均≤0 - 该字段为
undefined/null/字符串类型的数字,直接和0比较时返回false
修复步骤
- 先验证源数据有效性,可先添加调试代码确认字段值:
console.log('源数组内容:', grantedValuesFinal) console.log('所有元素的对应字段值:', grantedValuesFinal.map(o => o.numberOfShareOptionAwardGranted))
- 修改筛选逻辑,把字段先转为数字再比较,避免字符串类型的数字被过滤:
const filteredOptions = _.filter(grantedValuesFinal, o => Number(o.numberOfShareOptionAwardGranted) > 0);
如果调试后确认源数组确实没有符合条件的元素,那么filteredOptions为空属于正常逻辑,无需修改代码。
完整优化后代码
同时修复了你原有代码里的冗余逻辑、类型转换风险、性能问题:
// 转数字后再筛选,避免字符串类型数字被误过滤 const filteredOptions = _.filter(grantedValuesFinal, o => Number(o.numberOfShareOptionAwardGranted) > 0); let filteredArray = []; filteredOptions.forEach(function(item){ // 用find替代filter查找已存在项,性能更高 const existing = filteredArray.find(o => o.grantDate === item.grantDate); if (existing) { const addValue = Number(item.numberOfShareOptionAwardGranted); if(item.typeOfEquity === "Shares") { existing.numberOfShareOptionAwardGrantedShares += addValue; } if(item.typeOfEquity === "Options") { existing.numberOfShareOptionAwardGrantedOptions += addValue; } } else { let numberOfShareOptionAwardGrantedShares = 0, numberOfShareOptionAwardGrantedOptions = 0; const addValue = Number(item.numberOfShareOptionAwardGranted); if(item.typeOfEquity === "Shares") { numberOfShareOptionAwardGrantedShares += addValue; } if(item.typeOfEquity === "Options") { numberOfShareOptionAwardGrantedOptions += addValue; } filteredArray.push({ grantDate: item.grantDate, numberOfShareOptionAwardGrantedShares, numberOfShareOptionAwardGrantedOptions }); } })
优化点说明:
- 移除了原有代码里无用的luxon日期转换逻辑(转换后未参与判断,属于冗余代码)
- 所有数值计算前统一转Number,避免字符串拼接的异常
- 用find替代filter查找已存在项,找到匹配项就终止遍历,性能提升明显
- 用全等
===替代双等==,避免隐式类型转换导致的逻辑错误
内容的提问来源于stack exchange,提问作者Asha Shokand
相关产品推荐
相关产品推荐

