基于枚举构造对象

Question

我有一个基于枚举值构建对象的类。所以这些对象的几个属性基于一些枚举值，一个类型。

选项1：

typedef NS_ENUM (NSUInteger, ViewType) { 
    VTHouse, 
    VTCar, 
    VTChair, 
    ... 
}; 

我有几种方法，确定基于该类型的对象的属性。

- (NSURL*)urlForViewType:(ViewType)type { 

    NSURL *url = nil; 

    switch (type) { 
     case VTHouse: { 
      url = [NSURL URLWithString:@"House url"]; 
      break; 
     } 
     case VTCar: { 
      url = [NSURL URLWithString:@"Car url"]; 
      break; 
     } 
     case VTChair: { 
      url = [NSURL URLWithString:@"Chair url"]; 
      break; 
     } 
     ... 
    } 

    return url; 
} 

- (NSURL*)isSelectableViewType:(ViewType)type { 

    BOOL selectable = NO; 

    switch (type) { 
     case VTHouse: { 
      selectable = YES; 
      break; 
     } 
     case VTCar: { 
      selectable = YES; 
      break; 
     } 
     default: { 
      break; 
     } 
    } 

    return selectable; 
} 

- (NSURL*)colorForViewType:(ViewType)type { 

    UIColor *color = nil; 

    switch (type) { 
     case VTHouse: { 
      color = [UIColor redColor]; 
      break; 
     } 
     case VTCar: { 
      color = [UIColor blueColor]; 
      break; 
     } 
     case VTChair: { 
      color = [UIColor lightGrayColor]; 
      break; 
     } 
     ... 
    } 

    return color; 
} 

// And so on... 

然后我有一个这个类的用户会调用的方法。

- (SpecialView*)specialViewForType:(ViewType)type { 

    NSURL *url = [self urlForViewType:type]; 
    BOOL selectable = [self isSelectableViewType:type]; 
    UIColor *color = [self colorForViewType:type]; 
    ... 

    return [SpecialView specialViewURL:url selectable:selectable color:color ...]; 
} 

这一切都很好，但它给我一个不安的感觉。有些事情感觉不对。也许这是所有的开关。我觉得有一个更干净的方法来做到这一点。

摆脱大部分开关的另一种选择是类似的;

选项2：

- (SpecialView*)specialViewForType:(ViewType)type { 

    SpecialView *view = nil; 

    switch (type) { 
     case VTHouse: { 
      view = [self specialViewHouse]; 
      break; 
     } 
     case VTCar: { 
      view = [self specialViewCar]; 
      break; 
     } 
     case VTChair: { 
      view = [self specialViewChair]; 
      break; 
     } 
     ... 
    } 
    return view; 
} 

其中每种方法已经知道哪些属性为每个类型设置。但我更喜欢选项1.

所以我的问题是;有没有人对如何改进这种代码有任何建议？

Answer 1

开关是最后一个千年的亚类。最简单的方法是拥有（私人）子类。从该子类创建实例。

@implementation BaseClass 
+ (instancetype)newBaseClassForType:(ViewType)viewType 
{ 
    // Do a look-up to a array or a one-time switch to get the subclass 
    Class appropiateSubclass = …; 
    return [appropiateSubclass new]; 
} 

然后，子类可以覆盖方法，即。 e .:

@implementation HouswSubClass 
- (BOOL)isSelectable { return YES; } // BTW: The return type was wrong 

更重要的是：为什么你有一个枚举？