使一个属性强大，非客观的c

Question

我有一个多视图应用程序，并使用一个对象来跟踪我的登录用户。我的User.h看起来像这样

@interface User : NSObject

@property (strong, nonatomic) NSDictionary *data;

@property (weak, nonatomic) NSString *uid;
@property (weak, nonatomic) NSString *firstName;
@property (weak, nonatomic) NSString *lastName;
@property (weak, nonatomic) NSString *dob;
@property (weak, nonatomic) NSString *gender;
@property (weak, nonatomic) NSString *avatarURL;
@property (assign, nonatomic) NSInteger status;

- (void)setPropertiesWith:(NSDictionary *)data;

而User.m看起来像这样

#import "User.h"

@implementation User
/*
 * set properties
 */
- (void)setPropertiesWith:(NSDictionary *)data{
    self.data = data;

    self.uid = self.data[@"uid"];
    self.firstName = self.data[@"firstName"];
    self.lastName = self.data[@"lastName"];
    self.dob = self.data[@"dob"];
    self.gender = self.data[@"gender"];
    self.status = [[self.data valueForKeyPath:@"status"] intValue];
    self.avatarURL = self.data[@"avatarURL"];
}

@end

我的数据很弱，但在其中一个视图中它会变为空 - 我相信ARC正在释放它。如果我错了，请纠正我。

我有两个问题：

1. 使用此设置，数据为strong，其余属性为weak，是否存在任何潜在风险？
2. 我应该将数据设为ivar并保持其余部分吗？

对于物业的存在，没有实际的理由（除了我糟糕的阶级设计技巧）。我觉得它很有趣，想要了解发生了什么。

Answer 1

您询问：

1. 使用此设置，数据为strong，其余属性为weak，是否存在任何潜在风险？

是的，如果你nil dictionary，你的所有属性都可能成为nil，假设你没有在其他地方有其他强烈的引用。

2. 我应该将数据设为ivar并保持其余部分吗？

我甚至不会把它变成一个伊娃（除非有其他要求保存这个你还没有与我们分享）。它应该只是一个局部变量，并使您的属性copy（或strong）。

---

我建议（a）摆脱NSDictionary属性和（b）使NSString属性为copy（或strong），而不是weak。此外，我只是定义一个初始化器，而不是使用setPropertiesWith方法：

// User.h

@interface User : NSObject

@property (copy, nonatomic) NSString *uid;
@property (copy, nonatomic) NSString *firstName;
@property (copy, nonatomic) NSString *lastName;
@property (copy, nonatomic) NSString *dob;
@property (copy, nonatomic) NSString *gender;
@property (copy, nonatomic) NSString *avatarURL;
@property (assign, nonatomic) NSInteger status;

- (instancetype)initWithDictionary:(NSDictionary *)dictionary;

@end

和

// User.m

@implementation User

- (instancetype)initWithDictionary:(NSDictionary *)dictionary {
    if ((self = [super init])) {
        self.uid       = dictionary[@"uid"];
        self.firstName = dictionary[@"firstName"];
        self.lastName  = dictionary[@"lastName"];
        self.dob       = dictionary[@"dob"];
        self.gender    = dictionary[@"gender"];
        self.status    = [dictionary[@"status"] intValue];
        self.avatarURL = dictionary[@"avatarURL"];
    }

    return self;
}

@end

然后，调用者会这样做：

User *user = [[User alloc] initWithDictionary:someDictionary];

您可以在这里考虑其他改进（例如readonly公共接口，声明可空性，字典上的轻量级泛型等），但上述可能是一个很好的起点。

---

顺便说一句，如果你想知道为什么我制作这些copy而不是strong，我们只是想保护自己，以防呼叫者通过NSMutableString（这是一个NSString子类）并在以后偶然发生变异。这只是一个更安全，更防守的模式。