Các cách để thay thế câu lệnh if lớn với cấu trúc thay thế trong Mục tiêu-C

Question

Tôi có một câu lệnh if khá dài. Câu lệnh if kiểm tra một chuỗi "type" để xác định loại đối tượng nào cần được khởi tạo. Đây là mẫu ...

if ([type rangeOfString:@"coin-large"].location != NSNotFound) 
{ 
    ... create large coin ... 
    mgr = gameLayer.coinLargeMgr; 
} 
else if ([type rangeOfString:@"coin-small"].location != NSNotFound) 
{ 
    mgr = gameLayer.coinLargeMgr; 
} 

... more else statements ... 

myObject = [mgr getNewObject]; 

Câu lệnh "else-if" tiếp tục cho các loại đối tượng khác hiện có khoảng 20 và số đó có thể tăng lên. Điều này hoạt động khá tốt nhưng về mặt bảo trì và hiệu quả, tôi nghĩ rằng nó có thể được cải thiện. Ứng cử viên hàng đầu của tôi ngay bây giờ là tạo ra một khóa NSDictionary trên chuỗi loại đối tượng (đồng xu nhỏ, đồng xu lớn, v.v.) và với giá trị của đối tượng người quản lý cần được gắn với loại đó. Ý tưởng được rằng đây sẽ là một cái nhìn nhanh chóng cho các loại đối tượng tôi cần phải tạo ra. Bạn không chắc đây là cách tiếp cận tốt nhất, tiếp tục xem xét các tùy chọn khác nhưng tôi tò mò về những gì mà mọi người ở đây có thể đã làm cho một vấn đề tương tự. Bất kỳ trợ giúp/phản hồi nào được đánh giá cao.

Answer 1

Cách tiếp cận từ điển sẽ dễ dàng thực hiện được. Giả sử các nhà quản lý khác nhau đã được luộc xuống đến trường hợp cụ thể khi bạn tạo từ điển, nó muốn được chỉ giống như hầu hết các hướng đối tượng ngôn ngữ:

NSDictionary *stringsToManagers = 
    [NSDictionary dictionaryWithObjectsAndKeys: 
     @"coin-large", gameLayer.coinLargeMgr, 
     @"coin-small", gameLayer.coinSmallMgr, 
     nil]; 

// this is assuming that type may contain multiple types; otherwise 
// just use [stringsToManagers objectForKey:string] 
for(NSString *string in [stringsToManagers allKeys]) 
{ 
    if([type rangeOfString:string].location != NSNotFound) 
    { 
     [[stringsToManagers objectForKey:string] addNewObject]; 
     // or get it and store it wherever it should go 
    } 
} 

Nếu tất cả các nhà quản lý làm là vend đối tượng thích hợp, cách tiếp cận hướng đối tượng hơn có thể là:

NSDictionary *stringsToClasses = 
    [NSDictionary dictionaryWithObjectsAndKeys: 
     @"coin-large", [LargeCoin class], 
     @"coin-small", [SmallCoin class], 
     nil]; 

// this is assuming that type may contain multiple types; otherwise 
// just use [stringsToManagers objectForKey:string] 
for(NSString *string in [stringsToManagers allKeys]) 
{ 
    if([type rangeOfString:string].location != NSNotFound) 
    { 
     id class = [stringsToManagers objectForKey:string]; 

     id newObject = [[class alloc] init]; 
     // this is exactly the same as if, for example, you'd 
     // called [[LargeCoin alloc] init] after detecting coin-large 
     // within the input string; you should obviously do something 
     // with newObject now 
    } 
} 

Điều đó có thể giúp bạn viết bất kỳ người quản lý nào nếu cấu trúc chương trình của bạn phù hợp.

Answer 2

Bạn có thể sử dụng NSDictionary chứa ObjC 'Blocks' để thực hiện lệnh chuyển đổi giống như thực thi mã mong muốn. Vì vậy, hãy một cuốn từ điển với các phím chuỗi của bạn ánh xạ tới một khối mã để thực thi khi từng được tìm thấy:

NSDictionary *dict = [NSDictionary dictionaryWithObjectsAndKeys: 
         ^{ NSLog(@"found key1"); }, @"key1", 
         ^{ NSLog(@"found key2"); }, @"key2", 
         nil]; 

Bạn có lẽ sẽ chuẩn bị từ điển này chỉ một lần tại một số giai đoạn đầu như thế nào trong một constructor hoặc một initializer tĩnh để rằng nó đã sẵn sàng khi mã sau của bạn thực hiện.

Sau đó, thay vì bạn if/else khối, cắt ra chìa khóa chuỗi từ bất cứ intput bạn nhận được (hoặc có thể bạn sẽ không cần phải cắt nó, bất cứ điều gì):

NSString *input = ... 
NSRange range = ... 
NSString *key = [input substringWithRange:range]; 

Và làm (nhanh) tra cứu từ điển để mã thực thi. Sau đó, thực hiện:

void (^myBlock)(void) = [dict objectForKey:key]; 
myBlock();