Atomic свойства с пользовательским добытчиком

Question

У меня есть класс со свойством объявлено:

@property (nonatomic, strong) NSMutableArray *links; 

Я хочу лениво инстанцирую так иметь следующий пользовательский поглотитель:

- (NSMutableArray *)links { 

    if (!_links) { 
     _links = [NSMutableArray array]; 
    } 

    return _links; 
} 

Мое приложение несколько прогрессировало и теперь объект может быть доступен из разных потоков. Изменить декларацию:

@property (atomic, strong) NSMutableArray *links; 

Это создает предупреждение компилятора: Writeable атомно свойство не может объединить синтезированный сеттер с определенным пользователем геттером.

Что я понимаю - я думаю. Мой вопрос: правильный сделать следующее, чтобы создать атомную собственность с пользовательским геттером?:

• создать свою собственную переменную экземпляра _links
• Wrap мой добытчик в @synchronized
• Создание определенного пользователем сеттер, также завернутые в @synchronized

EDIT: это код мой новый пользовательский сеттер и геттер:

- (NSMutableArray *)links { 

    if (!_links) { 
     @synchronized(self) { 
      if (!_links) { 
       _links = [NSMutableArray array]; 
      } 
     } 
    } 

    return _links;  
} 

- (void)links:(NSMutableArray *)links { 
     @synchronized(self) { 
     _links = links; 
    } 
} 

Answer 1

@property (atomic, getter=yourGetter,setter=yourSetter:) NSMutableArray *links; 

Answer 2

Я удивляюсь, почему вы хотите сеттер. И мне интересно, почему вы хотите, чтобы свойство было атомарным.

Самый простой и правильный путь, если вы хотите геттер и сеттер, чтобы написать

- (NSMutableArray*)links 
{ 
    @synchronized (self) 
    { 
     if (_links != nil) _links = [[NSMutableArray alloc] init]; 
    } 
    return _links; 
} 

- (void)setLinks:(NSMutableArray*)links 
{ 
    @synchronized (self) 
    { 
     _links = links; 
    } 
} 

Answer 3

Во-первых, просто потому, что свойство может быть доступен из нескольких потоков, не означает, что а) она должна быть атомарным, или чтобы b) сделать его атомарным, чтобы сделать его потокобезопасным.

В частности, тип свойства NSMutableArray*, что означает, что любой вызывающий абонент получает изменяемый массив и может мутировать его без ограничений, что по своей сути является небезопасным. Вы должны в основном никогда сделать недвижимость с изменяемым типом. Все изменения в свойстве должны пройти через методы доступа, чтобы вы могли контролировать или, как минимум, реагировать на изменения.

Во-вторых, ваш геттер использует double-checked lock anti-pattern. Это не безопасно на языках C, таких как Objective-C. Не делай этого.

Сказанное: вы правы, что если вы сделаете свойство атомарным, и вы реализуете либо геттер, либо сеттер, тогда вы должны реализовать оба эти объекта, и вы также отвечаете за реализацию синхронизации для обеспечения атомарности. Использование @synchronized() подходит, если вы не используете метод двойной проверки блокировки.

Но вам действительно нужно сделать класс класс потокобезопасным по отношению к этому свойству (и любые другие, к которым можно получить доступ из нескольких потоков), что требует более чем атомарного свойства.