Почему у меня не работает NSLock? - PullRequest
Новая
       26

Почему у меня не работает NSLock?

3 голосов
/ 27 декабря 2011

Я пишу код для рендеринга и поворота изображения, детали которого одновременно рассчитываются и обновляются. Он работает без ошибок в одном потоке (с отображаемой ссылкой), но выглядит неуклюже, и я не хочу, чтобы вычисления запускались по отображаемой ссылке. Поэтому я хочу выполнить весь код, связанный с OpenGL, в главном потоке (с отображаемой ссылкой) и все вычисления во втором потоке (выполняя цикл while (YES)).

Я реализовал это с помощью NSThread. Некоторое время он прекрасно работает, а затем завершается с ошибкой «Поток 1: Программа получила сигнал:« EXC_BAD_ACCESS »» во время glDrawArrays, а иногда имеет странные вспышки графики. Это то, чего я ожидал, если основной поток считывает данные уровня модели в то время, когда второй поток перезаписывает их.

Затем я определил NSLock в объекте модели и заблокировал его для всех записей (в моем классе моделей) и чтения (в моем классе представлений) ... но это может привести к той же ошибке, а графика все еще время от времени странные вспышки.

Я что-то здесь не так сделал или моя проблема где-то еще?

Во-вторых, как правильно остановить второй поток в этом случае? Ссылка на класс NSThread предлагает использовать отмену, проверку isCancelled и выход, если это так, но при этом также говорится, что вызывающего выхода следует избегать.

Вот модификации кода - в моем классе контроллера (я использую XCode 4.2 с ARC; все мои ивары неатомные): 

@interface MyController : NSObject {
    NSThread *calcThread;
    ...
}
// (I do not give it an @property or @synthesize line)

@implementation MyController
- (void) awakeFromNib {
    calcThread = [[NSThread alloc] initWithTarget:self 
            selector:@selector(calcLoop:) object:nil];
    [calcThread start];
    ...
}
- (void) calcLoop:(id)arg { 
    @autoreleasepool {
        while (YES)
            [myModel calculate];
    }
}
...

Я поместил NSLock в свой модельный класс: 

@interface MyModel : NSObject {
    NSLock* oLock;
    ...
}
@property (nonatomic, strong) NSLock* oLock;

@implementation myModel
-(id) init {
    oLock = [[NSLock alloc] init];
    ...
}
-(void) changeModelAppearance {
    [oLock lock];
    ...
    [oLock unlock];
}
...

и, на мой взгляд, класс: 

@implementation MyView
-(void) modelUpdated:(NSNotification *) notification {
// modelUpdated is called via the NSNotificationCenter
    MyModel* myModel = (MyModel*) [notification object];
    [myModel.oLock lock];
    ... // update OpenGL structures with data from myModel
    [myModel.oLock unlock];
}
...

Спасибо!

1 Ответ

3 голосов
/ 14 января 2012

Я думаю, в этом случае вам было бы намного легче использовать грандиозную центральную диспетчеризацию. 

@interface MyController : NSObject { // Not use why you're not inheriting from NSController here.
    dispatch_queue_t calQueue;
    ...
}

- (void) awakeFromNib {
    calcQueue = dispatch_queue_create("com.yourApp.calc", DISPATCH_QUEUE_SERIAL);
    dispatch_async(calcQueue, ^{
        while(YES) // This will peg the CPU at 100%
            [myModel calculate];
    });
}

модель класса 

@interface MyModel : NSObject {
    dispatch_queue_t modelQueue;
    ...
}
@property dispatch_queue_t modelQueue;

@implementation myModel
-(id) init {
    modelQueue = dispatch_queue_create("com.yourApp.model", DISPATCH_QUEUE_SERIAL);
}

-(void) dealloc {
    dispatch_release(modelQueue);
}

-(void) changeModelAppearance {
    dispatch_async(modelQueue, ^{
        ...
    });
}
...

View 

@implementation MyView
-(void) modelUpdated:(NSNotification *) notification {
// modelUpdated is called via the NSNotificationCenter
    MyModel* myModel = (MyModel*) [notification object];
    dispatch_async(model.modelQueue, ^{
        ... // update OpenGL structures with data from myModel 
    });
}
...

Для приостановки любой из очередей, которые вы просто вызываете dispatch_suspend, и для перезапуска любой очереди используйте dispatch_resume

Если вы используете таймер вместо бесконечного цикла, вы можете уменьшить количество используемого процессора. 

calcTimer = dispatch_source_create(DISPATCH_SOURCE_TYPE_TIMER, 0, 0, dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0));
dispatch_source_set_timer(calcTimer, DISPATCH_TIME_NOW, DT, 1000);
dispatch_source_set_event_handler(calcTimer, ^{
    ...
});
dispatch_resume(calcTimer);

Это будет использовать гораздо меньше ресурсов процессора.

...