最近組內(nèi)做代碼評審，看到了如下一段代碼，這段代碼從功能上、甚至效率上來講都沒有問題，但從可讀性和可維護性來講，可優(yōu)化的地方一目了然：函數(shù)‘foobar’接收‘op_type’為輸入?yún)?shù)，該函數(shù)的調(diào)用完全可以放在整個switch之后，做一處調(diào)用即可，這樣后期‘foobar’有任何變化，這段代碼只需相應(yīng)修改一處代碼即可。

　　這段代碼出自一個有幾年工作經(jīng)驗的程序員之手，實屬不該 —— 即使一個編程初學者，也應(yīng)該知道盡量“避免冗余代碼”這一基本原則。

　　switch(op_type){

　　case ADD:

　　/*

　　*Codes for ADD

　　*/

　　foobar(ADD);

　　break;

　　}

　　case MODIFY:

　　{

　　/*

　　*Codes for MODIFY

　　*/

　　foobar(MODIFY);

　　break;

　　}

　　case DELETE:

　　{

　　/*

　　*Codes for DELETE

　　*/

　　foobar(DELETE);

　　break;

　　}

　　default:

　　{

　　log("invalid op_type");

　　return -1;

　　}

　　}

　　

?

總結(jié)

以上是生活随笔為你收集整理的编程基本原则——避免冗余代码的全部內(nèi)容，希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網(wǎng)站內(nèi)容還不錯，歡迎將生活随笔推薦給好友。