iOS代码重构之提升类的接口抽象能力

今天看到《代码大全第二版》里面对ADT及类抽象能力的描述让我受益不浅，查看咱们的iOS项目，立刻就找到了这样的坏味道（不良好的类接口设计），下面就分享一下个人改进方法。（这里的类名作了修改，并不是实际项目中的类名）

---

改进前:

这是一个自定义单元格类:

@protocol  CustomCellDeletage <NSObject>

- (void)checkDidTouch:(CustomCell *)cell;

@end

@interface CustomCell : UITableViewCell

@property (nonatomic, weak)id<CustomCellDeletage> delegate;

@property (nonatomic, retain)UIButton * checkBtn;
//这里直接暴露了类的成员变量细节，致使类对自身内部控制的松动，按钮的状态改变将难以追踪，而且这个类和按钮对象如何使用也会让调用方摸不着头脑，此次的改动将以此为重

@end

@implementation CustomCell

- (instancetype)initWithStyle:(UITableViewCellStyle)style reuseIdentifier:(NSString *)reuseIdentifier{
    self = [super initWithStyle:style reuseIdentifier:reuseIdentifier];
    if (self) {
        self.selectionStyle = UITableViewCellSelectionStyleNone;
        _checkBtn = [UIButton buttonWithType:UIButtonTypeCustom];
        [_checkBtn addTarget:self action:@selector(checkBtnPressed:) forControlEvents:UIControlEventTouchUpInside];
        //...省略部分代码

    }
    return self;
}
- (void)checkBtnPressed:(id)sender {
    // 切换选中状态
    _checkBtn.selected = !_checkBtn.selected;
    
    if (self.delegate && [self.delegate respondsToSelector:@selector(checkDidTouch:)]) {
        [self.delegate checkDidTouch:self];
    }
}
复制代码

外部Controller调用:

cell.checkBtn.selected = [self isChecked:orderDict[@"orderId"]];
复制代码

外部Controller回调：

#pragma mark CustomCellDeletage ---- 点击选择按钮事件
- (void)checkDidTouch:(CustomCell *)cell {
    // 若是是选中状态，就添加   反之这移除
    if (cell.checkBtn.selected) {
        [self.checkArray addObject:cell.orderDict];
    } else {
        [self.checkArray removeObject:cell.orderDict];
    }
    [self updateViewInfo];
}
复制代码

这里根据《代码大全第二版》做者书里对ADT好处的描述来作代码评价。因为这里直接暴露了类的成员变量checkBtn而带来了不少坏处。

ADT(抽象数据类型)

• 隐藏实现细节

若是数据类型发生改变，你只需在一处修改而不会影响程序的其他部分。例如这里若是由UIButton类型变为UISwitch,应只须要改动接口的具体实现，而不影响接口调用方

• 让接口能提供更多信息，程序更具自我说明性

例如这里后面改进了类的接口为selectOn和selectOff一组更具说明意义的方法

抽象

• 类的接口为隐藏在其后的具体实现提供一种抽象
• 类的接口应该展现一致的抽象层次

例如这里cell单元格类，应提供选中单元格和取消选中单元格的接口，而不是直接暴露成员变量

• 尽量地限制类和成员的可访问性以提升封装性
• 不要公开暴露成员数据
• 避免把私用的实现细节放入类的接口中

核心改动思想

把对按钮的操纵隔离到一组子程序里，为须要操做按钮的其它程序部分提供更好的抽象层，同时能够在针对按钮属性状态的操做发生变化时提供一层保护。 像在现实世界中那样操做实体，而不用在底层实现上操做它

---

改进

@protocol  CustomCellDeletage <NSObject>

- (void)customCellDidSelectOn:(CustomCell *)cell;

- (void)customCellDidSelectOff:(CustomCell *)cell;

@end

@interface CustomCell : UITableViewCell

@property (nonatomic, weak)id<CustomCellDeletage> delegate;

- (void)selectOn;

- (void)selectOff;

@end

@interface CustomCell ()

@property (nonatomic, strong)UIButton * checkBtn;

@end

@implementation CustomCell

- (instancetype)initWithStyle:(UITableViewCellStyle)style reuseIdentifier:(NSString *)reuseIdentifier{
    self = [super initWithStyle:style reuseIdentifier:reuseIdentifier];
    if (self) {
        self.selectionStyle = UITableViewCellSelectionStyleNone;
        _checkBtn = [UIButton buttonWithType:UIButtonTypeCustom];
        [_checkBtn addTarget:self action:@selector(checkBtnPressed:) forControlEvents:UIControlEventTouchUpInside];
         //...省略部分代码
    }
    return self;
}
- (void)checkBtnPressed:(id)sender {
    if (_checkBtn.selected) {
        [self selectOff];
        if (self.delegate && [self.delegate respondsToSelector:@selector(customCellDidSelectOff:)]) {
        [self.delegate customCellDidSelectOff:self];
    }
    } else {
        [self selectOn];
        if (self.delegate && [self.delegate respondsToSelector:@selector(customCellDidSelectOn:)]) {
        [self.delegate customCellDidSelectOn:self];
    }
    }
}
- (void)selectOn {
    _checkBtn.selected = YES;
}
- (void)selectOff {
    _checkBtn.selected = NO;
}
@end

复制代码

外部Controller调用:

if ([self isChecked:orderDict[@"orderId"]]) {
        [cell selectOn];
    } else {
        [cell selectOff];
    }
复制代码

外部Controller回调:

#pragma mark <CustomCellDelegate>
- (void)customCellDidSelectOn:(CustomCell *)cell {
    
    [self.checkArray addObject:cell.orderDict];
    [self updateViewInfo];

}

- (void)customCellDidSelectOff:(CustomCell *)cell {
    
    [self.checkArray removeObject:cell.orderDict];
    [self updateViewInfo];
}
复制代码

改动点:

1. 将按钮做为类的私有属性
2. 将改变按钮对象的selected状态包装起来，提供更高层的抽象selectOn和selectOff接口，抽象成更像现实世界同样操做实体，选中单元格和取消选中单元格，而不是直接接触底层细节，将按钮对象的selected属性设置成YES或NO。
3. 调用方调用更高层次、更具说明意义的抽象接口selectOn和selectOff方法，而无须关心实现细节。cell内部也可经过此方法改变外观，之后关于外观的改动也无须在多个地方改动了。
4. 回调方法职责更单一，原来在一个方法中处理选中和非选中，如今分开到两个方法中，干掉了一个if,也就下降了函数的复杂度，之后添加选中或取消选中后可能会处理其它事情将更容易维护。
5. 若之后将按钮对象更换为其它类型对象，或是更复杂的自定义按钮对象，那只需改动封装内的内部实现，不影响程序的其它地方的使用。

改进后的代码:

• 隐藏了实现细节，提升了类的封装性
• 类的接口和类保持一致的抽象层次
• 类接口更高层次的抽象
• 单一职责

有同窗可能以为这样的改动有那么必要吗，弄的如此大张旗鼓，但咱们应该时刻保持代码的干净整洁，若是每一个类都不能时刻保持对外接口的高度一致抽象层次，那么多个迭代之后，这样的类将愈来愈难以理解和难以维护，类持续的庞大和存在各类互不相关的数据和逻辑，增长新功能时更容易牵扯老功能的bug，咱们应该时刻记住

勿以恶小而为之，勿以善小而不为