最近在做项目的时候,碰到方法的前面和类的前面有时会出现@SuppressLint或者@SuppressWarnings这样的黄色警告,看起来很不舒服,于是上网搜集了一些相关资料。发现这些警告的出现其实是由于我们编写代码时的一些不规范的写法导致,解决这些问题其实是能提高我们程序的安全性、可用性和性能的。接下来就让我为大家一一说明。
1、@SuppressLint(“DrawAllocation”)
原因:我们都知道View及其子类的OnDraw(Canvas canvas)方法会实时调用来更新界面的,然而有时我们会在onDraw(Canvas canvas)方法中实例化对象(如Paint paint=new Paint()),这样就会影响我们程序的效率。
解决办法:将这些对象改为类的成员变量。
2、@SuppressWarnings(“rawtypes”)和@SuppressWarnings({ “unchecked”, “rawtypes” })
不规范写法:Class clazz = Class.forName(“android.view.Display”);
正确写法:Class<?> clazz = Class.forName("android.view.Display");
3、@SuppressLint(“HandlerLeak”)
原因:Handler在Android中用于消息的发送与异步处理,常常在Activity中作为一个匿名内部类来定义,此时Handler会隐式地持有一个外部类对象(通常是一个Activity)的引用。当Activity已经被用户关闭时,由于Handler持有Activity的引用造成Activity无法被GC回收,这样容易造成内存泄露。
解决办法:将其定义成一个静态内部类(此时不会持有外部类对象的引用),在构造方法中传入Activity并对Activity对象增加一个弱引用,这样Activity被用户关闭之后,即便异步消息还未处理完毕,Activity也能够被GC回收,从而避免了内存泄露。
public static class MyHandler extends Handler {
private WeakReference<Activity> reference
public MyHandler(Activity activity) {
reference = new WeakReference<Activity>(activity);
}
@Override
public void handleMessage(Message msg) {
if (reference.get() != null) {
switch (msg.what) {
case 0:
// do something...
break;
default:
// do something...
break;
}
}
}
}
另一种情况:
不规范的写法:
private Handler mHandler = new Handler() {
@Override
public void handleMessage(Message msg) {
};
};
正确的写法:
private Handler mHandler = new Handler(new Handler.Callback() {
@Override
public boolean handleMessage(Message msg) {
return false;
}
});
4、@SuppressLint(“SimpleDateFormat”)
不规范写法:SimpleDateFormat format = new SimpleDateFormat(pattern);
正确写法:SimpleDateFormat format = new SimpleDateFormat(pattern, Locale.getDefault());
5、@SuppressLint(“DefaultLocale”)
不规范写法:String lower = string.toLowerCase();
boolean b = “String”.toUpperCase().equals(“STRING”);
正确写法:String lower = string.toLowerCase(Locale.getDefault());
boolean b = “String”.toUpperCase().equals(“STRING”);