代码怎样review？

1. 说明
   一个好的产品开发，一定会有产品迭代这样的过程，而代码Review，即就是代码再次查看评审就显得尤为重要，那么接下来我们就来看下什么是代码Review、何时该进行代码Review、怎样对代码进行Review、对我们有什么好处等等这一系列的知识，只需要掌握以下几点，就可以对代码Review有一个比较好的认识了。

事先声明，这个是从网上找的资料，自己只是整理到一块，仅此而已，自己之前并没有这样去做，看了这篇文章后，觉得特别有道理，而且觉得人家写的特别好，对一些关键点的处理也是我们经常在项目中容易犯的问题，所以就把别人优秀的文章借鉴过来，希望把这篇好的文章分享给更多和我一样，觉得文章很不错的小伙伴。

2. 什么是代码review？
   其实就是代码再次查看评审。

3. 什么时间该去Review？
   在我们每一个小的版本迭代完后，就去Review代码。

4. 怎样对代码进行Review？
   下面是每次提交代码之前，可以参考的一份Review代码的清单，事实证明这样做可以提高代码的质量和功能的稳定性。以下可能整理的不是很全，仅供参考。

3.1 关于清理的操作
1>：页面退出时，必须要清理的一些操作
比如handler必须在onDestroy()中调用removeCallbacksAndMessages(null)来清空handler里边的消息；
2>：对于成对出现的，要养成好的习惯，比如使用广播、EventBus等都要记得注册、反注册；
3>：对于Sqlite数据库的操作，在使用完之后需要去关闭，cursor.close()及时去关闭、文件流使用完后要记得关闭、Bitmap使用后要调用recycler()方法回收；
4>：WebView使用完后记得在onDestroy中调用webview.destroy()，然后将其置为null；

3.2 是否可以进一步优化自己的代码
1>：保存在内存中的图片，是否压缩过再保存在内存中；
可能由于图片太大、或图片质量太高，导致OOM；
2>：intent传递的数据过大，导致页面跳转太慢，太大的数据可以通过持久化的形式传递，比如读写文件；
3>：频繁的操作同一个文件或者同一个数据库的操作，要考虑把它用作静态变量或者局部变量的形式缓存在内存中，用空间换时间；
4>：放在主页面的控件，是否考虑用ViewStub来优化启动速度；

3.3 要小心第三方包
1>：build.gradle远程依赖第三方包时，版本号尽量写死，不要写 + 号；
避免由于新版本的第三方包引入了新的问题
2>：导入第三方工程时记得把编码改为自己工程的编码；
3>：调用第三方包或者JDK方法时，跳进他们源码看下，是否需要添加try{}cache(Exception e)；
如果不加的话，可能会导致自己应用崩溃；
4>：系统应用添加so时，是否在固件对应的Android.mk文件上加入新增的so，否则系统可能编译不过；

@lib/armeabi/libcommon.so
@lib/armeabi/libabcdefg.so
3.4 注意成对出现的地方
1>：EventBus注册、反注册；
2>：在生命周期的回调方法中，创建和销毁的代码是否成对出现；
比如：在onCreate()中创建了mAdapter，就必须在onDestroy()中将mAdapter置为null；
还有类似成对处出现的：
onStart()、onStop()；
onResume()、onPause()；
onCreateView()、onDestroyView()；

3.5 仿内存泄露
1>：对于handler引起的异常：

public class MainActivity extends AppCompatActivity {

private Handler handler ;
private Runnable runnable ;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);

handler = new TestHandler(new WeakReference<Context>(this)) ;
runnable = new TestRunnable() ;
handler.postDelayed(runnable , 10*1000) ;

}

private static class TestHandler extends Handler{

private Context context ;
public TestHandler(WeakReference<Context> weakContext){
    this.context = weakContext.get() ;
}

}

private static class TestRunnable implements Runnable{

@Override
public void run() {

}

}

@Override
protected void onDestroy() {
super.onDestroy();
handler.removeCallbacks(runnable);
handler.removeCallbacksAndMessages(null);
}
}

一般使用弱引用，GC扫过这块区域时会回收；
2>：考虑使用Application Context而不是Activity Context

绝大部分必须使用Activity的 Context（Dialog），其余都都可以考虑使用Application Context，这样可以在不经意间避免Activity的泄露；
3>：注意Bitmap对象的回收

在bitmap对象使用完后调用recycler()方法回收
4>：对于一些成双成对出现的就养成好的习惯，注册反注册，可以有效的避免内存泄露；
5>：cursor使用完后需要关闭；

3.6 其他情况
1>：只要是变量，不管是string类型、int类型等，都需要进行非空判断；
2>：数据库修改后，需要把数据库版本号 +1；
3>：除数必须做非0判断；
4>：不要在Activity的onCreate()方法中调用PopWindow的showAsLocation()方法，因为Activity还没加载完，会报错；

4. 功能写完后，自测需要检查的地方
   4.1：凡是变量，在使用之前必须做非空判断，否则容易造成空指针问题；
   4.2：自己测试时，必须要检查的地方
   1>：按下home键再次返回看是否正常；
   2>：熄灯后屏幕再次打开会怎样；
   3>：切换其他应用再切换回来会怎样；
   4>：onResume()、onPause()是否处理好了；

4.3：从低版本升级上来，是否会有问题；
4.4：打开手机 开发者选项 -> 调试GPU过度绘制（打开，让其显示），看是否过度绘制、是否会丢掉帧；
4.5：对比看APK大小是否增大；
4.6：查看log是否关闭；
4.7：看App是否能被反编译；
4.8：跑1个小时的Monkey，测试其稳定性；
4.9：测试看是否影响启动速度；
4.10：看看是否走的https，能否被抓包；
转载：https://www.jianshu.com/p/da21ce04139d