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

原创文章，如有转载，请注明出处：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:
				//按键0
				break;
			case R.id.key_1:
				//按键1
				break;
			case R.id.key_2:
				//按键2
				break;
			case R.id.key_3:
				//按键3
				break;
			case R.id.key_4:
				//按键4
				break;
			case R.id.key_5:
				//按键0
				break;
			case R.id.key_6:
				//按键6
				break;
			case R.id.key_7:
				//按键7
				break;
			case R.id.key_8:
				//按键8
				break;
		}
	}

}

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