10

Maybe this question requires a bit of context.

I've been working on my persistence layer using Core Data and found out that Core Data isn't thread-safe and thus requires NSManagedObjectContext to be confined to each one thread only.

So my approach is to create custom background thread NSManagedObjectContext which executes fetching, saving etc, while also to create main thread NSManagedObjectContext which will be used to get NSManagedObject from fetched NSManagedObjectId and pass it to caller method.

By default, Xcode generates template code related to Core Data using lazy var for all NSManagedObjectContext, NSManagedObjectModel etc.

So my question is whether to

use the lazy var instantiation approach for creating NSManagedObjectContext, provided that lazy var initiates an object for each thread trying to access (not thread-safe?)

or

declare separate variables for NSManagedObjectContext in each thread and make all thread-related methods to reference two different NSManagedObjectContext provided that lazy var is thread-safe(?) and created only once when it is accessed regardless of thread.

Thank you in advance!

edit: Anyone who is struggling with Core Data concurrency issue, this article lays out a very nice design pattern to work with as pointed out by Aaron in the comment below!

1 Answer 1

31

From The Swift Programming Language: Properties:

If a property marked with the lazy modifier is accessed by multiple threads simultaneously and the property has not yet been initialized, there is no guarantee that the property will be initialized only once.

lazy var is not thread safe. You can use

  • dispatch_once (runs once per lifetime of the app)
  • a constant (let)
  • the nested struct pattern (typically used for singletons)

for thread safety. (See this question for some examples.)

You could also employ your own locking using NSRecursiveLock but that's probably not as efficient as dispatch_once.

14
  • Thanks for your answer. So does that mean that lazy var will create different instances for different threads trying to access its value? Commented Apr 21, 2015 at 2:29
  • 1
    No, it means if two threads try to access it at the same time, the second thread might receive a partially initialized object. And since NSManagedObjectContext isn't thread safe, you shouldn't access it from different threads. Commented Apr 21, 2015 at 2:31
  • 1
    Okay. So I should declare different instance of NSManagedObjectContext for each thread that I wish to use and enforce methods that execute in background thread to only access NSManagedObjectContext that was created in the same background thread previously? Thanks. Commented Apr 21, 2015 at 2:40
  • 4
    @AaronBrager A small remark though, dispatch_once uses a static predicate so it'll run once per lifetime of the app. If the lazy var is an instance variable you might want to use the other locking techniques you mentioned or a dispatch semaphore instead.
    – hennes
    Commented Sep 8, 2015 at 16:45
  • 1
    @hennes there's computational cost associated with locking, so it doesn't make sense for it to be the default. Would be nice to make it an option though (like lazy threadsafe var). Also, many vars are typically UIKit objects, which must only be accessed on the main thread for the most part. The Swiftier solution, though, is to just pass around immutable value types so you don't need locking. Commented Sep 8, 2015 at 18:34

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.