折射器 if 语句

问题描述

我想折射一段代码

if (res1 && res1.e > maxEnergy) {
    maxRes = res1;
    maxEnergy = res1.e;
}
if (res2 && res2.e > maxEnergy) {
    maxRes = res2;
    maxEnergy = res2.e;
}
if (res3 && res3.e > maxEnergy) {
    maxRes = res3;
    maxEnergy = res3.e;
}

有没有最好的方法在较短的段中完成这段代码？ 使用对象字面量还是 switch 语句？

解决方法

更短但仍然可读（恕我直言，可读性更强）：

[res1,res2,res3].forEach(res => {
  if (res?.e > maxEnergy) {
    maxRes = res;
    maxEnergy = res.e;
  }
});

,

只是对这个问题的另一种看法。当我看到某些操作被重复（if 语句）时，我喜欢将该代码移动到一个函数中，以便我稍后可以在其他上下文中使用它。

let currentMax= {
    e: 0
}

const res1 = {
    e: 90
};
const res2 = {
    e: 170
};
const res3 = {
    e: 10
};

function findMaxEnergy(res,currentMaxRes) {
  if (res && res.e > currentMaxRes.e) {
    return res;
  }
  return currentMaxRes;
}

// now you can use it one by one
// currentMax = findMaxEnergy(res1,currentMax);
// currentMax = findMaxEnergy(res2,currentMax);
// currentMax = findMaxEnergy(res3,currentMax);

// or with an array and a loop
[res1,res3].forEach(res => {
  currentMax = findMaxEnergy(res,currentMax);
});

console.log(currentMax);