不良代码展示-很多判断的写法-给新手

原创文章，如有转载，请注明出处：http://blog.csdn.net/yihui823/article/details/6801695

今天Review代码，看到了一串很“漂亮”的代码。这里把代码改了一下，贴给新手们，作为一个教训吧。
漂亮的意思，是有很工整的缩进。

package com.hoperun;import android.view.View;import android.view.View.OnClickListener;/** * 数字键盘监听器 */public class NumListener  implements OnClickListener {public void onClick(View v) {if (v.equals(btnNum0)) {//按键0} else { if (v.equals(btnNum1)) {//按键1} else {if (v.equals(btnNum2)) {//按键2} else {if (v.equals(btnNum3)) {//按键3} else {if (v.equals(btnNum4)) {//按键4} else {if (v.equals(btnNum5)) {//按键5} else {if (v.equals(btnNum6)) {//按键6} else {if (v.equals(btnNum7)) {//按键7} else {if (v.equals(btnNum8)) {//按键8} else {if (v.equals(btnNum9)) {//按键9} else {if (v.equals(btnNumAdd)) {//按键加法} else {if (v.equals(btnNumSubtraction)) {//按键减法} else {if (v.equals(btnNumEquals)) {//按键等于}}}}}}}}}}}}} }}

其实，这么写是完全没有必要的。这么多的else，其实可以叠在else if里面，如下：

package com.hoperun;import android.view.View;import android.view.View.OnClickListener;/** * 数字键盘监听器 */public class NumListener  implements OnClickListener {public void onClick(View v) {if (v.equals(btnNum0)) {//按键0} else { if (v.equals(btnNum1)) {//按键1} else if (v.equals(btnNum2)) {//按键2} else if (v.equals(btnNum3)) {//按键3} else if (v.equals(btnNum4)) {//按键4} else if (v.equals(btnNum5)) {//按键5} else if (v.equals(btnNum6)) {//按键6} else if (v.equals(btnNum7)) {//按键7} else if (v.equals(btnNum8)) {//按键8} else if (v.equals(btnNum9)) {//按键9} else if (v.equals(btnNumAdd)) {//按键加法} else if (v.equals(btnNumSubtraction)) {//按键减法} else if (v.equals(btnNumEquals)) {//按键等于}}}

其实这个效率不高。其实传入的参数View，就是被按住的那个按钮，不需要用equals判断，直接用“ == ”来判断就可以了。

当然，最好的还不是用“ == ”，而是用switch - case.

package com.hoperun;import android.view.View;import android.view.View.OnClickListener;/** * 数字键盘监听器 */public class NumListener  implements OnClickListener {public void onClick(View v) {switch (v.getId()) {case R.id.key_0://按键0break;case R.id.key_1://按键1break;case R.id.key_2://按键2break;case R.id.key_3://按键3break;case R.id.key_4://按键4break;case R.id.key_5://按键0break;case R.id.key_6://按键6break;case R.id.key_7://按键7break;case R.id.key_8://按键8break;}}}

仅给新人学习，高手勿喷。