类未按预期设置

Question

图标设置为 @property (nonatomic, keep) AHGridIcon *icon;

通常我只是这样做：

-(void)setIcon:(AHGridIcon *)iconLocal {
icon = iconLocal;
} 

但我在线阅读了 getters setters 和属性指南，这让我相信，这是正确的：

-(void)setIcon:(AHGridIcon *)iconLocal {
    if (iconLocal != self.icon)
    {
        NSLog(@"local: %@", iconLocal);
        NSLog(@"self.icon 1: %@", self.icon);
        [iconLocal retain];
        [icon release];
        icon = iconLocal;
        NSLog(@"self.icon 2: %@", self.icon);
    }
}

问题是，原始图标保持不变，没有被新图标替换。我做错了什么？我应该恢复到通常的方式吗？

Answer 1

您应该使用“@synthesize”，除非您确实需要自定义设置器行为。

Answer 2

就像我在评论中发布的那样：

最好的方法是使用 @synthesize ，它将根据您在属性中编写的属性创建一个 getter 和 setter （nonatomic，retain）< /代码> =>不是线程安全的，但快速 getter 和 setter 以及保留（也释放）setter。如果您不需要在设置器中执行复杂的操作，那么您不应该覆盖设置器。

.h：

@property (nonatomic, retain) AHGridIcon *icon;

.m：

@implementation Something
@synthesize icon;
...
@end

您在设置器中发布的代码几乎与编译器仅使用合成时生成的代码相同。

您通常的方式并不是很好，因为在您的标头中定义了（在您的属性中）设置器正在保留，但在您的实现中您正在覆盖不保留的正确设置器。它几乎与编译器使用（非原子、分配）属性生成的结果相同。

但是，如果您想覆盖您的设置器，那么它应该看起来与您编写的相同。对我来说它工作得很好。

1. 首先保留新对象，
2. 然后释放旧对象
3. ，然后将本地指针分配给新对象，

您甚至可以省略 if 但首先保留新对象，然后释放旧对象非常重要（就像您所做的那样 - 只是想要提一下）。

为了解决您使用覆盖设置器的问题：您的设置器在我眼中看起来不错。你也重写了 getter 吗？如果是，则将其发布在这里（您可以通过在日志调用中调用 self.icon 来使用它）。

我做了一个小测试程序

@synthesize str;

- (void)setStr:(NSString *)localStr
{
    if(str != localStr)
    {
        NSLog(@"old : %@", self.str);
        NSLog(@"new1: %@", localStr);
        [localStr retain];
        [str release];
        str = localStr;

        NSLog(@"new2: %@", self.str);
    }
}

，输出很好：

old : (null)
new1: Hello
new2: Hello

old : Hello
new1: World
new2: World