今天使用带参数的单例模式的时候遇到一个问题,先看代码:
private APTest(Context c)
{
mContext = c;
}
public static APTest singleton(Context context)
{
synchronized (lock)
{
if(gInstance == null)
{
gInstance = new APTest(context);
}
return gInstance;
}
}
如上图,这是一个常见的懒汉单例写法,如果没有传参的话。但是如果这里带上参数的话,就有可能存在逻辑问题。
设想下面的情况:
1、当单例第一次被调用的时候,参数context如果为null,此时单例仍然会被创建;
2、如果下一次单例被调用的时候,参数已经有值了,那么该值永远不能被赋值成功,因为此时“gInstance”已经被创建,单例已经不为null!
造成这个问题,是因为参数的赋值是在单例的创建里面进行的,这里是因为违背了单一职责原则。修改的方法也简单,这里只需要将context从单例的创建中分离出来,如下:
public static APTest singleton(Context context)
{
synchronized (lock)
{
if(gInstance == null)
{
gInstance = new APTest();
}
gInstance.mContext = context;
return gInstance;
}
}
这样,很简单的将两步操作分开即可,但如果不注意,很容易给自己挖坑。