也说魔数与魔字符串
???????? 看了敘遠(yuǎn)兄寫的.net中的魔字符串,還有birdsome的評論,頗有啟發(fā)。
???????? 所謂魔數(shù)和魔字符串就是在代碼中直接使用某一個數(shù)字或者字符串,而不是常量。譬如一個很簡單的根據(jù)職位計算薪水的方法:
????if?("Programmer".equals(title))?
????????return?grade?*?500?+?700;
????else?if?("Tester".equals(title))
????????return?grade?*?500?+?800;
????else?if?("Analyst".equals(title))
????????return?grade?*?800?+?1000;
}
????????在這個方法里面,"Programmer","Tester"和"Analyst"是所謂的魔字符串(Magic String),而500, 700,800和1000就是所謂的魔數(shù)(Magic Number)了。 咋一看,代碼這樣寫也沒有什么問題,但是,仔細(xì)思考一下就會發(fā)現(xiàn),如果這種隨手捻來的字符串和數(shù)字散布于程序當(dāng)中,隨處可見的話,是會有很多弊病的。我們先來分析三個魔字符串。雖然三個Magic String的意義很明顯,并不影響到代碼的可讀性,但是這樣卻增加了出錯的概率,并且忽略了具體的語義環(huán)境。我們很容易就會想到,像"Programmer"這個單詞散布在多個方法中,一個大小寫的筆誤就會產(chǎn)生bug。同時,"Programmer"在計算薪水的方法中代表著職位,但是在統(tǒng)計公司訂閱的雜志的方法中,也許就要代表一本雜志的名稱了。然而這種語義環(huán)境是無法通過一個單純的"Programmer"就能體現(xiàn)出來的。
??????? 而Magic Number的問題就更大了,首先是影響了代碼的可讀性,誰會知道500和800是薪水基數(shù),700是補(bǔ)貼呢?而且更糟糕的是,如果薪水基數(shù)發(fā)生改變的時候,那么就得找人把這些500,700,800的數(shù)字找出來一個一個地update,那可是一件夠郁悶的事情了。
?????? ?如果我們擁有一個常量定義的interface,代碼就會變漂亮起來了:
????if?(Constants.TITLE_PROGRAMMER.equals(title))?
????????return?grade?*?Constants.BASE_SALARY_LOW?+?Constants.ALLOWANCE_LOW;
????else?if?(Constants.TITLE_TESTER.equals(title))
????????return?grade?*?Constants.BASE_SALARY_LOW?+?Constants.ALLOWANCE_MEDIUM;
????else?if?(Constants.TITLE_ANALYST.equals(title))
????????return?grade?*?Constants.BASE_SALARY_HIGH?+?Constants.ALLOWANCE_HIGH;
}
??????? 從以上的分析,在一個Project里面,避免使用魔數(shù)(Magic Number)和魔字符串(Magic String)是相當(dāng)必要的。通過定義的常量去access特定的字符串和數(shù)字也已經(jīng)是軟件開發(fā)的standard。那么是不是所有的數(shù)字和字符串都應(yīng)該定義成常量呢?或許有朋友會認(rèn)為所有的數(shù)字和字符串都應(yīng)該定義成常量,但是我覺得,每個字符串確實(shí)是應(yīng)該定義成常量的,但是對于數(shù)字而言,如果數(shù)字本身的語義沒有得到延伸,那么就不應(yīng)該定義成常量。譬如數(shù)組的index就不應(yīng)該定義成變量。?像這樣的代碼:
String?building?=?address[Constants.ONE];?????????????//?在Constants這個interface中,ONE的定義為?final?int?ONE?=?1;
??????? 你一定會覺得這樣的代碼就是畫蛇添足, 因?yàn)镺NE就是1,它沒有其他特別的含義,不像上面代碼中的500和700。而且如果真的要這樣定義的話,出現(xiàn)了有上百個元素的數(shù)組的時候,那么你就得定義上百個沒有任何意義的常量了。是不是很FT呢?
??????? 總之,任何策略的使用,還是一個度最重要。
總結(jié)
- 上一篇: ZZULIOJ 1121: 电梯
- 下一篇: nginx 跨服务器显示图片,cento