Obj-C：if 语句和 NSString 比较函数以提高效率和可读性

Question

什么是高效编码的最佳方法，因为这将被递归调用，并且创建大量对象是不好的。 +*-/, sin,cos,sqrt,pi and x,y,a,b 应该放在三个单独的 if statements 中，不像现在所有东西都是分开的。同样为了可读性，制作包含 4 isEqualToString 的 if 语句也不是很好。

    NSString *operation = topOfStack;
    if ([operation isEqualToString:@"+"]) {

    } else if ([@"*" isEqualToString:operation]) {

    } else if ([operation isEqualToString:@"-"]) {

    } else if ([operation isEqualToString:@"/"]) {


    } else if ([operation isEqualToString:@"sin"]) {

    } else if ([operation isEqualToString:@"cos"]) {

    } else if ([operation isEqualToString:@"sqrt"]) {

    } else if ([operation isEqualToString:@"pi"]) {


    } else if ([operation isEqualToString:@"x"]) {

    } else if ([operation isEqualToString:@"y"]) {

    } else if ([operation isEqualToString:@"a"]) {

    } else if ([operation isEqualToString:@"b"]) {

    }

Answer 1

我认为没有最佳方法可以做到这一点，但这里有另一种选择来补充已经给出的答案。

创建一个将您的操作名称（sin、sqrt、-）映射到选择器的 NSDictionary，如下所示：

NSDictionary *operations = @{
    @"sin": [NSValue valueWithPointer:@selector(operationSin:)],
    /* other operations here */
};

那么你的 switch 语句就变成了一个查找。

NSString *operation = topOfStack;
if(operations[operation]) {
    SEL op = [operations[operation] pointerValue];
    [self performSelector:op withObject:value];  /* Or some approximation thereof */
}
else {
    /* Default action for unknown operation */
}

或者，您可以在识别的操作上创建一个枚举，使用您的 NSDictionary 将操作名称映射到一个 NSNumber 装箱您的枚举值。

typedef NS_ENUM(NSUInteger, OperationType) {
    OperationTypeSin = 0L,
    OperationTypeSqrt,
    /* and so on */
    OperationTypeUnknown
};

/* And later...  */
NSDictionary *operations = @{
    @"sin": @(OperationTypeSin),
    /* you get the idea */
};

/* Finally ... */
NSString *operation = topOfStack;
OperationType opType = [operations[operation] unsignedIntegerValue];
switch(opType) {
    case OperationTypeSin:
        /* Much cleaner, and type-safe too */
}

Answer 2

准备工作：
首先将 NSString 操作转换为 int hashcode，将该 hashcode 存储在 #define 或常数。

代码：

1) 将 NSString 操作转换为 int hashcode (tokenId)。
2) 然后对该 tokenId 执行 switch 语句。

int token = operationToToken(operation);
switch (token) {
case: OP_MINUS: break;
case: OP_SIN: break;
case: OP_COS: break;

}

Answer 3

首先，可能会退回到 C 字符串，然后利用 sin、cos、sqrt、pi、a、b、x 和 y（几乎）都以不同的字符开头：

const char *s = [operation UTF8String];
switch (s[0]) {
case '+':
    // addition
    break;
case '-':
    // subtraction
    break;
case 's':
    // sine or sqrt
    switch (s[1]) {
    case 'q':
        // sqrt
        break;
    case 'i':
        // sine
        break;
    }
    break;
case 'c':
    // cosine
    break;
// et cetera...
default:
    // not found
}

Answer 4

如果您使用的是 Objective-C，那么您可以使用一些巧妙命名的选择器来做到这一点。

首先，创建一个对象来封装Operation。

然后，创建一个-initWithOperation:(NSString *)op 方法。在此方法中，将任何符号操作（*、+ 等）转换为唯一的字符串标识符（仅限字母）。

然后，您可以调用 NSSelectorFromString(stringOp)，但如果您希望方法具有参数，则必须自己附加 : 字符。

获得选择器后，您可以使用-performSelector:、+performSelector: 执行选择器，或手动调用选择器。

如果您将每个选择器命名为与操作相同的名称，则不需要任何 if 或 switch 语句，但您必须处理发送无效选择器的情况。

例如：

@interface Operation : NSObject

- (id)initWithOperation:(NSString *)op;
- (void)performOperationWithValue:(float)value;
+ (float)sin:(float)value;
// ... Other operations
@property (nonatomic, copy) NSString *theOperation;

@end

@implementation Operation
- (id)initWithOperation(NSString *)op {
    self = [super init];
    if (self) {
        // Convert symbols to unique strings
        theOperation = [[NSString alloc] initWithString:op];
    }
    return self;
}

- (void)performOperationWithValue:(float)value {
    NSString *withOneParam = [self.theOperation stringByAppendingString:@":"];
    SEL sel = NSSelectorFromString(withOneParam);
    [Operation performSelector:withOneParam withObject:[NSNumber numberWithFloat:value]];
}

// Class methods are the actual operation implementation...

在上面的示例中，我假设您已将@"sin" 传递给Operation 对象。这只能在 Objective-C 中实现，因为您可以利用命名选择器。

Answer 5

您可以为每个符号分配一个数字，并可以实现一个开关盒。