程序員,拒絕垃圾注釋!
你是否有過復(fù)查程序時(shí)發(fā)現(xiàn)有些注釋毫無用處?程序注釋是為了提高代碼的可讀性,為了讓原作者以外的其他開發(fā)人員更容易理解這段程序,但更多時(shí)候,程序們們寫出的注釋語句都毫無意義。
51CTO推薦閱讀:程序員的十大技術(shù)煩惱
文章把這些讓人郁悶的注釋方式歸為了五類,同時(shí)把寫出這些注釋的程序員也歸為了五類。希望讀了這篇文章后你感覺自己不屬于其中的任何一種類型。
1. 高傲的程序員
- public class Program
- {
- static void Main(string[] args)
- {
- string message = “Hello World!”; // 07/24/2010 Bob
- Console.WriteLine(message); // 07/24/2010 Bob
- message = “I am so proud of this code!”; // 07/24/2010 Bob
- Console.WriteLine(message); // 07/24/2010 Bob
- }
- }
這種程序員是如此的欣賞自己的程序,以至于不得不在每行代碼上都署上自己的大名。應(yīng)該讓版本控制系統(tǒng)來提供程序變更的信息,他這樣做一眼看去并不能說明誰對(duì)這行代碼負(fù)責(zé)。
2. 過時(shí)的程序員
- public class Program
- {
- static void Main(string[] args)
- {
- /* 這段程序已經(jīng)不再有用
- * 因?yàn)槲覀儼l(fā)現(xiàn)千年蟲問題只是一場(chǎng)虛驚
- * 我們的系統(tǒng)不會(huì)恢復(fù)到1/1/1900 */
- //DateTime today = DateTime.Today;
- //if (today == new DateTime(1900, 1, 1))
- //{
- // todaytoday = today.AddYears(100);
- // string message = “The date has been fixed for Y2K.”;
- // Console.WriteLine(message);
- //}
- }
- }
如果一段程序不再有用(比如廢棄了),那就刪了它吧——不要被幾行沒用的注釋搞的程序混亂不堪。即使你可能以后重用這段代碼,你也可以使用版本控制系統(tǒng),用它把你的程序恢復(fù)到以前的樣子。
3. 天真的程序員
- public class Program
- {
- static void Main(string[] args)
- {
- /* 這個(gè)程序是用來在屏幕上
- * 循環(huán)打印1百萬次”I Rule!”
- * 每次輸出一行。循環(huán)計(jì)數(shù)
- * 從0開始,每次加1。
- * 當(dāng)計(jì)數(shù)器等于1百萬時(shí),
- * 循環(huán)就會(huì)停止運(yùn)行*/
- for (int i = 0; i < 1000000; i++)
- {
- Console.WriteLine(“I Rule!”);
- }
- }
- }
基本的編程語法規(guī)則我們大家都知道——我們不需要“編程入門”。你不需要浪費(fèi)時(shí)間來解釋一個(gè)顯而易見的東西,我們更希望知道的是你的程序功能——那是浪費(fèi)空間了。
4. 傳奇的程序員
- public class Program
- {
- static void Main(string[] args)
- {
- /* 有一天我在大街上的一家星巴克里
- * 和銷售部的Jim討論問題,他告訴我
- * 銷售代表是依據(jù)以下的比例提取傭金的。
- * 周五: 25%
- * 周三: 15%
- * 其它日期: 5%
- * 我是否告訴你過我點(diǎn)了一個(gè)卡拉梅
- * 鐵咖啡和兩份的Espresso?
- */
- double price = 5.00;
- double commissionRate;
- double commission;
- if (DateTime.Today.DayOfWeek == DayOfWeek.Friday)
- {
- commissionRate = .25;
- }
- else if (DateTime.Today.DayOfWeek == DayOfWeek.Wednesday)
- {
- commissionRate = .15;
- }
- else
- {
- commissionRate = .05;
- }
- commission = price * commissionRate;
- }
- }
如果你不得不在注釋里寫明需求,那也不要提到人名。銷售員Jim很可能在公司里不再是銷售。而且大多數(shù)讀到這段注釋的程序員未必都知道Jim是誰。你描述的是實(shí)際情況但跟我們的內(nèi)容不相干,所以就省掉吧。
5. 未來程序員
- public class Program
- {
- static void Main(string[] args)
- {
- //TODO: 將來我會(huì)修復(fù)這個(gè)問題 – 07/24/1995 Bob
- /* 我知道這個(gè)問題很難解決而且
- * 我現(xiàn)在依賴于這個(gè)Contains函數(shù),但
- * 我以后會(huì)用一種更有意義,更
- * 優(yōu)雅的方式打印這段代碼。
- * 我只是現(xiàn)在沒時(shí)間。
- */
- string message = “An error has occurred”;
- if(message.Contains(“error”))
- {
- throw new Exception(message);
- }
- }
- }
這種注釋是一種集大成者,它包含了上面所說的注釋的所有問題。TODO注釋在一個(gè)項(xiàng)目最初的開發(fā)階段是非常有用的,但這個(gè)注釋看起來是在好幾年前的產(chǎn)品程序里的——它證明了程序有問題。如果程序有問題需要解決,馬上解決,不要拖到日后再解決。
你是否在你的程序里還見到過其它種沒有意義的或討厭的注釋?
【編輯推薦】