Почему я не использую NSLock?

Question

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

Я реализовал это с помощью NSThread. Он работает красиво некоторое время, а затем не работает с «Thread 1: Program received signal:« 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]; 
} 
... 

Спасибо!

Answer 1

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

@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, ^{ 
     ... 
    }); 
} 
... 

Посмотреть

@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 и перезапустить любое использование Queue dispatch_resume

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

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); 

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