3

如果我有这个方法：

+ (NSDictionary *)dictionaryFromQueryString:(NSString *)queryString
{
    NSMutableDictionary *this = [[NSMutableDictionary alloc] init];

    NSArray *fields = [queryString componentsSeparatedByString:@"&"];

    for(NSString *field in fields)
    {
        NSArray *fieldParts = [field componentsSeparatedByString:@"="];

        NSString *value = @"";

        if(fieldParts.count > 1)
        {
            value = [[fieldParts subarrayWithRange:NSMakeRange(1, fieldParts.count - 1)] componentsJoinedByString:@"="];
        }

        [this setObject:unescape(value) forKey:unescape(fieldParts[0])];
    }

    return this;
}

那么我返回 aNSMutableDictionary而不是 a是不好的做法NSDictionary吗？

我应该将其转换为NSDictionarywithreturn [this copy];吗？

4

3 回答 3

5

这取决于。

塞尔吉奥的回答是正确的，除了一个非常重要的问题：

当包含可变字典的对象在另一个对象检索字典后对其进行变异时会发生什么？除非另一个对象是专门为支持字典可能发生变异的可能性而编写的，否则另一个对象现在将处于不一致的状态。

鉴于copy字典是一个浅的不可变副本，因此它的速度很快，因此通常总是返回一个副本比返回对可变版本的引用要好得多。如果您发现您的代码正在使用创建副本的方法，则在您的对象中缓存一个不可变副本并出售它，只要可变后备存储发生更改，它就会使其无效。

于 2012-09-12T17:32:19.387 回答
2

我不认为这是不好的做法。这样做的最终效果是您的接收者NSDictionary不会尝试修改对象（尽管对象是可变的）。这是非常安全的，而且很有意义，因为您的使用者方法保持更通用（它可以与可变和非可变对象一起使用）。

于 2012-09-12T17:26:48.993 回答
0

以这种方式返回一个不可变对象并不是一个坏习惯，因为NSMutableDictionary它是NSDictionary. 这是多态性，所以“一切都好”。

但无论如何，我可能会返回一个这样的自动发布副本：

return [NSDictionary dictionaryWithDictionary:this];
于 2012-09-12T17:32:55.207 回答