我在网上找到了那个单例模式。在我看来它有很多可以优化的地方。

-在 sharedMySingleton 方法中，不需要调用 retain 吗？我不确定...
- 如果不是，为什么在 allocWithZone 中有保留？
-@synchronized有什么用。 NSAssert 认为这个 block 可以被调用很多次，所以如果是，应该有更多的代码来释放以前的内存，或者清楚地退出 block 而不仅仅是 NSAsserting，如果不是，为什么会有这个 NSAssert？
- sharedMySingleton 和 alloc 之间的链似乎很奇怪。我给自己写了类似的东西:

+(MySingleton*)sharedMySingleton
{
    @synchronized([MySingleton class])
    {
        if (_sharedMySingleton == nil) _sharedMySingleton = [[self alloc] init];
        return _sharedMySingleton;
    }

    return nil;
}

+(id)alloc
{
    @synchronized([MySingleton class])
    {
        return [super alloc];
    }

    return nil;
} 

单例模式

#import "MySingleton.h"

@implementation MySingleton

// ##########################################################################################################
// ######################################## SINGLETON PART ##################################################
// ##########################################################################################################
static MySingleton* _sharedMySingleton = nil;

// =================================================================================================
+(MySingleton*)sharedMySingleton
// =================================================================================================
{
    @synchronized([MySingleton class])
    {
        if (_sharedMySingleton == nil) [[self alloc] init];
        return _sharedMySingleton;
    }

    return nil;
}

// =================================================================================================
+(id)alloc
// =================================================================================================
{
    @synchronized([MySingleton class])
    {
        NSAssert(_sharedMySingleton == nil, @"Attempted to allocate a second instance of a singleton.");
        _sharedMySingleton = [super alloc];
        return _sharedMySingleton;
    }

    return nil;
} 

+ (id)allocWithZone:(NSZone *)zone  { return [[self sharedMySingleton] retain]; }
- (id)copyWithZone:(NSZone *)zone   { return self; }
- (id)retain                        { return self; }
- (NSUInteger)retainCount           { return NSUIntegerMax;  /* denotes an object that cannot be released */}
- (oneway void)release              { /* do nothing */ }
- (id)autorelease                   { return self; }

// ##########################################################################################################
// ##########################################################################################################
// ##########################################################################################################

// =================================================================================================
-(id)init 
// =================================================================================================
{   
    if (!(self = [super init])) return nil;

    return self;
}

// =================================================================================================
-(void) dealloc
// =================================================================================================
{
    [super dealloc];
}

// =================================================================================================
-(void)test 
// =================================================================================================
{
    NSLog(@"Hello World!");
}

@end

最佳答案

您根本不应该使用这种模式(它适用于您几乎从不需要的非常特殊的单例情况，即使在那种情况下您通常也不应该使用它)。

What should my Objective-C singleton look like? 讨论了许多好的模式, 但自从 GCD 发布以来，它们中的大多数都已过时。在现代版本的 Mac 和 iOS 中，您应该使用 Colin Barrett 在链接问题中给出的以下模式:

+ (MyFoo *)sharedFoo
{
    static dispatch_once_t once;
    static MyFoo *sharedFoo;
    dispatch_once(&once, ^{ sharedFoo = [[self alloc] init]; });
    return sharedFoo;
}

我只在此处复制它而不是将问题标记为重复，因为旧问题的最高评分答案已过时。

关于iphone - Objective-C - 优化这个单例模式？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/10066288/