目錄

• 前言
• 一、何為嵌套代碼
• 二、避免嵌套
• 1.提煉抽取
• 2.反轉排列

前言

看過不少過度嵌套的代碼, 我真正意識到問題的嚴重性是剛入職那會, 我在一個老項目里看到了40個連續的else if, 套了6層的if, for和forEach, 因為我們并沒有做什么限制代碼嵌套的提前約定.

呃, 那之后認識到會寫代碼和代碼寫得好完全是兩種概念, 能夠實現復雜的需求也并不能說明代碼寫得好, 開始注重代碼結構方面.

事實是, 很多時候需要編寫的邏輯本身就很惡心, 乍看之下, 堆頁巖般的判定嵌套里似乎每一層都是必要的, 也只能說盡量讓它看起來不那么惡心.

嗯, 比如少來幾次Tab.

一、何為嵌套代碼

嵌套代碼是在函數內部添加更深層級的代碼塊, 放在javascript里, 常用的嵌套手段都包含符號’{‘, 那么對于一個代碼塊, 刨除平級的情況, 其內部的’{'越多就說明這個代碼塊的嵌套深度越大.

也就是: 禁止套娃.

對于以下代碼, 它的嵌套深度為1:

function fun1 () {
  console.log(1);
}

而如果在內部加上if語句, 其深度將變為2:

function fun1 () {
  if (true) {
    console.log(1);
  }
}

而如果再加一個循環進去, 深度將變為3:

function fun1 () {
  if (true) {
    for (let i = 0; i < 5; i++) {
      console.log(1);
    }
  } else {
    console.log(2);
  }
}

而…

好的各位, 我最多最多就到這了, 再套下去我就要開始覺得惡心了.在這里可能沒有那么直觀, 而這段代碼放在編輯器里, console.log前面已經有三道豎線了, 光是tab提行就已經開始不舒服了.

在三層嵌套以上, 你所做的一切就不再是一套單一的算法, 這已經開始逐漸演變為多個算法的組合了, 是可以做一些封裝抽離而最好不要就這樣混寫在一起.

實戰中三層嵌套絕對連半數以下的計算都處理不了, 那如果還有邏輯沒編寫呢.

二、避免嵌套

1.提煉抽取

提煉(Extraction), 我一般管這叫抽離, 當然, 不一定要抽到外面, 只要能維持嵌套深度處于穩定的水平就好(不過函數內實在不能在消減嵌套深度那還是抽到外面形成另外一個函數吧).

比如這段嵌套:

function fun1() {
  const arr = [1, 2, 3, 4]
  if (arr.length = 4) {
    arr.forEach((ele) => {
      if (a === 4) {
console.log(4);
      }
    });
  }
}
fun1();

可以改為這樣:

function fun1() {
  const arr = [1, 2, 3, 4]
  const xxx = (a) => {
    if (a === 4) {
      console.log(4);
    }
  }
  if (arr.length = 4) {
    arr.forEach(xxx);
  }
}
fun1();

嵌套深度由4減小為3.

原理十分明了, 就好像在原生環境獲取DOM, 有的人喜歡這樣:

function change() {
  document.querySelector("#scar").style.display = "none";
}

有的人喜歡:

function change() {
  const scar = document.querySelector("#scar");
  scar.style.display = "none";
}

抽離提煉就類似于將前者轉化為了后者.

封裝axios也是一樣的道理(不過那更多還是為了避免接口變動導致的被動局面).

2.反轉排列

反轉(Inversion), 對于判定語句, 把正面條件排在負面條件前通常會需要更多的判定, 所以改為優先處理負面條件.

先把正面條件放前面:

function justice(e) {
  if(e.length > 5) {
    for(let i = 0; i < e.length; i++) {
      console.log(e);
    }
  } else if (e.length === 2){
    return 2;
  } else {
    return false
  }
}

但是如果先進行負面條件判定:

function justice(e) {
  if(e.length === 2) {
    return 2;
  } else if (e.length < 5) { // 這里也可以另起一個if, 不過這樣可以節約一行 )
    return false;
  } 
  for(let i = 0; i < e.length; i++) {
    console.log(e);
  }
}

可以看到現在深度層級由3減小到2.

這種優化方法需要先把少數, 需要特殊處理的情況在前面解決完及時退出, 剩下的多數情況就可以不放在判定語句中.

而在這個過程中, 需要把最特殊, 且不將其他特殊情況包含在內的情況寫在前面, 越特殊, 越提前處理, 此處e.length === 2為最特殊, 而e.length < 5這個特殊情況將e.length === 2包含在內, 所以應當第二個處理.

我在前面也寫過這種做法, 將判定嵌套改為平次的衛語句, 稱作validation gatekepping, 感興趣的話可以去看這篇:

不過還可以在平次判定這個基礎上使用這個技巧, 我們把負面情況放在靠前的平次判定處理, 如果處理中途出現過多嵌套, 那就提煉抽離, 把正面條件放最后:

function justice(e) {
  if(e.length === 2) {
    return 2;
  }
  if(e.length === 3) {
    return 3;
  }
  if (e.length < 5) {
    return false;
  } 
  for(let i = 0; i < e.length; i++) {
    console.log(e);
  }
}

截取最近項目里的代碼作為例子, 現在有兩個world, 一個新一個舊, 如果需要讓舊world的視圖更新, 那么需要將新world的world.webglGroup.children內的元素部分替換, 其他除world.frameInfo外也要全替換.

async changeWorld(oldFrame, newWorld) {
  for (const key in newWorld) {
    if (key === "frameInfo") {
    } else if (key === "webglGroup") {
      for (const pro in newWorld[key]) {
if (pro === "children") {
  this.worldList[oldWorldIndex][key][pro] = this.worldList[oldWorldIndex][key][pro].filter((ele) => { return ele.type !== "Group" });
  this.worldList[oldWorldIndex][key][pro].push(...newWorld[key][pro].filter((ele) => { return ele.type === "Group" }));
} else {
  this.worldList[oldWorldIndex][key][pro] = newWorld[key][pro];
}
      }
    } else {
      this.worldList[oldWorldIndex][key] = newWorld[key];
    }
  }
}

以上是初版, 現在用Extraction提煉和Inversion反轉去嘗試降低嵌套深度:

先把world.webglGroup.children局部替換的代碼提煉為replace,

已知world.frameInfo不需要替換, 那么正常的負面條件寫法應當為key === 'frameInfo', 但即便如此key === 'frameInfo'和key === 'webglGroup'也是必須用else if處理的, 如果改成平次if又不能終止執行, 那么這兩個特殊條件在一輪循環中都會被執行.

本著要把正面條件處理方案寫最后的原則, 直接在最后一個特殊條件不滿足(按照上文所述寫法, 最后一個特殊條件不滿足說明前面所列特殊條件均不滿足)時執行正面條件處理方案.

else if (key !== "frameInfo") {
  this.worldList[oldWorldIndex][key] = newWorld[key];
}

async changeWorld(oldFrame, newWorld) {
  let oldWorldIndex = this.worldList.findIndex((w) => w.frameInfo.frame === oldFrame);
  const replace = () => {
    this.worldList[oldWorldIndex][key]["children"] = this.worldList[oldWorldIndex][key]["children"].filter((ele) => { 
      return ele.type !== "Group";
    });
    this.worldList[oldWorldIndex][key]["children"].push(...newWorld[key]["children"].filter((ele) => { 
      return ele.type === "Group";
    }));
  }
  for (const key in newWorld) {
    if (key === "webglGroup") {
      replace(key);
    } else if (key !== "frameInfo") {
      this.worldList[oldWorldIndex][key] = newWorld[key];
    }
  }
  return this.worldList[oldWorldIndex];
}

只是判定需求不同罷了.

上面這種寫法是在所有負面條件不滿足時執行正面條件處理方案.

前面反轉的例子是在任意負面條件不滿足時結束執行.

但遵循兩種優化手段的原則都可以實施優化.

到此這篇關于JavaScript避免嵌套代碼淺析的文章就介紹到這了,更多相關JS避免嵌套代碼內容請搜索以前的文章或繼續瀏覽下面的相關文章希望大家以后多多支持！