swiftthread-safetyswift4

Thread safe singleton in swift


I have and Application which has a singleton that stores information across the whole app. However, this is creating some data race issues when using the singleton from different threads.

Here there is a very dummy and simplistic version of the problem:

Singleton

class Singleton {
    static var shared = Singleton()

    var foo: String = "foo"
}

Use of the singleton (from the AppDelegate for simplicity)

class AppDelegate: UIResponder, UIApplicationDelegate {

    var window: UIWindow?


    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?) -> Bool {
        // Override point for customization after application launch.

        DispatchQueue.global().async {
            var foo = Singleton.shared.foo // Causes data race
        }

        DispatchQueue.global().async {
            Singleton.shared.foo = "bar"   // Causes data race
        }

        return true
    }
}

Is there any way to ensure that a singleton is thread safe, so it can be used from anywhere in the app without having to worry about which thread you are in?

This question is not a duplicate of Using a dispatch_once singleton model in Swift since (if I understood it correctly) in there they are addressing the problem of accessing to the singleton object itself, but not ensuring that the reading and writing of its properties is done thread safely.


Solution

  • Thanks to @rmaddy comments which pointed me in the right direction I was able to solve the problem.

    In order to make the property foo of the Singleton thread safe, it need to be modified as follows:

        class Singleton {
    
        static let shared = Singleton()
    
        private init(){}
    
        private let internalQueue = DispatchQueue(label: "com.singletioninternal.queue",
                                                  qos: .default,
                                                  attributes: .concurrent)
    
        private var _foo: String = "aaa"
    
        var foo: String {
            get {
                return internalQueue.sync {
                    _foo
                }
            }
            set (newState) {
                internalQueue.async(flags: .barrier) {
                    self._foo = newState
                }
            }
        }
    
        func setup(string: String) {
            foo = string
        }
    }
    

    Thread safety is accomplished by having a computed property foo which uses an internalQueue to access the "real" _foo property.

    Also, in order to have better performance internalQueue is created as concurrent. And it means that it is needed to add the barrier flag when writing to the property.

    What the barrier flag does is to ensure that the work item will be executed when all previously scheduled work items on the queue have finished.