3

我的网络活动指示器有问题，有时它会在不应该显示的时候继续显示。

我为它写了我自己的经理，然后把它换成了一个使用这样的NSAssert语句的经理......

- (void)setNetworkActivityIndicatorVisible:(BOOL)setVisible {
    static NSInteger NumberOfCallsToSetVisible = 0;
    if (setVisible)
        NumberOfCallsToSetVisible++;
    else
        NumberOfCallsToSetVisible--;

    // The assertion helps to find programmer errors in activity indicator management.
    // Since a negative NumberOfCallsToSetVisible is not a fatal error,
    // it should probably be removed from production code.
    NSAssert(NumberOfCallsToSetVisible >= 0, @"Network Activity Indicator was asked to hide more often than shown");

    // Display the indicator as long as our static counter is > 0.
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:(NumberOfCallsToSetVisible > 0)];
}

我在 SO 上找到了它，它立即指出我使用这个功能出了点问题。

我所有的网络活动都完全通过一个NSOperationQueue由单例类管理的单一运行。每个操作都是 NSOperation 的子类（实际上是 TemplateOperation 的子类，它是 NSOperation 的子类）。

无论如何，所有的下载和上传都工作正常，我都是这样做的......

- (void)sendRequest:(NSURLRequest *)request
{
    NSError *error = nil;
    NSURLResponse *response = nil;

    [[NetworkManager sharedInstance] setNetworkActivityIndicatorVisible:YES];
    NSData *data = [NSURLConnection sendSynchronousRequest:request
                                         returningResponse:&response
                                                     error:&error];
    [[NetworkManager sharedInstance] setNetworkActivityIndicatorVisible:NO];

    // other stuff...

    [self processData:data];
}

NSURLConnection重要的行是在我同步发送之前和之后。

在我发送请求之前，我将网络活动指示器设置为可见（使用我的管理器类），然后在我将其设置回不可见之后立即。

除了 NSAssert 指出，这在某处没有正常发生。

难道是从多个线程运行这个函数可能会导致问题？我怎么能解决这个问题？

4

1 回答 1

7

整数增量或减量不是线程安全的（据我所知），因此如果两个线程“同时”调用您的方法，则计数可能无法正确更新。

一种解决方案是在您的方法中添加一些同步指令（例如@synchronized）。或者您使用原子增量/减量函数：

#include <libkern/OSAtomic.h>

- (void)setNetworkActivityIndicatorVisible:(BOOL)setVisible {
    static volatile int32_t NumberOfCallsToSetVisible = 0;
    int32_t newValue = OSAtomicAdd32((setVisible ? +1 : -1), &NumberOfCallsToSetVisible);

    NSAssert(newValue >= 0, @"Network Activity Indicator was asked to hide more often than shown");
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:(newValue > 0)];
}
于 2013-05-07T13:54:32.503 回答