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

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

　　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;

　　}

　　}

　　

?

總結

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

如果覺得生活随笔網站內容還不錯，歡迎將生活随笔推薦給好友。