关于 obj-c 中的单实例模式，静态分析器总给我报一个奇怪的提示

[adow]

最近我在用Xcode 静态分析器时发现我所有单实例代码中的

+(id)allocWithZone:(NSZone *)zone

都有警告:

Object with a +0 retain count returned to caller where a +1 (owning) retain count is expected

意思是allocWithZone返回的object必须是retain+1的，而单实例中的-(release)是直接返回self的，所以没有retain。

我记得以前版本的Xcode中是没有这个提示的，我不知道是不是有别的实现方法了吗？

我现在的写法类似下面这样：

@implementation MySingleton

static MySingleton *sSharedSingleton = nil;

+ (MySingleton *)sharedSingleton
{
if (sSharedSingleton == nil)
{
sSharedSingleton = [[super allocWithZone:NULL] init];
}
return sSharedSingleton;
}

+ (id)allocWithZone:(NSZone *)zone
{
return [[self sharedSingleton] retain];
}

- (id)copyWithZone:(NSZone *)zone
{
return self;
}

- (id)retain
{
return self;
}

- (unsigned)retainCount
{
return NSUIntegerMax; //denotes an object that cannot be released
}

- (oneway void)release
{
//do nothing
}

- (id)autorelease
{
return self;
}

@end

[chisj]

我们的单例模式是这么写的，是我最喜欢的写法。给LZ一个参考：

static id shareInstance = nil;

+(void)initialize {
if (self == [EUHttpCache class]) {
shareInstance = [[self alloc] init];
}
}

+(EUHttpCache *)shareInstance {
return shareInstance;
}

[fly2never]

用 dispatch_once吧,标准写法

[DropperY]

是因为retaincount 写死了 NSUIntegerMax 的原因吧，预期的retaincount应该比原来大1

[alexrezit]

這是什麼鬼寫法...

[chchwy]

樓主的寫法已經過時了。

[ultragtx]

不能吐槽更多 这都是在哪学的奇葩写法

[satanwoo]

@ultragtx objetive-c设计模式里面好像就是这么写的。。。。

[adow]

这样的写法我是从一本obj-c设计模式书上看来的，今天我google了一圈，现在的确都是用 dispatch_once 的写法了，我也改成了这样。不过这个问题在Xcode5.2下依旧。我想应该不是初始化时的问题吧。

[ylovesy]

上面的单例不好，应该这样：

+(instancetype)sharedInstance
{
static dispatch_once_t once;
static MyObject *sharedInstance = nil;

dispatch_once(&once,^{
sharedInstance = [[MyObject alloc] init];
});

return sharedInstance;
}

[ultragtx]

这不只是单例的问题 你这里面除了sharedSingleton以外的所有method都不需要

[jesse0628]

@ultragtx 他想达到的目的是，把单例当做正常非单例使用，但不管你alloc还是retain、release，永远只存在一份真正的内存对象。

[satanwoo]

楼上正解。而且这样的写法还要考虑到单例中的继承问题。。

[ultragtx]

@jesse0628 按我理解这么写的动机是为了弥补语法层面不支持而加上的保护措施吧

但这么做也就违反了内存管理的规则 而且在ARC下也没法这么写

[hengxin196]

`#import "__class__.h"

@implementation __class__

+ (__class__ *)__accessor__
{
static dispatch_once_t once;
static __class__ *__singleton__;
dispatch_once(&once, ^ { __singleton__ = [[__class__ alloc] init]; });
return __singleton__;
}

@cursor

@end`

[wangccddaa]

+ (id)sharedList
{
static id _sharedInstance = nil;
static dispatch_once_t once;

dispatch_once(&once, ^{
_sharedInstance = [[self alloc] init];
});
return _sharedInstance;
}

上一份我们的代码 把类名改成 self 是不是就是万能单例了，哈哈~

[krafttuc]

https://github.com/mattt/Xcode-Snippets/blob/master/singleton.m

目前大家都趋向用 `dispatch_onc` 了

[msching]

lz这个写法多线程环境下会生成多个实例。。