不使用JS匿名函數的三個理由
無論你在什么時候讀代碼,您都必須注意到匿名函數。有時它們被稱為 lambda,有時是匿名函數,不管怎樣,我認為他們是不好使用的。
如果你不知道匿名函數是什么,這里有一個引語:
匿名函數是一種在運行時動態聲明的函數。它們之所以被稱為匿名函數是因為不同于普通函數,它們并沒有函數名。 — Helen Emerson, Helephant.com
匿名函數形式如下:
- function () { ... code ... }
- OR
- (args) => { ... code .. }
今天我嘗試讓大家理解只有在絕對需要的情況下才使用匿名函數的想法。匿名函數不應該是***,而且你自己也應該知道為什么使用它。當理解這種想法之后,你的代碼會變得更簡潔,更容易維護,并且更容易跟蹤bug。
先從避免使用匿名函數的三個理由開始:
無論你多么擅長寫代碼,出現錯誤也是不可避免的。有時候,這些錯誤很容易被查出,有時候并不容易。
如果你知道這些錯誤來自哪里,那么錯誤會很容易被查出來。為了容易查出錯誤,我們使用這個被叫做堆棧軌跡的工具。如果你不了解堆棧軌跡,goole給出了很棒的介紹。
假設現在有一個非常簡單的項目:
- function start () {
- (function middle () {
- (function end () {
- console.lg('test');
- })()
- })()
- }
上面代碼里面有一個非常愚蠢的錯誤,拼寫錯誤(console.log)。在小項目里面,這個拼寫錯誤不是什么大問題。如果這是一個有非常多模塊非常大的項目一小段,問題就大了。假設這個愚蠢的錯誤不是你犯的,那么新來的初級工程師將會在他休假之前把這個錯誤提交到代碼庫!
現在,我們必須追查。 使用我們精心命名的函數,我們得到如下的堆棧跟蹤:
謝謝你命名你的函數,初級開發者們! 現在我們可以輕松地追蹤到這個bug。
但是..一旦我們解決了這個問題,就會發現還有另一個bug。 這次是一位更資深的開發人員介紹的。這個人知道lambdas(匿名函數),并在代碼中大量使用它們。 結果他們偶然發現了一個bug,我們的工作就是追蹤它。
下面是代碼:
- (function () {
- (function () {
- (function () {
- console.lg('test');
- })();
- })();
- })();
吃不吃驚,這名開發者也忘記了如何拼寫console.log了!這也太巧合了吧!令人感到遺憾的是,他們都沒有命名他們的函數。
那么控制臺會輸出什么呢?
好吧,我們至少還有行號,對吧?在這個例子中,看起來我們有大約7行代碼。如果我們處理一大段代碼會如何呢?比如一萬行代碼?行號的跨度如此之大該怎么辦呢?如果代碼被折疊后有沒有一個代碼地圖文件,那么對行號的渲染是不是根本就是沒有什么用了呢?
我想對這些問題的回答相當簡單,答案就是:想這些會讓你一整天都會過的相當糟心。
可讀性
咦,我聽說你還不信。你仍舊對你的匿名函數戀戀不舍,并且還從未發生過bug。我的錯,你的代碼是完整的。但是讓我們看看這個!
看看下面兩段代碼:
- function initiate (arguments) {
- return new Promise((resolve, reject) => {
- try {
- if (arguments) {
- return resolve(true);
- }
- return resolve(false);
- } catch (e) {
- reject(e);
- }
- });
- }
- initiate(true)
- .then(res => {
- if (res) {
- doSomethingElse();
- } else {
- doSomething();
- }
- ).catch(e => {
- logError(e.message);
- restartApp();
- }
- );
這是一個非常不正常的例子,但是我相信你已經明白我要說什么了。我們反悔了一個promise方法,我們用這個promise對象/方法處理不同的響應。
你也許會認為幾段代碼讀起來并不難,但我認為它們可以變得更好!
如果我們去掉所有的匿名函數會怎樣呢?
- function initiate (arguments) {
- return new Promise(checkForArguments);
- }
- function checkForArguments (resolve, reject) {
- try {
- if (arguments) {
- return resolve(true);
- }
- return resolve(false);
- } catch (e) {
- reject(e);
- }
- }
- function evaluateRes (res) {
- if (res) {
- doSomethingElse();
- } else {
- doSomething();
- }
- }
- function handleError (e) {
- logError(e.message);
- restartApp();
- }
- initiate(true)
- .then(evaluateRes)
- .catch(handleError);
好,先講清楚:這部分代碼更長,但我不認為只是增加了可讀性!我們精心命名的函數與匿名函數不一樣,只要我們一看到它們的名字就知道它們的功能是什么。這避免了在評估代碼時出現心理障礙。
這也有助于分清楚其中的關系。與創建一個方法、將其傳遞、然后運行邏輯不同,在第二個例子中的參數被給到了then,catch只是指向了發生所有事情的函數。
關于更具有可讀性,我沒有什么再能說服你的了。但是也許你還沒被說服的話,我可以試一下***的論據。
可重用性
你注意到上一個例子了嗎?上個例子中的函數的使用范圍從參數和初始化函數,變為讓所有函數都能使用。
當你使用匿名函數時這些函數很難在你的應用程序內重復使用。
可重用性將不復存在,最終你會一遍又一遍地寫重復的代碼。正如我們所見的,代碼寫的越少引入的Bug就越少,用戶必須加載的內容就越少。所有人都會因此獲益!
相反的,命名函數可以全局使用,而不需要像變量一樣到處傳遞。你的代碼的可重用性會更好,
匿名函數有可取的地方嗎?
有。雖然很不愿意承認,但有時候使用匿名函數是***的選擇。
- const stuff = [
- { hide: true, name: 'justin' },
- { hide: false, name: 'lauren' },
- { hide: false, name: 'max' },
- ];
- const filteredStuff = stuff.filter(s => !s.hide);
上邊代碼中的匿名函數s => !s.hide非常簡單,即使不能在別的地方使用也不會對別人有任何影響,而且也可以在stuff.filter中顯示出堆棧調用。如果想要重用這段代碼,***重用整段代碼:
- function filterByHide (array) {
- return array.filter(item => !item.hide);
- }
有時你想把你所有的代碼封裝到匿名函數中,以保證全局范圍不會被污染。
- (() => {
- ... your code here ...
- })();
在棧空間中擁有一個***的匿名函數真得不會有什么錯誤。沒有代碼重用是痛苦的,因為完整的目的是保持方法內含。
我確定這會有其他好的用法,請在評論中自由分享之!
感謝閱讀,現在跳出這些,并停止編寫匿名函數!