14

我正在做一些涉及解析 NSXmlDocument 并从结果中填充对象属性的 Objective-C 编程。

第一个版本看起来像这样:

if([elementName compare:@"companyName"] == 0) 
  [character setCorporationName:currentElementText]; 
else if([elementName compare:@"corporationID"] == 0) 
  [character setCorporationID:currentElementText]; 
else if([elementName compare:@"name"] == 0) 
  ...

但我不喜欢if-else-if-else这种产生的模式。查看switch语句,我发现我只能处理ints, charsetc 而不能处理对象......所以有没有更好的实现模式我不知道?

顺便说一句,我确实想出了一个更好的解决方案来设置对象的属性,但我想具体了解Objective-C 中ifelse-vs模式switch

4

14 回答 14

13

您应该利用键值编码:

[character setValue:currentElementText forKey:elementName];

如果数据不受信任,您可能需要检查密钥是否有效:

if (![validKeysCollection containsObject:elementName])
    // Exception or error
于 2008-09-21T04:37:51.907 回答
12

我希望你们都原谅我在这里跑题了,但是我想解决在 Cocoa 中解析 XML 文档而不需要 if-else 语句的更一般的问题。最初陈述的问题将当前元素文本分配给字符对象的实例变量。正如 jmah 指出的,这可以使用键值编码来解决。但是,在更复杂的 XML 文档中,这可能是不可能的。例如,考虑以下内容。

<xmlroot>
    <corporationID>
        <stockSymbol>EXAM</stockSymbol>
        <uuid>31337</uuid>
    </corporationID>
    <companyName>Example Inc.</companyName>
</xmlroot>

有多种方法可以处理这个问题。在我的脑海中,我可以想到两个使用 NSXMLDocument。第一个使用 NSXMLElement。它相当简单,根本不涉及 if-else 问题。您只需获取根元素并一一遍历其命名元素。

NSXMLElement* root = [xmlDocument rootElement];

// Assuming that we only have one of each element.
[character setCorperationName:[[[root elementsForName:@"companyName"] objectAtIndex:0] stringValue]];

NSXMLElement* corperationId = [root elementsForName:@"corporationID"];
[character setCorperationStockSymbol:[[[corperationId elementsForName:@"stockSymbol"] objectAtIndex:0] stringValue]];
[character setCorperationUUID:[[[corperationId elementsForName:@"uuid"] objectAtIndex:0] stringValue]];

下一个使用更通用的NSXMLNode,遍历树,直接使用if-else结构。

// The first line is the same as the last example, because NSXMLElement inherits from NSXMLNode
NSXMLNode* aNode = [xmlDocument rootElement];
while(aNode = [aNode nextNode]){
    if([[aNode name] isEqualToString:@"companyName"]){
        [character setCorperationName:[aNode stringValue]];
    }else if([[aNode name] isEqualToString:@"corporationID"]){
        NSXMLNode* correctParent = aNode;
        while((aNode = [aNode nextNode]) == nil && [aNode parent != correctParent){
            if([[aNode name] isEqualToString:@"stockSymbol"]){
                [character setCorperationStockSymbol:[aNode stringValue]];
            }else if([[aNode name] isEqualToString:@"uuid"]){
                [character setCorperationUUID:[aNode stringValue]];
            }
        }
    }
}

这是消除 if-else 结构的一个很好的候选,但是像原来的问题一样,我们不能在这里简单地使用 switch-case。但是,我们仍然可以通过使用 performSelector 来消除 if-else。第一步是为每个元素定义一个方法。

- (NSNode*)parse_companyName:(NSNode*)aNode
{
    [character setCorperationName:[aNode stringValue]];
    return aNode;
}

- (NSNode*)parse_corporationID:(NSNode*)aNode
{
    NSXMLNode* correctParent = aNode;
    while((aNode = [aNode nextNode]) == nil && [aNode parent != correctParent){
        [self invokeMethodForNode:aNode prefix:@"parse_corporationID_"];
    }
    return [aNode previousNode];
}

- (NSNode*)parse_corporationID_stockSymbol:(NSNode*)aNode
{
    [character setCorperationStockSymbol:[aNode stringValue]];
    return aNode;
}

- (NSNode*)parse_corporationID_uuid:(NSNode*)aNode
{
    [character setCorperationUUID:[aNode stringValue]];
    return aNode;
}

魔法发生在 invokeMethodForNode:prefix: 方法中。我们根据元素的名称生成选择器,并使用 aNode 作为唯一参数来执行该选择器。Presto Bango,我们已经消除了对 if-else 语句的需要。这是该方法的代码。

- (NSNode*)invokeMethodForNode:(NSNode*)aNode prefix:(NSString*)aPrefix
{
    NSNode* ret = nil;
    NSString* methodName = [NSString stringWithFormat:@"%@%@:", prefix, [aNode name]];
    SEL selector = NSSelectorFromString(methodName);
    if([self respondsToSelector:selector])
        ret = [self performSelector:selector withObject:aNode];
    return ret;
}

现在,我们可以简单地编写一行代码,而不是我们更大的 if-else 语句(区分 companyName 和corporateID 的语句)

NSXMLNode* aNode = [xmlDocument rootElement];
while(aNode = [aNode nextNode]){
    aNode = [self invokeMethodForNode:aNode prefix:@"parse_"];
}

现在,如果我有任何错误,我深表歉意,自从我用 NSXMLDocument 编写任何东西以来已经有一段时间了,现在已经很晚了,我实际上并没有测试这段代码。因此,如果您发现任何错误,请发表评论或编辑此答案。

但是,我相信我刚刚展示了如何在 Cocoa 中使用正确命名的选择器来完全消除这种情况下的 if-else 语句。有一些陷阱和极端情况。performSelector: 系列方法只接受 0、1 或 2 个参数方法,它们的参数和返回类型都是对象,所以如果参数和返回类型的类型不是对象,或者有两个以上的参数,那么你会必须使用 NSInvocation 来调用它。您必须确保您生成的方法名称不会调用其他方法,特别是如果调用的目标是另一个对象,并且此特定方法命名方案不适用于具有非字母数字字符的元素。您可以通过以某种方式转义方法名称中的 XML 元素名称来解决此问题,或者通过使用方法名称作为键和选择器作为值来构建一个 NSDictionary。这可能会占用大量内存并最终花费更长的时间。像我描述的 performSelector 调度非常快。对于非常大的 if-else 语句,此方法甚至可能比 if-else 语句更快。

于 2008-09-26T09:26:59.737 回答
7

我敢建议使用宏吗?

#define TEST( _name, _method ) \
  if ([elementName isEqualToString:@ _name] ) \
    [character _method:currentElementText]; else
#define ENDTEST { /* empty */ }

TEST( "companyName",      setCorporationName )
TEST( "setCorporationID", setCorporationID   )
TEST( "name",             setName            )
:
:
ENDTEST
于 2008-09-26T19:49:43.037 回答
7

如果您想使用尽可能少的代码,并且您的元素名称和 setter 都已命名,因此如果 elementName 为 @"foo" 则 setter 为 setFoo:,您可以执行以下操作:

SEL selector = NSSelectorFromString([NSString stringWithFormat:@"set%@:", [elementName capitalizedString]]);

[character performSelector:selector withObject:currentElementText];

甚至可能:

[character setValue:currentElementText forKey:elementName]; // KVC-style

虽然这些当然比使用一堆 if 语句要慢一些。

[编辑:第二个选项已经有人提到过;哎呀!]

于 2008-09-26T21:35:18.383 回答
4

我使用 NSStrings 完成此操作的一种方法是使用 NSDictionary 和枚举。它可能不是最优雅的,但我认为它使代码更具可读性。以下伪代码是从我的一个项目中提取的:

typedef enum { UNKNOWNRESIDUE, DEOXYADENINE, DEOXYCYTOSINE, DEOXYGUANINE, DEOXYTHYMINE } SLSResidueType;

static NSDictionary *pdbResidueLookupTable;
...

if (pdbResidueLookupTable == nil)
{
    pdbResidueLookupTable = [[NSDictionary alloc] initWithObjectsAndKeys:
                          [NSNumber numberWithInteger:DEOXYADENINE], @"DA", 
                          [NSNumber numberWithInteger:DEOXYCYTOSINE], @"DC",
                          [NSNumber numberWithInteger:DEOXYGUANINE], @"DG",
                          [NSNumber numberWithInteger:DEOXYTHYMINE], @"DT",
                          nil]; 
}

SLSResidueType residueIdentifier = [[pdbResidueLookupTable objectForKey:residueType] intValue];
switch (residueIdentifier)
{
    case DEOXYADENINE: do something; break;
    case DEOXYCYTOSINE: do something; break;
    case DEOXYGUANINE: do something; break;
    case DEOXYTHYMINE: do something; break;
}
于 2008-09-23T00:14:44.380 回答
3

您拥有的if-else实现是执行此操作的正确方法,因为它switch不适用于对象。除了可能有点难以阅读(这是主观的)之外,以if-else这种方式使用语句并没有真正的缺点。

于 2008-09-19T19:07:00.043 回答
3

虽然不一定有更好的方法来做这样的事情一次性使用,但是当你可以使用“isEqualToString”时,为什么还要使用“compare”呢?这似乎更有效率,因为比较会在第一个不匹配的字符处停止,而不是通过整个事情来计算有效的比较结果(尽管想想看,比较可能在同一点很清楚) - 虽然它看起来更干净一些,因为该调用返回一个 BOOL。

if([elementName isEqualToString:@"companyName"] ) 
  [character setCorporationName:currentElementText]; 
else if([elementName isEqualToString:@"corporationID"] ) 
  [character setCorporationID:currentElementText]; 
else if([elementName isEqualToString:@"name"] ) 
于 2008-09-21T04:30:17.653 回答
3

实际上有一种相当简单的方法来处理像 Objective-C 这样的语言中的级联 if-else 语句。是的,您可以使用子类化和覆盖,创建一组以不同方式实现相同方法的子类,在运行时使用公共消息调用正确的实现。如果您希望选择几个实现之一,这很有效,但如果您有许多小的、稍微不同的实现,就像您在长 if-else 或 switch 语句中往往有的那样,它可能会导致子类不必要的扩散。

相反,将每个 if/else-if 子句的主体分解为它自己的方法,都在同一个类中。以类似的方式命名调用它们的消息。现在创建一个包含这些消息的选择器的 NSArray(使用 @selector() 获得)。使用 NSSelectorFromString() 将您在条件句中测试的字符串强制转换为选择器(您可能需要首先将其他单词或冒号连接到它,具体取决于您如何命名这些消息,以及它们是否带有参数)。现在使用 performSelector: 自行执行选择器。

这种方法的缺点是它可以用许多新消息将类弄乱,但是与使用新子类的整个类层次结构相比,将单个类弄乱可能更好。

于 2008-09-27T08:00:28.147 回答
3

将其发布为对 Wevah 上述回答的回应——我会编辑,但我还没有足够高的声誉:

不幸的是,第一种方法会破坏其中包含多个单词的字段——比如 xPosition。capitalizedString 会将其转换为 Xposition,当与格式结合使用时,将为您提供 setXposition: 。绝对不是这里想要的。这是我在代码中使用的内容:

NSString *capName = [elementName stringByReplacingCharactersInRange:NSMakeRange(0, 1) withString:[[elementName substringToIndex:1] uppercaseString]];
SEL selector = NSSelectorFromString([NSString stringWithFormat:@"set%@:", capName]);

不如第一种方法漂亮,但它有效。

于 2009-08-01T03:47:27.103 回答
3

我想出了一个解决方案,它使用块为对象创建类似开关的结构。它是这样的:

BOOL switch_object(id aObject, ...)
{
    va_list args;
    va_start(args, aObject);

    id value = nil;
    BOOL matchFound = NO;

    while ( (value = va_arg(args,id)) )
    {
        void (^block)(void) = va_arg(args,id);
        if ( [aObject isEqual:value] )
        {
            block();
            matchFound = YES;
            break;
        }
    }

    va_end(args);
    return matchFound;
}

如您所见,这是一个带有可变参数列表的老式 C 函数。我在第一个参数中传递要测试的对象,然后是 case_value-case_block 对。(回想一下,Objective-C 块只是对象。)while循环不断提取这些对,直到对象值匹配或没有剩余案例(见下面的注释)。

用法:

NSString* str = @"stuff";
switch_object(str,
              @"blah", ^{
                  NSLog(@"blah");
              },
              @"foobar", ^{
                  NSLog(@"foobar");
              },
              @"stuff", ^{
                  NSLog(@"stuff");
              },
              @"poing", ^{
                  NSLog(@"poing");
              },
              nil);   // <-- sentinel

// will print "stuff"

笔记:

  • 这是没有任何错误检查的第一个近似值
  • 案例处理程序是块的事实,在涉及从内部引用的变量的可见性、范围和内存管理时需要额外注意
  • 如果你忘记了哨兵,你注定要失败:P
  • 当没有匹配的情况下,您可以使用布尔返回值触发“默认”情况
于 2012-04-12T13:16:27.633 回答
2

建议用于消除 if-else 或 switch 语句的最常见重构是引入多态性(参见http://www.refactoring.com/catalog/replaceConditionalWithPolymorphism.html)。当它们被重复时,消除这些条件是最重要的。在像您的示例这样的 XML 解析的情况下,您实际上是将数据移动到更自然的结构中,这样您就不必在其他地方复制条件。在这种情况下,if-else 或 switch 语句可能就足够了。

于 2008-09-19T19:03:00.753 回答
1

在这种情况下,我不确定您是否可以像 Bradley 建议的那样轻松重构该类以引入多态性,因为它是一个 Cocoa-native 类。相反,Objective-C 的做法是使用类类别elementNameCode向 NSSting 添加方法:

   typedef enum { 
       companyName = 0,
       companyID,  
       ...,
       Unknown
    } ElementCode;

    @interface NSString (ElementNameCodeAdditions)
    - (ElementCode)elementNameCode; 
    @end

    @implementation NSString (ElementNameCodeAdditions)
    - (ElementCode)elementNameCode {
        if([self compare:@"companyName"]==0) {
            return companyName;
        } else if([self compare:@"companyID"]==0) {
            return companyID;
        } ... {

        }

        return Unknown;
    }
    @end

在您的代码中,您现在可以使用一个开关[elementName elementNameCode](如果您忘记测试枚举成员之一等,则会获得相关的编译器警告)。

正如布拉德利指出的那样,如果逻辑只用在一个地方,这可能不值得。

于 2008-09-20T23:00:21.143 回答
1

在我们的项目中,我们需要一遍又一遍地做这种事情,就是设置一个静态 CFDictionary,将要检查的字符串/对象映射到一个简单的整数值。它导致代码如下所示:

static CFDictionaryRef  map = NULL;
int count = 3;
const void *keys[count] = { @"key1", @"key2", @"key3" };
const void *values[count] = { (uintptr_t)1, (uintptr_t)2, (uintptr_t)3 };

if (map == NULL)
    map = CFDictionaryCreate(NULL,keys,values,count,&kCFTypeDictionaryKeyCallBacks,NULL);


switch((uintptr_t)CFDictionaryGetValue(map,[node name]))
{
    case 1:
        // do something
        break;
    case 2:
        // do something else
        break;
    case 3:
        // this other thing too
        break;
}

如果您只针对 Leopard,您可以使用 NSMapTable 而不是 CFDictionary。

于 2008-11-04T17:48:20.240 回答
0

与 Lvsti 类似,我使用块在对象上执行切换模式。

我编写了一个非常简单的基于过滤器块的链,它采用 n 个过滤器块并对对象执行每个过滤器。
每个过滤器都可以更改对象,但必须返回它。无论。

NSObject+Functional.h

#import <Foundation/Foundation.h>
typedef id(^FilterBlock)(id element, NSUInteger idx, BOOL *stop);

@interface NSObject (Functional)
-(id)processByPerformingFilterBlocks:(NSArray *)filterBlocks;
@end

NSObject+Functional.m

@implementation NSObject (Functional)
-(id)processByPerformingFilterBlocks:(NSArray *)filterBlocks
{
    __block id blockSelf = self;
    [filterBlocks enumerateObjectsUsingBlock:^( id (^block)(id,NSUInteger idx, BOOL*) , NSUInteger idx, BOOL *stop) {
        blockSelf = block(blockSelf, idx, stop);
    }];

    return blockSelf;
}
@end

现在我们可以设置nFilterBlocks 来测试不同的情况。

FilterBlock caseYES = ^id(id element, NSUInteger idx, BOOL *breakAfter){ 
    if ([element isEqualToString:@"YES"]) { 
        NSLog(@"You did it");  
        *breakAfter = YES;
    } 
    return element;
};

FilterBlock caseNO  = ^id(id element, NSUInteger idx, BOOL *breakAfter){ 
    if ([element isEqualToString:@"NO"] ) { 
        NSLog(@"Nope");
        *breakAfter = YES;
    }
    return element;
};

现在我们将要测试的那些块作为过滤器链粘贴到数组中:

NSArray *filters = @[caseYES, caseNO];

并且可以在一个对象上执行它

id obj1 = @"YES";
id obj2 = @"NO";
[obj1 processByPerformingFilterBlocks:filters];
[obj2 processByPerformingFilterBlocks:filters];

这种方法可用于切换,也可用于任何(条件)过滤器链应用程序,因为块可以编辑元素并将其传递。

于 2013-02-09T15:16:35.807 回答