8

我有一个解决通知问题的方法，效果很好，但我担心这可能是个坏主意。

我有一个需要由类的每个实例和类本身处理的通知。为了处理这个问题，我正在注册类和类实例的通知。因为它是完全相同的通知，所以我将类和实例方法命名为相同。这遵循我们为如何命名通知处理程序设置的标准。

这是一个坏主意吗？是否有一些我错过的隐藏的got'ca。我会混淆未来的开发人员吗？

+ (void)initialize
{
    if (self == [SICOHTTPClient class]) {
        [[self notificationCenter] addObserver:self
                                      selector:@selector(authorizationDidChangeNotification:)
                                          name:SICOJSONRequestOperationAuthorizationDidChangeNotification
                                        object:nil];
    }
}

- (id)initWithBaseURL:(NSURL *)url
{
    self = [super initWithBaseURL:url];

    if (self) {
        self.parameterEncoding = AFJSONParameterEncoding;
        [self registerHTTPOperationClass:[SICOJSONRequestOperation class]];
        [self setDefaultHeader:@"Accept" value:@"application/json"];

        if ([[self class] defaultAuthorization])
            [self setDefaultHeader:@"Authorization" value:[[self class] defaultAuthorization]];

        [[[self class] notificationCenter] addObserver:self
                                              selector:@selector(authorizationDidChangeNotification:)
                                                  name:SICOJSONRequestOperationAuthorizationDidChangeNotification
                                                object:nil];
    }

    return self;
}

- (void)dealloc
{
    [[[self class] notificationCenter] removeObserver:self
                                                 name:SICOJSONRequestOperationAuthorizationDidChangeNotification
                                               object:nil];
}

#pragma mark Notifications

- (void)authorizationDidChangeNotification:(NSNotification *)notification
{
    NSString *authorization = notification.userInfo[SICOJSONRequestOperationAuthorizationKey];

    if ([authorization isKindOfClass:[NSString class]]) {
        [self setDefaultHeader:@"Authorization" value:authorization];
    } else {
        [self clearAuthorizationHeader];
    }
}

+ (void)authorizationDidChangeNotification:(NSNotification *)notification
{
    NSString *authorization = notification.userInfo[SICOJSONRequestOperationAuthorizationKey];

    if ([authorization isKindOfClass:[NSString class]]) {
        [self setDefaultAuthorization:authorization];
    } else {
        [self setDefaultAuthorization:nil];
    }
}
4

2 回答 2

4

这就是代码注释的用途:)

在 Objective C 中，类方法和实例方法同名是没有问题的。

我会建议：

  • 修改您的通知方法名称规范以处理此问题（然后使用不同的适当命名的方法处理类通知），或

  • 添加适当的评论来解释正在发生的事情，以使未来可能感到困惑的开发人员受益

于 2012-12-06T16:28:01.667 回答
1

语言本身和运行时不会对您正在做的事情产生歧义。所以你在这方面是安全的。

就混淆未来的维护者而言，我想您不必太担心愚蠢的自动完成错误，因为它不是您打算手动调用的方法。

也就是说，我很想将类的东西移到一个人为的类别中。这不仅会在页面上进行分离，而且会明确表明该类打算作为与实例响应分开的功能部分进行响应。

于 2012-12-06T16:31:00.520 回答