程序员

《重构》第五、六章 - 常用的重构方法推荐?

2022-12-15  本文已影响0人  李周

书中每个章节都推荐了大量的重构手段,当开发者在面对大量无论是已经熟悉的还是新了解的重构手段时,如何快速的回忆并且选择更高效的方式进行重构? 对于这个问题书中推荐了一种重构记录方式:

采用从名称、速写、动机、做法以及范例五个方面进行记录。

下面通过文字的方式记录作者常使用的几种重构方式:

提炼函数

内联函数

- (void)getRating:(Driver *)driver {
    return [self moreThanFiveLateDeliveries:driver] ? 2: 1;
}

- (BOOL)moreThanFiveLateDeliveries:(NSInteger)driver {
    return driver.numberOfLateDeliveries > 5;
}

以简单的例子解释,moreThanFiveLateDeliveries: 函数名作为中间层并没有为原来的逻辑增彩,无任何的价值,所以这时候可以直接将函数去掉直接使用实现中的代码:

- (void)getRating:(Driver *)driver {
    return driver.numberOfLateDeliveries > 5 ? 2: 1
}

提炼变量(Extract Variable)

- (CGFloat)price:(Order *)order {
    return order.quantity * order.itemPrice - Math.max(0, order.quantity - 500) * order.itemPrice * 0.05 + Math.min(order.quantity * order.itemPrice * 0.1, 100);
}

在阅读以上函数内容时,虽然使用的参数较少、命名规范,但是短时间内还是无法明白其中计算逻辑,所以如果将表达式赋值给局部变量:

首先  order.quantity * order.itemPrice = 底价
其次 Math.max(0, order.quantity - 500) * order.itemPrice * 0.05 = 批发折扣
最后  Math.min(order.quantity * order.itemPrice * 0.1, 100) = 运费
通过定义参数名,针对每个变量加一些业务注释,就能更好的理解这段代码:
CGFloat basePrice = order.quantity * order.itemPrice 
CGFloat quantityDiscount = Math.max(0, order.quantity - 500) * order.itemPrice * 0.05
CGFloat freight = Math.min(basePrice * 0.1, 100)

最终代码呈现:

- (CGFloat)price:(Order *)order {
    // 底价
    CGFloat basePrice = order.quantity * order.itemPrice; 
    // 批发折扣
    CGFloat quantityDiscount = Math.max(0, order.quantity - 500) * order.itemPrice * 0.05;
    // 运费
    CGFloat freight = Math.min(basePrice * 0.1, 100);
    return basePrice - quantityDiscount + freight;
}

内联变量

NSInteger age = lizhou.age;

这种局部变量毫无价值,应使用该重构方法去除。

改变函数声明

@interface TopModel: NSObject 
@property (nonatomic, strong) NavModel *nav;
@property (nonatomic, strong) contentModel *content;
@property (nonatomic, strong) BottomModel *bottom;
@end

1.改变参数列表:定义合适的范围

- (void)bindData:(TopModel *)model {
    // 内部只使用了TopModel中的nav 和 bottom中的A个字段
    // 从某种意义上来说,可以只传递 nav 和 bottom中的 A字段
    // 在开发递进的过程中,当参数不断增加后可以自定义独立的Model,但是一次性将顶层的 TopModel 传递进来,反而限制了这个类的使用范围
}

2.当修改某个方法名或参数列表后,应该使用 deprecated 标注,而不是直接删除。等到版本升级一段时间后再进行移除。

- (void)bindData:(TopModel *)model __attribute__((deprecated));
- (void)bindData:(NavModel *)nav withA:(NSString *)A;

引入参数对象

@interface NavModel: NSObject 
@property (nonatomic, strong) NavLeftModel *left;
@property (nonatomic, strong) NavRightModel *right;
@property (nonatomic, strong) NavCenterModel *center;
@end
@interface TopModel: NSObject 
@property (nonatomic, strong) NavModel *nav;
@property (nonatomic, strong) contentModel *content;
@property (nonatomic, strong) BottomModel *bottom;
@end

在传递时,有多个地方使用AView,传递的数据较多但是数据结构不尽相同时,为了提高AView的复用性这时候可以定义AModel来封装所有需要传递的字段。

@interface AModel: NSObject 
// 自定义需要的字段
@end

使用这种方法重构会导致更深层次的改变,也就相当于开发者只要在使用该类时就需要将自己持有的数据结构转换成AModel的格式,当然如果AModel中字段定义存在歧义,那么就达不到想要的效果。所以开发者应尽可能的增加通用化数据结构设计!

上一篇 下一篇

猜你喜欢

热点阅读