KVO 绑定导致程序冻结？

Question

我有一个奇怪的问题，涉及一个进程，该进程似乎以随机的间隔陷入等待状态，希望我能得到一些有关此问题的帮助。

当用户启动应用程序时，会显示一个窗口，该窗口收集用户的各种数据和选择列表文件条目。然后，用户按下“运行”按钮，与“运行”按钮相关的应用程序委托方法会实例化一个类 (TestBindingClass)，该类中包含一个最终将在后台运行的方法。然后它会启动第二个 xib 文件，该文件显示第二个窗口，其中包含一个空的滚动视图。作为在 windowDIdLoad 方法中启动第二个窗口的一部分，第二个窗口代码注册为 TestBindingClass 属性的观察者。最后，主窗口“run”方法执行 TestBindingClass 内的第二个线程方法，该方法将条目添加到 NSMutableArray 中，并且每次添加新条目时都会触发受监视的属性。 NSMutableArray 中的每个条目都是一行文本（字符串），它将显示在第二个 xib 滚动视图中。

TestBindingClass 是一个类的（相对）空壳，其中将进行大量计算，当到达某些检查点时将发出状态消息。这些检查点消息将显示在滚动视图中。

大多数情况下，这工作得很好。事实上，有时这很有效。其他时候，该进程似乎冻结了，并且在挂起等待循环之前仅显示一些状态行。通常这是滚动视图即将扩展超过窗口大小并且滚动条将被激活的点。奇怪的是，如果我将调试断点添加到第二个 xib 代码中的添加观察者方法中，它总是可以正常工作。

描述就这么多...让我们展示一些代码...

这是主窗口中“运行”按钮方法的代码

-(IBAction)runButtonPressed:(id)sender
{
 // do a bunch of stuff

 TestBindingClass* tempTestBindingClass = [[TestBindingClass alloc] init];

 RunResultWindowController = [[RunResultWindow alloc] initWithWindowNibName:@"RunResultWindow"];
RunResultWindowController.localTestBindingClass=tempTestBindingClass;
[RunResultWindowController showWindow:self];

[self.StatusDisplayOutput setStringValue:@"Run Complete"];

[tempTestBindingClass submitStatusStringSequence];

}

这是上面引用的 TestBindingClass 的代码。 StatusStrings 是可变数组，它将包含状态字符串列表，该列表最终应显示在通过 arraystatuscounter 属性观察此类的滚动视图中。

 //  TestBindingClass.h

 #import <Foundation/Foundation.h>

 @interface TestBindingClass : NSObject {

NSMutableArray *StatusStrings;
   int arraystatuscounter;


 }

 @property (nonatomic, retain) NSMutableArray *StatusStrings;
 @property  int arraystatuscounter;

 - (void) runStatusStringSequence:(id)param;
 - (void) submitStatusStringSequence;

 @end

这显示了 TestBindingClass 的代码，非常简单。只需一次将一个愚蠢的小字符串放入 NSMutableArray 中，并在每次将字符串添加到数组中时调用 arraystatuscounter 属性即可。

 //  TestBindingClass.m

 - (void) submitStatusStringSequence
 {

     [NSThread detachNewThreadSelector:@selector(runStatusStringSequence:) toTarget:self withObject:nil];

 }


 - (void) runStatusStringSequence:(id)param 
 {


NSMutableArray *StatusStringsAlloc = [[NSMutableArray alloc] initWithCapacity:1];
StatusStrings = StatusStringsAlloc;

[StatusStrings addObject:[NSString stringWithString:@"first string"]];
[self setArraystatuscounter:1];

[StatusStrings addObject:[NSString stringWithString:@"second string"]];
[self setArraystatuscounter:2];

[StatusStrings addObject:[NSString stringWithString:@"third string"]];
[self setArraystatuscounter:3];

[StatusStrings addObject:[NSString stringWithString:@"fourth string"]];
[self setArraystatuscounter:4];

[StatusStrings addObject:[NSString stringWithString:@"fifth string"]];
[self setArraystatuscounter:5];

[StatusStrings addObject:[NSString stringWithString:@"sixth string"]];
[self setArraystatuscounter:6];

[StatusStrings addObject:[NSString stringWithString:@"seventh string"]];
[self setArraystatuscounter:7];

[StatusStrings addObject:[NSString stringWithString:@"last string"]];
[self setArraystatuscounter:8];

[StatusStrings addObject:[NSString stringWithString:@"first string"]];
[self setArraystatuscounter:1];

[StatusStrings addObject:[NSString stringWithString:@"second string"]];
[self setArraystatuscounter:2];

[StatusStrings addObject:[NSString stringWithString:@"third string"]];
[self setArraystatuscounter:3];

[StatusStrings addObject:[NSString stringWithString:@"fourth string"]];
[self setArraystatuscounter:4];

[StatusStrings addObject:[NSString stringWithString:@"fifth string"]];
[self setArraystatuscounter:5];

[StatusStrings addObject:[NSString stringWithString:@"sixth string"]];
[self setArraystatuscounter:6];

[StatusStrings addObject:[NSString stringWithString:@"seventh string"]];
[self setArraystatuscounter:7];

[StatusStrings addObject:[NSString stringWithString:@"last string"]];
[self setArraystatuscounter:8];

 }

这是 RunResultWindows 窗口控制器代码。 TestBindingClass 的地址被添加到 ivars 中，以便它可以正确设置必要的 KVO 观察器设置。

 //  RunResultWindow.h

 @interface RunResultWindow : NSWindowController {

  NSTextView *RunResultWindowTextView;
  TestBindingClass *localTestBindingClass;
 }

 - (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change context:(void *)context;

 @property (strong) IBOutlet NSTextView *RunResultWindowTextView;
 @property (nonatomic, retain) TestBindingClass *localTestBindingClass;

 @end

以下是感兴趣的 RunResultWindow 方法。请原谅 arraycount if 语句，该语句是出于一些临时调试目的而添加到其中并且从未被删除。

 //  RunResultWindow.m methods of interest

 - (void)windowDidLoad
 {
     [super windowDidLoad];

     NSWindow *wcWindow;
     wcWindow = [self window];
     [wcWindow makeKeyAndOrderFront:self];

     NSString *teststring;
     teststring = [NSString stringWithString: @"show first time window did load "];
     [RunResultWindowTextView setString:teststring];
     [RunResultWindowTextView display];

     [localTestBindingClass addObserver:self
       forKeyPath:@"arraystatuscounter"
          options:NSKeyValueObservingOptionNew
          context:NULL];

 }

 - (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change 
    context:(void *)context
{
  NSInteger arrayCount;
  NSString* localDisplayString;
  NSString* localNewlinePlusDisplayString;
  NSTextStorage *tempTextStorage;

  tempTextStorage = [RunResultWindowTextView textStorage];

  arrayCount = [ localTestBindingClass.StatusStrings count ];
  if (arrayCount >= 1) {
    arrayCount--;
    localDisplayString = [localTestBindingClass.StatusStrings objectAtIndex:arrayCount];
    localNewlinePlusDisplayString = [@"\n" stringByAppendingString:localDisplayString];
    [tempTextStorage beginEditing];
    [tempTextStorage replaceCharactersInRange:NSMakeRange([tempTextStorage length] - 1, 0) 
                             withString:localNewlinePlusDisplayString];
    [tempTextStorage endEditing];
    [RunResultWindowTextView display];
  }

 }

Answer 1

我认为 KVO 与此没有太大关系。你所描述的“非常简单”实际上非常非常复杂。这就是罪魁祸首：-

[NSThread detachNewThreadSelector:@selector(runStatusStringSequence:) toTarget:self withObject:nil]

一旦你这样做了，你需要的不仅仅是对 Cocoa 框架的良好理解 - 你需要对多线程编程有很好的理解，并且对 Cocoa 框架如何在多线程世界中工作有很好的理解。这很复杂也很困难。

您至少需要研究这些

http: //developer.apple.com/library/ios/#documentation/General/Conceptual/ConcurrencyProgrammingGuide/Introduction/Introduction.html

http://developer.apple.com/library/mac/documentation /Cocoa/Conceptual/Multithreading/Multithreading.pdf

如果您完全删除第二个线程，我仍然不希望它一直工作（尽管它可能在某些时候工作）。您不应该直接在文本视图上调用 -display 。视图的绘制、设置绘制上下文、刷新率等均由框架为您处理。

只要在不知道当前绘图上下文状态的情况下随机调用 -display 就足以使您的应用程序崩溃（您可以轻松地绘制到任何随机的内存中）。将线程添加到混合物中，您就会遇到更多麻烦。

您的方法

- (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change context:(void *)context

在后台线程上调用，但 gui 是不是线程安全的，必须在主线程上运行，包括与文本视图等 gui 对象的任何交互。

Answer 2

这是实际有效的代码。请原谅错误的大小写以及解决方案的先前迭代中偶尔留下的一些奇怪代码。

这是主窗口中“运行”按钮方法的代码...

- (IBAction)runButtonPressed:(id)sender
{

    // do a bunch of stuff

    TestBindingClass* tempTestBindingClass = [[TestBindingClass alloc] init];

    RunResultWindowController = [[RunResultWindow alloc] initWithWindowNibName:@"RunResultWindow"];
    RunResultWindowController.localTestBindingClass=tempTestBindingClass;
    [RunResultWindowController showWindow:self];

    [self.outputfilestring1 becomeFirstResponder];

    [tempTestBindingClass submitStatusStringSequence];

}

这是上面引用的 TestBindingClass 的代码。 StatusStrings 是可变数组，它将包含状态字符串列表，该列表最终应显示在通过 arraystatuscounter 属性观察此类的滚动视图中。

 //  TestBindingClass.h

 #import <Foundation/Foundation.h>

 @interface TestBindingClass : NSObject {

NSMutableArray *StatusStrings;
   int arraystatuscounter;


 }

 @property (nonatomic, retain) NSMutableArray *StatusStrings;
 @property  int arraystatuscounter;

 - (void) runStatusStringSequence:(id)param;
 - (void) submitStatusStringSequence;

 @end

这里展示了TestBindingClass的实现代码。它只是每次将一个愚蠢的小字符串放入 NSMutableArray 中，并在每次将字符串添加到数组中时调用 arraystatuscounter 属性以启动 KVO 观察方法。第一个主要区别是我使用了上面建议的 GCD 方法...

//  TestBindingClass.m

 - (void) submitStatusStringSequence
 {
     NSString *parameterString;
    [self runStatusStringSequence: parameterString];
 }


 - (void) runStatusStringSequence:(id)param 
 {

dispatch_queue_t backgroundQueue = dispatch_queue_create("Background Queue",NULL);

dispatch_async(backgroundQueue, ^{ 

NSMutableArray *StatusStringsAlloc = [[NSMutableArray alloc] initWithCapacity:1];
StatusStrings = StatusStringsAlloc;

[StatusStrings addObject:[NSString stringWithString:@"first string"]];
[self setArraystatuscounter:1];

[StatusStrings addObject:[NSString stringWithString:@"second string"]];
[self setArraystatuscounter:2];

[StatusStrings addObject:[NSString stringWithString:@"third string"]];
[self setArraystatuscounter:3];

[StatusStrings addObject:[NSString stringWithString:@"fourth string"]];
[self setArraystatuscounter:4];

[StatusStrings addObject:[NSString stringWithString:@"fifth string"]];
[self setArraystatuscounter:5];

[StatusStrings addObject:[NSString stringWithString:@"sixth string"]];
[self setArraystatuscounter:6];

[StatusStrings addObject:[NSString stringWithString:@"seventh string"]];
[self setArraystatuscounter:7];

[StatusStrings addObject:[NSString stringWithString:@"last string"]];
[self setArraystatuscounter:8];

[StatusStrings addObject:[NSString stringWithString:@"first string"]];
[self setArraystatuscounter:1];

[StatusStrings addObject:[NSString stringWithString:@"second string"]];
[self setArraystatuscounter:2];

[StatusStrings addObject:[NSString stringWithString:@"third string"]];
[self setArraystatuscounter:3];

[StatusStrings addObject:[NSString stringWithString:@"fourth string"]];
[self setArraystatuscounter:4];

[StatusStrings addObject:[NSString stringWithString:@"fifth string"]];
[self setArraystatuscounter:5];

[StatusStrings addObject:[NSString stringWithString:@"sixth string"]];
[self setArraystatuscounter:6];

[StatusStrings addObject:[NSString stringWithString:@"seventh string"]];
[self setArraystatuscounter:7];

[StatusStrings addObject:[NSString stringWithString:@"last string"]];
[self setArraystatuscounter:8];

dispatch_release(backgroundQueue);

});


 }

这是 RunResultWindows 窗口控制器代码。 TestBindingClass 的地址被添加到 ivars 中，以便它可以正确设置必要的 KVO 观察器设置。另外，我强制该方法在主线程上运行，因为我想它是在后台线程上运行的，因为它是由已经在后台线程上运行的进程调用的。

@interface RunResultWindow : NSWindowController {

    NSTextView *RunResultWindowTextView;
    TestBindingClass *localTestBindingClass;

}

- (IBAction)FinishButtonPush:(id)sender;
- (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change context:(void *)context;

@property (strong) IBOutlet NSTextView *RunResultWindowTextView;
@property (nonatomic, retain) TestBindingClass *localTestBindingClass;

@end

以下是感兴趣的 RunResultWindow 方法。请原谅 arraycount if 语句，该语句是为了一些临时调试目的而添加的，并且从未被删除。请注意，它被强制到主线程。

- (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change 
        context:(void *)context
{    

    dispatch_queue_t mainQueue = dispatch_get_main_queue();
    dispatch_async(mainQueue, ^{ 

    NSInteger arrayCount;
    NSString* localDisplayString;
    NSString* localNewlinePlusDisplayString;
    NSTextStorage *tempTextStorage;

    tempTextStorage = [RunResultWindowTextView textStorage];

    arrayCount = [ localTestBindingClass.StatusStrings count ];
    if (arrayCount >= 1) {
        arrayCount--;
        localDisplayString = [localTestBindingClass.StatusStrings objectAtIndex:arrayCount];
        localNewlinePlusDisplayString = [@"\n" stringByAppendingString:localDisplayString];
        [tempTextStorage beginEditing];
        [tempTextStorage replaceCharactersInRange:NSMakeRange([tempTextStorage length] - 1, 0) 
                                 withString:localNewlinePlusDisplayString];
        [tempTextStorage endEditing];
        [RunResultWindowTextView setNeedsDisplay:YES];}

    });
}

非常感谢 hooleyhoop 为我们指明了正确的方向。