《重构 改善既有代码的设计》读书笔记【更新中。。。】

读这本书的初衷是想对自己的代码以旁观者的角度进行一番审查,所以选择了这本网上大家都推荐的书籍——《重构 改善既有代码的设计》。这本书上提供了很多重构代码的技巧方法。个人认为重构最直观的感受是“提高代码的阅读效率”,什么意思?就是当接手你代码的同事能够很快的理解这段代码的意思以及编写上的思维角度,就是能够读懂你当时处理数据的逻辑方式。另一方面就是“提高代码的质量”,接手工作的同事在阅读的同时能够搁在心里连连称赞这段代码编写方式,说明你的重构是有效。

一、重新组织函数

1、Extract Method(提炼函数)(P110)

“你有一段代码可以被组织在一起并独立出来”

Extract Method个人感觉是重构代码最常用的一个方法,在code review或者自我检查代码的时候,应该都会用到这个方法。

动机:

过长的函数或者需要注释才能让人理解用途的代码,这个时候就需要考虑将这段代码放进一个独立的函数。

当你发现代码中有大量的注释来解释这段代码的用途的时候,你就需要着手对这段代码进行重构提炼了,这也是新手重构代码的入口。

做法:

(1)创造一个新函数,根据这个函数的意图对它来命名(命名也是重构的一种方法),命名要做到“见名之意”,让别人看到名字就知道这段代码的用途

(2)将提炼出的代码从源函数复制到新建的目标函数中

(3)检查变量(局部变量和源函数参数),当被提炼代码段中需要读取局部变量时,这是就需要将局部变量当作参数传给目标函数;当有临时变量仅作用于被提炼代码段时,在目标函数中将它们声明为临时变量。这部分是整个Extract Method的精髓所在,需要处理好

(4)处理完所有局部变量之后,进行编译

(5)在源函数中,将被提炼断码段替换为对目标函数的调用

(6)编译,测试

提炼函数这种方式,可以直接使用IDE工具中的快捷键,可以很方便的帮你处理第三个步骤中的变量,自动的检测被提炼代码段所需要的一些参数

2、Replace Temp with Query(以查询取代临时变量)

将这个表达式提炼到一个独立函数中。将这个临时变量的所有引用点替换为对新函数的调用。此后,新函数就可能被其他函数使用。

看到这中重构方式之后,自己也经常在代码中运用这中手段。去掉一些临时变量的运用,简化一些代码。

动机:

临时变量是暂时的,只能在所属函数内使用。由于临时变量只在所属函数内可见,所以会驱使写出更长的函数。如果把临时变量替换为一个查询,那么同一个类中的所有函数都将可以获得这份信息。

这种重构的最简单情况就是:临时变量只被赋值一次,或者赋值给临时变量的表达式不受其他条件影响

做法:

(1)找出只被赋值一次的临时变量

(2)将该临时变量声明为final

(3)编译(确保该临时变量的确只被赋值一次)

(4)将“对该临时变量赋值”的语句的等号右侧部分提炼到一个独立函数中

(5)编译,测试

(6)在该临时变量身上实施Inline Temp(P119)

范例:

示例函数

double getPrice(){
    int basePrice=_quantity*_itemPrice;
    double discountFactor;
    if(basePrice>1000) 
        discountFactor=0.95;
    else 
        discountFactor=0.98;
    return basePrice*discountFactor;
}

声明为final

double getPrice(){
    final int basePrice=_quantity*_itemPrice;
    final double discountFactor;
    if(basePrice>1000) 
            discountFactor=0.95;
        else 
            discountFactor=0.98;
    return basePrice*discountFactor;
}

替换临时变量

double getPrice(){
    final int basePrice=basePrice();
    final double discountFactor;
    if(basePrice>1000) 
            discountFactor=0.95;
        else 
            discountFactor=0.98;
    return basePrice*discountFactor;
}

private int basePrice(){
    return _quantity*_itemPrice;
}

使用Inline Temp

    double getPrice(){
    final int basePrice=basePrice();
    final double discountFactor;
    if(basePrice()>1000) 
            discountFactor=0.95;
        else 
            discountFactor=0.98;
    return basePrice*discountFactor;
}

private int basePrice(){
    return _quantity*_itemPrice;
}

替换其他临时变量,删掉临时变量

    double getPrice(){
    //final int basePrice=basePrice();
    final double discountFactor;
    if(basePrice()>1000) 
            discountFactor=0.95;
        else 
            discountFactor=0.98;
    return basePrice()*discountFactor;
}

private int basePrice(){
    return _quantity*_itemPrice;
}

提炼discountFactor()

double getPrice(){
    final double discountFactor=discountFactory();
    return basePrice()*discountFactor;
}

private int basePrice(){
    return _quantity*_itemPrice;
}

private double discountFactor(){
    if(basePrice()>1000)
        reurn 0.95;
    else 
        return 0.98;
}

最终重构函数getPrice()

double getPrice(){
    return basePrice()*discountFactor();
}

private int basePrice(){
    return _quantity*_itemPrice;
}

private double discountFactor(){
    if(basePrice()>1000)
        reurn 0.95;
    else 
        return 0.98;
}

在实际的开发过程中,如果能一眼就能发现的简单的可替换的临时变量,不妨动手试试!

重构之后,发现代码行数变多了,方法增加了,但是整体看上去,发现逻辑清楚了。

3、Remove Assignments to Parameters(移除对参数的赋值)

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,189评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,577评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 150,857评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,703评论 1 276
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,705评论 5 366
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,620评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 37,995评论 3 396
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,656评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,898评论 1 298
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,639评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,720评论 1 330
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,395评论 4 319
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,982评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,953评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,195评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 44,907评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,472评论 2 342

推荐阅读更多精彩内容