JavaScript短路评估 - 缺少财产价值

Question

我正在尝试改进我的编码实践，所以我试图重构以下代码：

编辑我的问题是关于短路评估https://codeburst.io/javascript-what-is-short-circuit-evaluation-ff22b2f5608c的最佳实践

var idArray = [
    { id: 15 },
    { id: -1 },
    { id: 0 },
    { id: 3 },
    { },
    { id: null },
    { id: NaN },
    { id: 'undefined' }
  ]

let idFilteredArray0 = []

idArray.forEach(idObj  =>{
if(typeof idObj.id === 'number' && !isNaN(idObj.id))
    idFilteredArray0.push(idObj.id)
})

// from a forEach loop which returns [ 15, -1, 0, 3, 12.2 ]

let idF0 = idArray.reduce((acc, obj) => {
  if((obj.id || obj.id)  && !isNaN(obj.id))
    acc.push(obj.id)
  return acc
},[]) 

// to reduce which returns [15, -1, 3, 12.2]

我认为&& ing是问题所在，但我看不出以不同的方式排除NaN值。我决心让0回到结果数组中。最后，如果有人知道学习短路评估的好资料，我会非常感激。谢谢。

J.

Answer 1

你可以过滤和映射项目，并检查类型和id值不是NaN。

var idArray = [{ id: 15 }, { id: -1 }, { id: 0 }, { id: 3 }, { id: 12.2 }, {}, { id: null }, { id: NaN }, { id: 'undefined' }],
    idFiltered = idArray
        .filter(({ id }) => typeof id === 'number' && !isNaN(id))
        .map(({ id }) => id);

console.log(idFiltered);

Answer 2

当if(obj.id)是false时，obj.id将返回0。你应该重构这样的东西来解释为零：

let idF0 = idArray.reduce((acc, obj) => {
  if(typeof obj.id === 'number'  && !isNaN(obj.id))
    acc.push(obj.id)
  return acc
},[]) 

如果你想要获得更多的性能，请考虑以下解决方案，它以线性时间复杂度运行，如.reduce()，但比.reduce()解决方案快得多：

let idF0 = [], i = 0, length = idArray.length, id;

for (; i < length; i++) {
  id = idArray[i].id;
  if (id === 'number' && !isNaN(id)) {
    idF0.push(id);
  }
}

return idF0;