NSLock "контроллер" класс - PullRequest
Новая
       41

NSLock "контроллер" класс

2 голосов
/ 15 сентября 2011

Я пишу многопоточное приложение для iPhone и использую NSLock для того, чтобы некоторые операции (например, загрузка звуков из файла) работали как атомарные. Чтобы упростить получение блокировок из разных частей моего приложения, я написал следующий класс, который позволяет мне блокировать и разблокировать блокировки, просто передавая NSString с именем. Если такой блокировки не существует, он создает ее и сохраняет на будущее. Я проверил его, и он, кажется, работает нормально (поскольку он просто обеспечивает доступ к объектам NSLock и не меняет их поведение).

Мой вопрос: нормально ли иметь и использовать такой класс, или у меня есть некоторое недопонимание в концепции замков? 

Locker.h

#import <Foundation/Foundation.h>

@interface Locker : NSObject { }

+ (void) purgeLocks;
+ (NSLock*) lockWithName: (NSString*) name;
+ (void) lock: (NSString*) name;
+ (void) unlock: (NSString*) name;
+ (BOOL) tryLock: (NSString*) name;

@end



Locker.m

#import "Locker.h"

static NSMutableDictionary* locks = nil;

@implementation Locker

+ (void) initialize {
    locks = [[NSMutableDictionary dictionary] retain];
}

+ (void) purgeLocks {
    [locks release];
    [Locker initialize];
}

+ (NSLock*) lockWithName: (NSString*) name {
    NSLock* lock = nil;
    @synchronized([Locker class]) {
        lock = [locks objectForKey: name];
        if(!lock) {
            lock = [[[NSLock alloc] init] autorelease];
            [lock setName: name];
            [locks setObject: lock forKey: name];
        }
    }
    return lock;
}

+ (void) lock: (NSString*) name {
    [[Locker lockWithName: name] lock];
}

+ (void) unlock: (NSString*) name {
    [[Locker lockWithName: name] unlock];
}

+ (BOOL) tryLock: (NSString*) name {
    return [[Locker lockWithName: name] tryLock];
}

@end

Ответы [ 2 ]

1 голос
/ 15 сентября 2011

@synchronized медленно , избегайте этого.Выделите блокировку в + initialize и используйте ее вместо этого.Но я бы выбрал синглтон вместо использования методов класса, но это более или менее дело вкуса.

Главный недостаток вашего подхода заключается в том, что вам нужно заблокировать блокировку, чтобы получить другую блокировку.,В сильно многопоточном приложении эта мета-блокировка (ваша текущая @synchronized) может стать узким местом производительности, даже если все потоки хотят получить доступ к различным блокировкам.

0 голосов
/ 15 сентября 2011

Это выглядит довольно тяжело для меня.Вы уверены, что не можете упростить дизайн?(GCD приходит на ум.) Почти каждое iOS-приложение является многопоточным, и ИМХО редко можно встретить таких помощников блокировки, как вы написали.Если есть место, где светит принцип KISS , я уверен, что это многопоточность.

...