Лучшая практика при реализации copyWithZone:

Question

Я пытаюсь прояснить несколько вещей в моей голове о реализации copyWithZone:Кто-нибудь может прокомментировать следующее...

// 001: Crime is a subclass of NSObject.
- (id)copyWithZone:(NSZone *)zone {
    Crime *newCrime = [[[self class] allocWithZone:zone] init];
    if(newCrime) {
        [newCrime setMonth:[self month]];
        [newCrime setCategory:[self category]];
        [newCrime setCoordinate:[self coordinate]];
        [newCrime setLocationName:[self locationName]];
        [newCrime setTitle:[self title]];
        [newCrime setSubtitle:[self subtitle]];
    }
    return newCrime;
}

// 002: Crime is not a subclass of NSObject.
- (id)copyWithZone:(NSZone *)zone {
    Crime *newCrime = [super copyWithZone:zone];
    [newCrime setMonth:[self month]];
    [newCrime setCategory:[self category]];
    [newCrime setCoordinate:[self coordinate]];
    [newCrime setLocationName:[self locationName]];
    [newCrime setTitle:[self title]];
    [newCrime setSubtitle:[self subtitle]];
    return newCrime;
}

В 001 году:

1. Лучше всего написать имя класса напрямую [[Crime allocWithZone:zone] init] или я должен использовать [[[self Class] allocWithZone:zone] init]?

2. Это нормально использовать [self month] для копирования iVars или я должен получить доступ к iVars напрямую, т.е. _month?

Answer 1

1. Вы должны всегда использовать [[self class] allocWithZone:zone] чтобы убедиться, что вы создаете копию, используя соответствующий класс. Пример, который вы приводите для 002, показывает, почему: подклассы будут вызывать [super copyWithZone:zone] и ожидайте получить экземпляр соответствующего класса, а не экземпляр суперкласса.

2. Я получаю доступ к ivars напрямую, поэтому мне не нужно беспокоиться о каких-либо побочных эффектах, которые я мог бы добавить к установщику свойств (например, генерировать уведомления) позже. Имейте в виду, подклассы могут переопределить любой метод. В вашем примере вы отправляете два дополнительных сообщения на каждый ivar. Я бы реализовал это следующим образом:

Код:

- (id)copyWithZone:(NSZone *)zone {
    Crime *newCrime = [super copyWithZone:zone];
    newCrime->_month = [_month copyWithZone:zone];
    newCrime->_category = [_category copyWithZone:zone];
    // etc...
    return newCrime;
}

Конечно, копируете ли вы ивары, сохраняете их или просто назначаете, они должны отражать то, что делают сеттеры.

Answer 2

Поведение копирования по умолчанию copyWithZone: Метод с предоставленными SDK объектами - "мелкая копия". Это означает, что если вы позвоните copyWithZone: на NSString объект, он создаст мелкую копию, но не глубокую копию. Разница между мелкой и глубокой копией:

Мелкая копия объекта будет только копировать ссылки на объекты исходного массива и помещать их в новый массив.

Глубокая копия фактически скопирует отдельные объекты, содержащиеся в объекте. Это делается путем отправки каждого отдельного объекта copyWithZone: сообщение в вашем пользовательском методе класса.

INSHORT: чтобы получить мелкую копию, звоните retain или же strong на все переменные экземпляра. Чтобы получить глубокую копию звоните copyWithZone: на все переменные экземпляра в вашем пользовательском классе copyWithZone: реализация. Теперь это ваш выбор.

Answer 3

Как насчет этого, который реализует глубокое копирование:

/// Class Foo has two properties: month and category
- (id)copyWithZone:(NSZone *zone) {
    Foo *newFoo;
    if ([self.superclass instancesRespondToSelector:@selector(copyWithZone:)]) {
        newFoo = [super copyWithZone:zone];
    } else {
        newFoo = [[self.class allocWithZone:zone] init];
    }
    newFoo->_month = [_month copyWithZone:zone];
    newFoo->_category = [_category copyWithZone:zone];
    return newFoo;
}

Answer 4

Это моя модель.

#import <Foundation/Foundation.h>
@interface RSRFDAModel : NSObject


@property (nonatomic, assign) NSInteger objectId;

@property (nonatomic, copy) NSString *name;

@property (nonatomic, strong) NSArray<RSRFDAModel *> *beans;


@end


#import "RSRFDAModel.h"

@interface RSRFDAModel () <NSCopying>

@end

@implementation RSRFDAModel 


-(id)copyWithZone:(NSZone *)zone {
    RSRFDAModel *model = [[[self class] allocWithZone:zone] init];

    model.objectId = self.objectId;
    model.name = self.name;
    model.beans = [self.beans mutableCopy];

    return model;
}

@end