javascript - 在 if else block 中重构重复的 if 语句

Question

我有这个 if block ，我必须在两个 block 中测试相同的东西:

   if (download !== 'true') {
     if(index == undefined){
       res.json(req.doc);
     }else{
       res.json(index)
     }
   } else {
     if(index == undefined){
       exports.download(res, req.doc);
     }else{
       res.json(index)
     }
   } 

有没有办法以一种我不会重复同样的事情的方式重构它？

Answer 1

由于当 index == undefined 为 false 时，您在两个分支中执行相同的操作，因此只需先执行该测试并将其反转即可:

if (index != undefined) {
  res.json(index);
} else if (download !== 'true') {
  res.json(req.doc);
} else {
  exports.download(res, req.doc);
}

---

旁注:

• == undefined 和 != undefined 将以相同的方式处理 undefined 和 null。如果您不希望您的条件将 null 视为 undefined，请使用 === 和 !== .
• download 是一个字符串 有点奇怪，当然这有时确实会发生。 如果 download 实际上是一个 bool 值，那么 !== 'true' 将永远为真(因为没有 bool 值会严格等于一个字符串). If 它是一个 bool 值，使用 if (download) 或 if (!download) 而不是 === true或 !== true。如果它是一个字符串，注意开头或结尾的空格和大写(' true' !== 'true' 是 true 因为空间的；'True' !== 'true' 是 true 因为大写 T)。 FWIW。