新参与一个项目之中，分配了一个任务很快完成后就开始读团队中其余队友的代码，算是熟悉一下项目的整体流程及实现方式。并且在读代码的过程中发现了不少的问题或者毛病，在此进行总结以勉励自己不要犯这种低级的错误。（我保持着经常阅读自己代码并修改重构自己代码的习惯，因为在你某一时刻完成工作任务的时候，感觉当时自己代码写的很完美，这是正常的现象---完成功能后的自我满足感！不过如果你过段时间再次看自己当初以为写的很牛叉的代码可以发现好多吐槽点，好多可以优化重构的地方），保持这种经常阅读自己代码并重构的习惯真的可以提高好多。也可以经常阅读别人的代码看别人的代码风格有何借鉴之处。有一个有意思的现象就是自己看自己的代码给满分，当看别人写的代码的时候估计有时候会吐槽：卧槽，怎么写的这么垃圾。如果把自己的代码比作女友的话，就是自己的女友比别人的漂亮，完全跟现实相反，现实中总觉得别人的女友漂亮，（写到这一句自嘲一下，特么你一个单身程序狗拿女朋友做比喻木有说服力哇）。闲言少叙，开始

1)Button点击监听方法onClick(View view) 中方法参数的不合理应用

让我们看下面onClick一段代码：

switch (v.getId()) {		case R.id.detail_type_1:typeIntent.putExtra("type", tags[0]);startActivity(typeIntent);break;case R.id.detail_type_2:typeIntent.putExtra("type", tags[1]);startActivity(typeIntent);break;case R.id.detail_type_3:typeIntent.putExtra("type", tags[2]);startActivity(typeIntent);break;case R.id.detail_type_4:typeIntent.putExtra("type", tags[3]);startActivity(typeIntent);break;case R.id.detail_type_5:typeIntent.putExtra("type", tags[4]);startActivity(typeIntent);break;}}

相信很多人写过onClick方法中然后直接v.getId然后在各个case语句写对应的业务逻辑，这无可厚非，关键是上面的代码通篇case语句就是根据点击的各个TextView来跳转到一个activity，除了参数之外全部代码跟一个模子刻出来一样。跟写这段代码的小伙伴沟通过后，我直接修改的代码提交git如下

    onClick(Vivew   view){TextView textView = (TextView)view;String type = textView.getText().toString();typeIntent.putExtra("type",type);startActivity(typeIntent);}

结果令我气愤的是我让他pull了一下代码，结果他更新代码过后又改回之前的那种写法又重新提交了，让我很是无语！！！！

这虽然是一个小的知识点，但是我们要知道 onClick方法参数中的view，并不只是获取getId来用的。甚至有些人都不明白这个方法总的view其实就是你点击的那个view本身！！！

2) OnClickListener或者OnFocusChanageListener对象的重复创建

比如下面的这段代码,以及对应的：

btn1.setOnClickListener(new MyOnClickListener);
btn2.setOnClickListener(new MyOnClickListener);
btn3.setOnClickListener(new MyOnClickListener);
btn4.setOnClickListener(new MyOnClickListener);
btn5.setOnClickListener(new MyOnClickListener);

上述设置onClickListener的代码看起来代码很整洁，但是却是创建了五个同样的对象，类似于这种代码我归因于ctrl+c ,ctrl+v的顺手而为,根本没有意识到这段代码的可优化之处（万恶的粘贴复制），其实优化起来很简单：

	MyOnClickListener clickListener = new MyOnClickListener() btn1.setOnClickListener(clickListener);btn2.setOnClickListener(clickListener);btn3.setOnClickListener(clickListener);btn4.setOnClickListener(clickListener);btn5.setOnClickListener(clickListener);

3)List<View>集合的不必要使用

有些小伙伴开发的时候习惯用List<View>集合来保存布局文件中的View,然后在某个地方调用list.get(index)来获取指定位置的view并对之进行操作，其实个ListView完全是多余的，错误案例就像下面的布局（布局文件的id命名在此是随便写的）：

	        <LinearLayoutandroid:id="@+id/parent_view"android:layout_width="match_parent"android:layout_height="wrap_content"               android:orientation="horizontal" ><TextViewandroid:id="@+id/txt1"style="@style/actors_name_style" /><TextViewandroid:id="@+id/txt2"                         style="@style/actors_name_style" /><TextViewandroid:id="@+id/txt3"style="@style/actors_name_style" /><TextViewandroid:id="@+id/txt4"                        style="@style/actors_name_style" /></LinearLayout>

某小伙伴写在初始化TextView之后，使用了如下代码来保存TextView，并从集合里面循环取各个TextView并为它们这是Text字符

				List<TextView> lists = new ArrayList<TextView>();list.add(txt1);list.add(txt2);list.add(txt3);list.add(txt4);//使用的时候是String txts[] = {"aaa","bbbb","cccc","dddd"}for(int i=0;i<txts.length;i++){list.get(i).setText(txts[i])}

其实我们知道，ViewGroup里面本身就有一个数组来保存ViewGroup的childAt(index)获取ViewGroup中指定位置的childView,并且如果index非法的话返回努力；所以上面的List集合完全没必要存在，而且通读小伙伴的代码，充斥了三四个类似的List<View>，所以上面的代码我建议他写成了如下形式：

LinearLayout linearLayout = (LinearLayout)findViewById(R.id.<span style="font-family: Arial, Helvetica, sans-serif;">parent_view</span>)//使用的时候是String txts[] = {"aaa","bbbb","cccc","dddd"}for(int i=0;i<txts.length;i++){linearLayout.getChildAt(i).setText(txts[i])}

删除了类中类似的集合类，即减少了代码量，也可以节省点内存开销，何乐而不为呢？

4)for循环的重复：

问题代码如下：

int pageSize = 20;if(list.size()>pageSize){for(int i=0;i<pageSize;i++){Object obj = list.get(i);doSomething(obj);}}else{for(int i=0;i<list.size();i++){Object obj = list.get(i);doSomething(obj);}}

用当前页和数据集合的大小进行比对，如果集合数据总数大于当前页数pageSize，就用当前页数pageSize作为循环条件来执行for循环，否则就直接循环集合的全部数据；

上面这段代码的可读性以及以后的操作修改上的缺点大家都能看出来，下面直接贴上优化后的代码：

     int size = list.size()>pageSize?pageSize:list.size();for(int i=0;i<size;i++){Object obj = list.get(i);doSomething(obj);}

5)不必要的if else判断

这个问题的出现，应该是敲代码的时候手指头随着脑袋的思路随手写出来的，问题不大，但是能优化还是优化一下，减少不必要的代码量

问题代码：

if(hasFocus){someView.setSelected(true);}else{someView.setSelected(false);}

优化后的代码：

  someView.setSelected(hasFocus);

6）Dialog或者AlertDialog的重复创建

某些童鞋在创建dialog的时候，初始化Dialog的代码和显示show方法的代码总是放在一块儿，导致了页面中的Dialog每次show的时候又调用Build方法创建了一遍，也算是个优化点吧！

问题代码：

private void showDialog() {AlertDialog mDialog = new AlertDialog.Builder(context, R.style.SearchDialogBackground).create();dialogWindow = mDialog.getWindow();WindowManager.LayoutParams lp = dialogWindow.getAttributes();// 设置透明度为0.3lp.alpha = 0.98f;lp.dimAmount = 0.0f;// activity背景不变暗dialogWindow.setAttributes(lp);dialogWindow.setGravity(Gravity.LEFT | Gravity.CENTER);mDialog.show();ViewGroup dialogRootView = (ViewGroup) View.inflate(context, R.layout.xxx, null);mDialog.setContentView(dialogRootView);doSomthing()}

优化后的代码：

AlertDialog mDialog;private void initDialog() {mDialog = new AlertDialog.Builder(context, R.style.SearchDialogBackground).create();dialogWindow = mDialog.getWindow();WindowManager.LayoutParams lp = dialogWindow.getAttributes();// 设置透明度为0.3lp.alpha = 0.98f;lp.dimAmount = 0.0f;// activity背景不变暗dialogWindow.setAttributes(lp);dialogWindow.setGravity(Gravity.LEFT | Gravity.CENTER);}private void showDialog() {mDialog.show();ViewGroup dialogRootView = (ViewGroup) View.inflate(context, R.layout.xxx, null);mDialog.setContentView(dialogRootView);doSomthing()}

这样在初始化对象的时候调用initDialog()方法完成了初始化工作之后，就可以直接调用showDialog()方法了，不用每次都对dialog进行重复性的创建工作；说道Dialog的时候，某些童鞋喜欢定义一个额外的isShowing方法来判断Dialog框是否关闭，在调用showDialog的时候设置isShowing变量为true，然后关闭的时候设施dialog的dimisslistener来设置isShowing为fale，其实也没那个必要，因为Dialog本身就提供了isShowing()方法来判断是否关闭；

问题代码：

class MyDialog{boolean isShowing;initDialog(){     dialog.setOnDismissListener(new OnDismissListener() {Overridepublic void onDismiss(DialogInterface dialog) {isShowing = false;//该Listener只做了这么一件事儿});
}public boolean isShowing() {
return isShowing;
}public void exitDilog() {</span>exitDialog.dismiss();
}}

修改后的代码：

	class MyDialog{boolean isShowing;initDialog(){     //去掉了setOnDimissListener()}public boolean isShowing() {return exitDialog.isShowing();}
}

当然项目中的代码问题不一而足，这里只是简单的列举几个个人觉得有必要的地方总结出来；比如一个方法过长啦，变量命名不规范啦这类的问题就不在此占用篇幅说明了，本篇就写这么多吧，有不当的地方欢迎批评指正，共同学习和提高

题外话：

  什么是工作经验？私人觉得经验！= 一个个完成的项目（最起码不成正比）。比如别人问你工作经验，你就回答在某某项目中做了神马工作，我觉得这并不是经验这个词能真正代表的东西。在开发软件的过程中，大到解决一个难题的方法（怎么解决的，为什么会出现这个问题）总结，小到代码细节的优化处理等 这才是你通过项目能够真实提高自己的地方，这才是开发一个软件中对程序员来说最重要的经验！我经常完成一个任务后阅读自己的代码（或者边开发边重构），能重构的地方重构，有时候会提出一个子类，有时候会简单的只是删除一个没有被用到的import或者变量；有时候会把某个全局变量删除，用一个局部变量搞定，甚至是看到一个过长的方法去提炼出来若干个子方法来减少方法过长不便于阅读的entity等等。通过重构自己的代码能发现好多问题，并修改之，感觉并不比开发完一个功能差多少，甚至满足感犹有过之。

如果你在开发一个软件的过程中对自己写的代码写过了就扔掉，不进行阅读修改的话，那么在开发下一个新的项目的时候问题代码仍然会在新的项目中体现出来。这只能在你简历上增加了开发软件的个数，但是真实的经验来说，你并没有吸收和提高多少。当然在开发软件的过程中可以注意很多东西《程序员你惹不起》这本书有一节《一个完整的软件项目，最起码要关注哪些》上面列举出来的一条条感觉很有启发。

   虽然现在自己的写代码的水平还是很菜，距离大神还有九重天那样的差距，但是一点点的提高迟早有一天也会“会当凌绝顶，一览众山小”吧，期待之，努力之