何时覆盖目标 c 吸气剂

Question

在过去的一年中，我第一次与其他人一起参与了一些 Objective-C 项目。

偶尔（并且越来越多）我看到其他人重写 getter/accessor 方法，并在此方法中包含实现代码！对我来说，这是一个疯狂的小镇，因为这是拥有一个 setter 的全部意义......这也意味着在 setter 中设置的属性将在 getter 中被覆盖，因此毫无意义。

是这些人行为不端，还是我错过了什么？是否需要覆盖合成属性的 getter 方法？

例子：

@synthesize width;

- (CGFloat)width {
  NSNumber *userWidth = [[NSUserDefaults standardUserDefaults] objectForKey:@"USER_WIDTH"];

  if (userWidth != nil) 
  {
    width = [NSNumber floatValue];
  }     

  //Shouldn't the above lines be done in the SETTER? I have SET the property!
  //Why would I ever need to write anything BUT the below line??       
  return width;
}

- (void)setWidth:(CGFloat)newWidth {
  //THIS is where you do the the setting!
  width = newWidth;
}

更新：

好的宽度是一个不好的例子。太多人陷入了“变量是什么”和“不包括 get 在objective-c 访问器中”的语义上。所以我更新了上面的例子，忽略了不相关的语义，专注于概念。这个概念是......当你想要覆盖 GETTER （不是setter，只有getter。我多次覆盖setter，这个问题是关于getter的）时，是否有任何例子？ p>

返回另一个属性，例如层（如下所述）是一个真实的例子。但更具体地说，是否需要在 GETTER 中 SET 属性？这是我看到的一些奇怪之处，所以我更新了上面的 getter 以从 NSUserDefaults 中提取一个值来帮助我的观点......

Answer 1

首先，Cocoa 命名约定将 getter 称为 -width，而不是 -getWidth。 “Get”用于填写传入的参数：

- (void) getWidth:(CGFloat *)outWidth
{
    if (outWidth) *outWidth = _width;
}

也就是说，回到你原来的问题：

在过去，在 @property 和 @synthesize 之前，我们必须像上面那样手动编写访问器。

但是，在其他情况下，您可能希望手动编写访问器。

一个常见的方法是延迟初始化，直到需要一个值。例如，假设有一个图像需要一段时间才能生成。每次修改会改变图像的属性时，您都不想立即重绘图像。相反，您可以将抽签推迟到下一次有人询问时：

- (UIImage *) imageThatTakesAwhileToGenerate
{
    if (!_imageThatTakesAwhileToGenerate) {
        // set it here
    } 

    return _imageThatTakesAwhileToGenerate;
} 


- (void) setColorOfImage:(UIColor *)color
{
    if (_color != color) {
        [_color release];
        _color = [color retain];

        // Invalidate _imageThatTakesAwhileToGenerate, we will recreate it the next time that the accessor is called
        [_imageThatTakesAwhileToGenerate release];
        _imageThatTakesAwhileToGenerate = nil;
    }
}

另一个用途是将访问器/修改器的实现转发到另一个类。例如，UIView 将其许多属性转发给支持CALayer：

// Not actual UIKit implementation, but similar:
- (CGRect) bounds { return [[self layer] bounds]; }
- (void) setBounds:(CGRect)bounds { [[self layer] setBounds:bounds]; }
- (void) setHidden:(BOOL)hidden { [[self layer] setHidden:hidden]; }
- (BOOL) isHidden { return [[self layer] isHidden]; }
- (void) setClipsToBounds:(BOOL)clipsToBounds { [[self layer] setMasksToBounds:clipsToBounds]; }
- (BOOL) clipsToBounds { return [[self layer] masksToBounds]; }

---

更新到提问者的更新：

在您的更新中，看起来有问题的代码要么尝试使用 NSUserDefaults 保留宽度值，要么尝试允许用户指定自定义值以覆盖所有返回的宽度。如果是后者，你的例子很好（尽管我会限制这种做法，因为它可能会给新手造成混淆）。

如果是前者，您希望从 NSUserDefaults 加载一次值，然后在 setter 中将新值保存回 NSUserDefaults。例如：

static NSString * const sUserWidthKey = @"USER_WIDTH";

@implementation Foo {
    CGFloat _width;
    BOOL _isWidthIvarTheSameAsTruthValue;
}

@synthesize width = _width;

- (CGFloat) width
{
    if (!_isWidthIvarTheSameAsTruthValue) {
        NSNumber *userWidth = [[NSUserDefaults standardUserDefaults] objectForKey:sUserWidthKey];
        if (userWidth != nil) _width = [NSNumber doubleValue];
        _isWidthIvarTheSameAsTruthValue = YES;
    }

    return _width;
}

- (void) setWidth:(CGFloat)newWidth
{
    if (_width != newWidth) {
        _width = newWidth;
        NSNumber *userWidthNumber = [NSNumber numberWithDouble:_width];
        [[NSUserDefaults standardUserDefaults] setObject:userWidthNumber forKey:sUserWidthKey];
        _isWidthIvarTheSameAsTruthValue = YES;
    }
}

@end

_width ivar 被用作缓存。真相存储在 NSUserDefaults 中。

注意：我在这个例子中使用了 NSUserDefaults，因为你在你的例子中使用了它。在实践中，我更喜欢不将 NSUserDefault 与我的访问器混合使用 ;)

Answer 2

第一个问题是您不想使用 getWidth。 objC 中的模式是 name 和 setName。不要使用 getName。它搞乱了绑定和 KVO。

此外，如果只是设置/获取 iVar，则没有理由重写。如果您要进行额外的处理/验证，则可以覆盖。

编辑：

您还应该尽量避免在 getter 中设置数据和进行繁重的处理。 getter 应该封装一些状态并返回数据。期望它的重量很轻。繁重的处理和/或修改应在方法或设置器中完成。例如，人们在 getter 上设置调试监视，并且不期望对状态进行繁重的处理和修改。

Answer 3

如果你懒惰地创建你的属性对象呢？ 我经常使用这个模式，也用在 Xcode 的 CoreData 模板等中。

- (NSString *)string
{
    if (!_string) {
        // Create the string property lazily
        // Create is using some other internal, etc values
        _string = [NSString alloc] initWith...]
    }
    return _string;
}

还有：

- (void)setString:(NSString *)string
{
    if (![string isEqualToString:_string]) {
        // Probably you want to make your property observable here too :)

        [_setString release];
        _setString = [string retain];

        // Update other things that depend on _string for example redraw the view, etc
        [self setNeedsDisplay];
    }
}

Answer 4

重写 getter 和 setter 方法的原因有很多，例如我在创建自定义 UITableViewCell 对象时重写了 setter 方法，这样我就可以设置一个属性，然后在该 setter 方法中，它会自动更新一个标签或单元格内的东西，而不是我必须在设置属性后调用更新函数。

如果您想以不同于接收信息的方式存储信息，您可能需要覆盖 getter，例如电话号码对象，您可以将其存储为 5551234567，它会自动检索为 555-123-4567 或类似的东西。我自己很少重写 getter 方法，但我经常重写 setter 方法。

Answer 5

这是面向对象编程中完全可以接受的做法。但是，需要注意副作用。例如，您不应该在 setter 方法中执行网络访问之类的操作。

但是，在您上面列出的代码中，由于它们与合成方法所做的没有任何不同，因此没有理由包含实现。他们只是把代码弄得乱七八糟。