NSColor 和 NSUserDefaults 原因；程序收到信号：“EXC_BAD_ACCESS”；

Question

我有 3 种方法，一种获取颜色并将其保存为用户默认值中的数据对象，一种方法绘制一个矩形并用用户默认值中保存的颜色填充它，另一种方法使用用户默认值中保存的颜色初始化 NSColor 实例应用程序启动时默认。

下面是 3 种方法，但问题是我在构建和运行应用程序时收到此错误，任何人都可以理解为什么我收到此错误以及我的代码有什么问题。

程序收到信号：“EXC_BAD_ACCESS”。

- (void)setColor:(NSColor *)color
{
    _color = [color copy];

    NSData *data = [NSArchiver archivedDataWithRootObject:_color];
    [[NSUserDefaults standardUserDefaults] setObject:data forKey:@"MyColor"];

    [self setNeedsDisplay:YES];
}


    - (void)drawRect:(NSRect)rect 
{

     NSRect rect1 = NSInsetRect([self bounds], 4.0, 4.0);
     NSBezierPath * path;
     [_color set];
     path = [NSBezierPath bezierPathWithRoundedRect:rect1
                                                   xRadius:6.0
                                                   yRadius:6.0];
     [path fill];
}

- (id)initWithFrame:(NSRect)frame
{
    self = [super initWithFrame:frame];
    if (self == nil)
        return nil;

    _color = [NSColor blackColor];
    NSData *data = [[NSUserDefaults standardUserDefaults] objectForKey:@"MyColor"];
    if (data != nil){
    NSColor *color1 = [NSUnarchiver unarchiveObjectWithData:data];
    _color = color1;}

    return self;
}

Answer 1

问题出在您的 initWithFrame: 方法中。 _color 使用 [NSColor blackColor] 或 [NSUnarchiver unarchiveObjectWithData:data] 设置。这两个方法都返回自动释放的对象。您需要保留它们以确保在调用 drawRect: 时它们仍然存在。最简单的方法是在取消存档后调用 [_color keep] ，这样如果 blackColor 被替换，它仍然会自动释放。

另外，请确保在 setColor: 中释放旧的_color。现在，每次调用它时都会泄漏一个 NSColor 对象。

Answer 2

我认为这是您的问题：

NSColor *color1 = [NSUnarchiver unarchiveObjectWithData:data];
_color = color1;

您不拥有 unarchiveObjectWithData: 返回的对象，但您将其分配给实例变量并将其视为您拥有的对象。您可以使用 copy 方法显式获取对象的所有权，就像在 setter 中一样：

_color = [color1 copy];