當(dāng)很多前輩教育后輩應(yīng)當(dāng)多寫注釋的時(shí)候,當(dāng)網(wǎng)絡(luò)上充滿了有關(guān)程序員從不寫注釋的段子的時(shí)候,這是一個(gè)非常有爭議的話題。作為一個(gè)標(biāo)題黨,容我先修正一下我的觀點(diǎn):我認(rèn)為如果代碼寫得足夠好,那么大多數(shù)注釋是多余的,我們應(yīng)該通過寫出更好的代碼來代替更多注釋。
注釋的確有其用途,但大部分情況下,程序員在濫用注釋。我是反對(duì)夾雜在代碼間的注釋的,我認(rèn)為注釋應(yīng)當(dāng)從代碼中獨(dú)立出來——通常被稱為文檔。
請(qǐng)看下面一段代碼。
復(fù)制代碼 代碼如下:
/* /static/market/checkout.js
2014.7.2 create by orzfly
2014.7.29 update by jysperm: fixbugs
TODO: 這段代碼中注釋太多了,需要移除一些 -- jysperm
*/
var raw_products = req.query['products'].split(',');
// 商品 ID 的數(shù)組
var products = []
// 過濾每個(gè)參數(shù)
for(var i = 0, i raw_products.length, i++) {
if (!raw_products[i])
return;
// 前端傳來的數(shù)據(jù)中居然會(huì)有空格
if (!raw_products[i].trim())
return
/* 2014.7.22: 現(xiàn)在可以使用非數(shù)字 ID 了
// 略過非數(shù)字條目
if (isNan(raw_products[i].trim().toFixed()))
return;
*/
products.push(raw_products[i].trim().toFixed());
}
// 總錢數(shù)
var sum = 0;
// 計(jì)算每個(gè)商品的總錢數(shù)
for(var i = 0, i products.length, i++) {
// 從數(shù)據(jù)庫中查商品信息
var data = db.product.byID(products[i]);
// TODO: 誰來寫一下沒查到商品的情況
// 把商品的價(jià)格加到總錢數(shù)上, a += b 是 a = a + b 的縮寫
sum += data.price;
}
你居然花了一半的時(shí)間在讀注釋上面,這是多么浪費(fèi)生命的事情,在代碼中每加一行注釋,都會(huì)增加代碼的閱讀成本——即使閱讀者已經(jīng)了解了注釋所要傳達(dá)的精神;同時(shí)也會(huì)增加維護(hù)成本:修改這段代碼的人不得不連同注釋一起修改——而且你不能確定他到底會(huì)不會(huì)這么做。
所以只有當(dāng)非常必要的情況下,才應(yīng)該添加注釋,而且應(yīng)當(dāng)言簡意賅。注釋不應(yīng)當(dāng)解釋一段代碼在做什么,因?yàn)檫@是每個(gè)合格的程序員都應(yīng)該知道的事情,而是應(yīng)該解釋這段代碼為什么要這樣做。
由此引出幾種明顯不應(yīng)該添加的注釋:
本應(yīng)由版本控制系統(tǒng)記錄的信息、對(duì)代碼的評(píng)論,以及不是很重要的 TODO.
代碼并不是全部,一個(gè)但凡靠譜一點(diǎn)的項(xiàng)目,都應(yīng)當(dāng)有自己的版本控制系統(tǒng),除了記錄代碼差異之外,還應(yīng)該有工單和 Issue 的功能。
閱讀代碼的人通常不需要了解幾個(gè)程序員之間的恩怨,很多時(shí)候也不關(guān)心這段代碼的歷史,這些信息只會(huì)把代碼拖得越來越長。
廢棄的代碼
被棄用的代碼應(yīng)該被刪掉,這些代碼會(huì)非常影響閱讀,而且它們一般又很長。
在絕大多數(shù)情況下,被棄用的代碼不會(huì)重新派上用場,即使出現(xiàn)了少數(shù)情況,你也可以從版本控制系統(tǒng)中找到它們。
對(duì)變量和函數(shù)名的解釋
這種情況下顯然你需要一個(gè)更恰當(dāng)?shù)拿?,如果這個(gè)標(biāo)識(shí)符有一個(gè)比較小的作用于,你可以使用一個(gè)比較長的名字以便容納更多信息。
例如上文中的:
products 應(yīng)改為 products_id
sum 應(yīng)改為 total_amount
data 應(yīng)改為 product_record
對(duì)語法的解釋,以及顯而易見的事情
例如上文中的「把商品的價(jià)格加到總錢數(shù)上, a += b 是 a = a + b 的縮寫」,這顯然是任何一個(gè)人都知道的事情。
也許有人愿意通過寫這樣的注釋來梳理思路:
復(fù)制代碼 代碼如下:
// 過濾參數(shù):
// 去掉 ID 里的空格
// 去掉非數(shù)字 ID
// 循環(huán)每一個(gè)商品:
// 去數(shù)據(jù)庫查記錄
// 把商品的價(jià)格加到總錢數(shù)上
但是當(dāng)代碼寫完的時(shí)候記得刪掉。
對(duì)邏輯塊的概括
例如上文中的「過濾每個(gè)參數(shù)」和「計(jì)算每個(gè)商品的總錢數(shù)」,這情況下通常是你沒有對(duì)邏輯進(jìn)行抽象,具體表現(xiàn)就是像下面這樣:
復(fù)制代碼 代碼如下:
// 首先有 25 行代碼去做事情 A
// 然后有 5 行代碼去做事情 B
// 這里有 90 行代碼去做事情 C
// 最后有 45 行代碼去做事情 D
這導(dǎo)致你需要一些注釋來分割這四個(gè)部分。如果這四個(gè)部分都是一個(gè)函數(shù)調(diào)用的話,那么函數(shù)名本身就是對(duì)邏輯的一種解釋,讀者可以快速地找到函數(shù) B, 而不必在前 25 行中搜索做事情 B 的五行代碼。
綜上,我對(duì)這段代碼的改善意見如下:
復(fù)制代碼 代碼如下:
var filterProductID = function(raw_products_id) {
result = []
raw_products_id.forEach(function(product_id) {
if (product_id and product_id.trim())
products_id.push(product_id.trim().toFixed());
});
return result;
};
var getPriceOfProduct = function(id) {
var product_record = db.product.byID(products[i]);
if (product_record)
return product_record.price;
else
return 0;
};
var products_id = filterProductID(req.query['products'].split(','));
var tatol_amount = 0;
products_id.forEach(function(product_id) {
tatol_amount += getPriceOfProduct(product_id);
});
雖然我在以一段虛構(gòu)的,刻意編造的代碼來佐證我的觀點(diǎn),但我相信在實(shí)際的項(xiàng)目中,同樣可以通過改善代碼來減少注釋,而且總體上來講會(huì)節(jié)約更多的時(shí)間和精力。
您可能感興趣的文章:- jQuery 表格隔行變色代碼[修正注釋版]
- HTML代碼中標(biāo)簽的全部屬性 中文注釋說明
- JavaScript 事件監(jiān)聽實(shí)例代碼[兼容IE,firefox] 含注釋
- FCKeditor 源代碼分析附中文注釋
- asp.net畫曲線圖(折線圖)代碼 詳細(xì)注釋
- Javascript 倒計(jì)時(shí)源代碼.(時(shí).分.秒) 詳細(xì)注釋版
- PHP壓縮html網(wǎng)頁代碼(清除空格,換行符,制表符,注釋標(biāo)記)
- 網(wǎng)頁中返回頂部代碼(多種方法)另附注釋說明