OOP，目标 c 中的聚合

Question

我有一些复杂的对象，我不想实现它并且稍后会出现内存泄漏，所以必须询问:) 这是进行聚合的好方法吗？我是否需要以及在哪里从用户中清除角色对象？

#import "Role.h"

@interface User : NSObject {
    NSString *firstName;
    NSString *lastName;

    Role *role;
}
@property (nonatomic, retain) NSString *firstName;
@property (nonatomic, retain) NSString *lastName;

@property (nonatomic, retain) Role *role;
@end
#import "User.h"

@implementation User
@synthesize firstName;
@synthesize lastName;

@synthesize role;
@end

@interface Role : NSObject {
    NSInteger *roleId;
    NSString *title;
    NSString *description;
}
@property (nonatomic, retain) NSInteger *roleId;
@property (nonatomic, retain) NSString *title;
@property (nonatomic, retain) NSString *description;
@end
#import "Role.h"

@implementation Role
@synthesize roleId;
@synthesize title;
@synthesize description;
@end

Answer 1

您的 dealloc 方法需要如下所示：

对于用户：

-dealloc
{
    [firstName release];
    [lastName release];
    [role release];
    [super dealloc];
}

对于角色：

-dealloc
{
    [title release];
    [description release];
    [super dealloc];
}

roleId 不需要释放，因为 NSInteger 实际上是原始整数类型的 typedef。这也意味着你的属性声明是错误的，它应该是：

@property (nonatomic, assign) NSInteger roleId;

另外，NSString 是不可变的，它实现了复制协议，所以你的 NSString 属性应该是 copy 而不是 retain。例如

@property (nonatomic, copy) NSString *firstName;

编辑

正如 Björn 指出的，Role 的界面应该是：

@interface Role : NSObject {
    NSInteger roleId;  // not a pointer
    NSString *title;
    NSString *description;
}
@property (nonatomic, assign) NSInteger roleId;
@property (nonatomic, copy) NSString *title;
@property (nonatomic, copy) NSString *description;
@end