Singleton Class NetworkManager trong MVCNetworking của Apple có đúng không?

Question

Dưới đây là liên kết tới các mẫu mã http://developer.apple.com/library/ios/#samplecode/MVCNetworking/Introduction/Intro.html

Dưới đây là đoạn mã từ tập tin NetworkManager.m

+ (NetworkManager *)sharedManager 
// See comment in header. 
{ 
    static NetworkManager * sNetworkManager; 

    // This can be called on any thread, so we synchronise. We only do this in 
    // the sNetworkManager case because, once sNetworkManager goes non-nil, it can 
    // never go nil again. 

    if (sNetworkManager == nil) { 
     @synchronized (self) { 
      sNetworkManager = [[NetworkManager alloc] init]; 
      assert(sNetworkManager != nil); 
     } 
    } 
    return sNetworkManager; 
} 

Rõ ràng là có chủ đề vấn đề an toàn ở đây. Hai phiên bản NetworkManager có thể được tạo khi có nhiều hơn một luồng. Vì vậy, Apple đã phạm sai lầm, phải không?

Answer 1

Có, không đúng. Bắt đầu với sNetworkManager là nil và xem xét hai chuỗi T1 và T2.

Một có thể, nếu không, kịch bản là:

T1: Determines (sNetworkManager == nil) is true 
T2: Determines (sNetworkManager == nil) is true 
T1: Takes the @synchronized lock 
    Creates a NetworkManager 
    Sets sNetworkManager 
    Releases the lock 
T2: Takes the @synchronized lock 
    Creates a NetworkManager 
    Sets sNetworkManager, LEAKING the first one 
    Releases the lock 

This question có một số cách an toàn hơn để làm việc đó.

Answer 2

Không có lỗi trong mã này. Chỉ một sNetworkManager được tạo ra vì một lý do đơn giản là từ "tĩnh" được sử dụng. Từ khóa tĩnh được sử dụng ở đây để xác định biến là toàn cầu nhưng chỉ hiển thị với hàm đó. Biến được phân bổ trong lần gọi đầu tiên của + (NetworkManager *) sharedManager sau đó nó không còn null và không được khởi tạo nữa.

Answer 3

Có, bạn đã đúng. Nó sẽ có vấn đề trong môi trường tương tranh. Cách tốt hơn là sử dụng kiểm tra lại trước khi alloc:

+ (NetworkManager *)sharedManager 
{ 
    static NetworkManager * sNetworkManager; 
    if (sNetworkManager == nil) { 
     @synchronized (self) { 
      if (sNetworkManager == nil) { 
       sNetworkManager = [[NetworkManager alloc] init]; 
       assert(sNetworkManager != nil); 
      } 
     } 
    } 
    return sNetworkManager; 
} 

Và có rất nhiều cách để viết singleton sử dụng Ojbective-C, kiểm tra bài đăng này: What should my Objective-C singleton look like?

Cập nhật

BobCromwell là đúng . Các double check lock không được khuyến khích bởi táo, các tài liệu táo của Threading Programming Guide:

Một khóa kiểm tra lại là một nỗ lực để giảm chi phí tham gia một khóa bằng cách kiểm tra các tiêu chí khóa trước khi tham gia khóa. Bởi vì ổ khóa đôi kiểm tra là có khả năng không an toàn, hệ thống không cung cấp hỗ trợ rõ ràng cho họ và việc sử dụng chúng là discouraged.`