简单的多线程应用程序有时会失败

Question

我正在做这个多线程应用程序只是为了看看@synchronized 指令是如何工作的。我读过如果所有线程都具有与@synchronized 的参数相同的对象，那么它们都在同一个锁上等待。所以我想使用 self 作为参数，因为它对所有线程都是相同的。
在这个应用程序中，有一个文本字段被所有线程多次编辑。我不关心性能，这只是一个测试，所以我没有将@synchronized 指令放在 for 之前，但在里面。

我使用的属性：

@property (weak) IBOutlet NSTextField *textField;
@property (nonatomic, copy) NSNumber* value;
@property (nonatomic,copy) NSMutableArray* threads;

代码：

- (void)applicationDidFinishLaunching:(NSNotification *)aNotification
{
    value= @0;
    textField.objectValue= value;
    for(int i=0; i<10; i++)
    {
        NSThread* thread=[[NSThread alloc]initWithTarget: self selector: @selector(routine:) object: nil];
        [threads addObject: thread];
        [thread start];
    }
}

- (void) routine : (id) argument
{
    for(NSUInteger i=0; i<100; i++)
    {
        @synchronized(self)
        {
            value= @(value.intValue+1);
            textField.objectValue= value;
        }
    }
}

有时应用程序成功，我看到 1000 作为文本字段值。但有时不是，我担心这是饥饿，我在文本字段上看不到任何内容，它是空的。我尝试了调试，但很难看看有什么问题，因为失败的标准对我来说似乎很随意，有时它就可以正常工作。

解决方案

@synthesize threads,value, textField;

- (void)applicationDidFinishLaunching:(NSNotification *)aNotification
{
    value= @0;
    threads=[[NSMutableArray alloc]initWithCapacity: 100];
    for(NSUInteger i=0; i<100; i++)
    {
        NSThread* thread=[[NSThread alloc]initWithTarget: self selector: @selector(routine:) object: nil ];
        [threads addObject: thread];
        [thread start];
    }
}

- (void) routine : (id) arg
{
    for(NSUInteger i=0; i<1000; i++)
    {
        @synchronized(self)
        {
            value= @(value.intValue+1);
            [textField performSelectorOnMainThread: @selector(setObjectValue:) withObject: value waitUntilDone: NO];
        }
    }
}

Answer 1

您总是在后台线程中更新您的 UI。这是不好的。你应该这样做；

 (void)applicationDidFinishLaunching:(NSNotification *)aNotification
{
    value= @0;
    textField.objectValue= value;
    for(int i=0; i<10; i++)
    {
        NSThread* thread=[[NSThread alloc] initWithTarget: self selector: @selector(routine:) object: nil];
        [threads addObject: thread];
        [thread start];
    }
}

- (void) routine : (id) argument
{
    for(NSUInteger i=0; i<100; i++)
    {
        @synchronized(self)
        {
            value= @(value.intValue+1);
            [textField performSelectorOnMainThread:@selector(setObjectValue:) withObject:value waitUntilDone: NO];    
        }
    }
}

尝试锁定实例并使用 sleep 方法使其更流畅。

要在后台线程中锁定一个变量，首先要锁定它，然后设置它的值并解锁它；

[NSLock lock];
value=@(value.intValue+1)
[NSLock unlock];

但是，您已经拥有@synchronized，这使得它可以保护同时从多个线程访问的变量。我认为@synchonized 块更容易理解。

在这种情况下，睡眠更合适。如果您在线程中放置 2 秒的睡眠时间，您可以看到 textField 每 2 秒更改一次，并且更明显且更有意义。

[NSThread sleepForTimeInterval:2000] // 更新 textField 后将其放入循环中并查看效果。

另外，最好创建一个runloop并在一些runloop中执行线程，这是一个更好的做法。

Answer 2

您正在从非主线程访问NSTextField，它是NSView 的子类。 That is not a safe thing to do。结果未定义；有时它似乎有效，有时却无效。