类跟踪和使用NSSet限制实例

Question

我希望我的类检测到一个新实例与某个现有实例等效(相对于isEqual:和hash),并且只创建唯一实例.这是我认为可以完成工作的代码,但是我担心它会做一些我无法发现的愚蠢行为......

假设它是一个NSURLRequest子类,如下所示:

// MyClass.h
@interface MyClass : NSMutableURLRequest
@end

// MyClass.m

@implementation MyClass

+ (NSMutableSet *)instances {

    static NSMutableSet *_instances;
    static dispatch_once_t once;

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

- (id)initWithURL:(NSURL *)URL {

    self = [super initWithURL:URL];
    if (self) {
        if ([self.class.instances containsObject:self])
            self = [self.class.instances member:self];
        else
            [self.class.instances addObject:self];
    }
    return self;
}


// Caller.m
NSURL *urlA = [NSURL urlWithString:@"http://www.yahoo.com"];

MyClass *instance0 = [[MyClass alloc] initWithURL: urlA];
MyClass *instance1 = [[MyClass alloc] initWithURL: urlA];  // 2

BOOL works = instance0 == instance1;  // works => YES, but at what hidden cost?

问题:

1. 在init中对self进行的第二次任务看起来很奇怪,但并非疯狂.或者是吗？
2. 只是一厢情愿的编码认为第二个alloc(of instance1)被神奇地清理干净了吗？

Answer 1

1. 这不是疯了,但在手动保留/释放模式下,您需要self事先发布,否则每次运行此方法时都会泄漏未初始化的对象.在ARC中,将自动为您发布原始实例.

2. 见#1.

顺便说一句,对于那些通常只停留在一个答案的读者,bbum的答案包括一个线程安全实现的完整工作示例.强烈建议任何人做一个这样做的课程.

Answer 2

想一个更好的方法(线下的原始答案),假设你真的想通过URL唯一.如果没有,这也演示了同步原语的使用.

@interface UniqueByURLInstances:NSObject
@property(strong) NSURL *url;
@end

@implementation UniqueByURLInstances
static NSMutableDictionary *InstanceCache()
{
    static NSMutableDictionary *cache;
    static dispatch_once_t onceToken;
    dispatch_once(&onceToken, ^{
        cache = [NSMutableDictionary new];
    });
    return cache;
}

static dispatch_queue_t InstanceSerializationQueue()
{
    static dispatch_queue_t queue;
    static dispatch_once_t onceToken;
    dispatch_once(&onceToken, ^{
        queue = dispatch_queue_create("UniqueByURLInstances queue", DISPATCH_QUEUE_SERIAL);
    });
    return queue;
}

+ (instancetype)instanceWithURL:(NSURL*)URL
{
    __block UniqueByURLInstances *returnValue = nil;
    dispatch_sync(InstanceSerializationQueue(), ^{
        returnValue = [InstanceCache() objectForKey:URL];
        if (!returnValue)
        {
            returnValue = [[self alloc] initWithURL:URL];
        }
    });
    return returnValue;
}

- (id)initWithURL:(NSURL *)URL
{
    __block UniqueByURLInstances* returnValue = self;
    dispatch_sync(InstanceSerializationQueue(), ^{
        returnValue = [InstanceCache() objectForKey:URL];
        if (returnValue) return;

        returnValue = [super initWithURL:URL];
        if (returnValue) {
            [InstanceCache() setObject:returnValue forKey:URL];
        }

        _url = URL;
    });

    return returnValue;
}

- (void)dealloc {
    dispatch_sync(InstanceSerializationQueue(), ^{
        [InstanceCache() removeObjectForKey:_url];
    });
    // rest o' dealloc dance here 
}
@end

警告:上面输入了SO - 从未运行过.我可能搞砸了.假设ARC已启用.是的,当使用工厂方法时,它最终会查找URL两次,但是在分配和初始化的噪声中应该丢失额外的查找.这样做意味着开发人员可以使用工厂或初始化程序,但仍然可以看到唯一的实例,但是当该URL的实例已经存在时,将不会对工厂方法的执行进行分配.

(如果您不能通过URL唯一,那么请返回NSMutableSet并完全跳过工厂方法.)

查克说的是什么,但还有一些补充说明:

像这样重构你的代码:

+(NSMutableSet*)instances
{
    static NSMutableSet *_instances;
    dispatch_once( ...., ^{ _instances = [[NSMutableSet alloc] init];});
    return instances;
}

然后在您想要访问时调用该方法instances.它将所有代码本地化,并将其隔离+initialize(这不是什么大问题).

如果您的类可能是从多个线程实例化的,那么您将希望用同步原语包围check-allocate-or-return.我建议一个dispatch_queue.