分享让人折服的优秀代码基因

| 阅读数:--次| 作者:热点
摘要:近来参与一个较大团队的项目实施,项目的金额几千万,人数近百。但是,项目实施后,暴露出几个问题.

1 背景

近来参与一个较大团队的项目实施,项目的金额几千万,人数近百。但是,项目实施后,暴露出以下几个问题:

(1)质量不佳,团队成员水平参差不齐,软件外部质量、内部质量一致性差;

(2)需求不确定,时间非常紧,代码频繁修改,越来越丑,效率变低。

为了保证项目按时按质交付,质量改善刻不容缓。因此,在项目的初中期开始,做了以下三件事情:

(1)制定统一的界面规范,制定了统一参考实例,为所有成员进行定期界面规范的培训和评审;

(2)制定统一的代码规范,制定了《评审文化构建》、《代码之丑》PPT,培养团队的质量文化和评审文化,实施评审;

(3)引入管理工具ReviewBoard。

项目统一构建了标准、规范,有效保证了界面的一致性,但是代码质量的提高却不是那么简单的事情。在这里,我暂不分享项目的情况,未来会通过更加详细的文章来介绍标准、规范化软件开发方法对于项目的重要性。

2 ReviewBoard介绍

这里倒是引出了一款优秀工具ReviewBoard。这款工具简单易用,功能强大。通过 它来实施代码评审非常有效。代码评审分为提交前评审(Pre-Review)和提交后评审(Post-Review)两种方式。提交前评审,即开发人员代 码变更后,需要提交到ReviewBoard,经过评审通过后,才能提交到SVN源码服务器,如果没有通过评审,则代码提交会失败,一行都无法提交;提交 后评审,即开发人员先提交代码,然后再提交变更到ReviewBoard,如果评审未通过,则修改代码,更新评审直到通过为止。这两种方式,各有优劣,我 们采用的是后一种,它不阻塞开发人员提交代码,无法100%控制所有质量,但是可以达到80%以上。

ReviewBoard设计之初更多考虑的是支持Pre-Review方式,因此,存在 以下问题:(1)提交通过后,Review无法自动进行状态变更为关闭,会与所有通过评审的ReviewRequest混在一起;(2)无法看出未通过评 审的请求进行再次更新,因为如果再次更新后,意味着要进行第二次评审。在管理过程中,会查询哪些请求没有通过评审,定期给开发人员发送通知。

因此,我决定对ReviewBoard进行订制来支持以上功能。不 过,ReviewBoard是基于Python语言和基于Django框架开发,我从来没有学习过Python,更没有学习过Django,那如何来修 改?所以,我开始对ReviewBoard做研究。ReviewBoard最新版本只能在非Windows运行,我在Ubuntu 14.04安装部署,同时使用MySQL数据库。因此,我开始来尝试做修改。

3 尝试修改ReviewBoard

3.1 探索第一步:数据库

首先,研究数据库,惊讶发现其数据库设计非常的整齐,压根不需要查看任何文档即可知道数 据库的表、字段的意义。我找到了几个关键的表,分别是reviews_reviewrequest,看看其字段,与界面显示几乎一致,一下子就明白什么意 思,顺着字段意思,找到了另外的两个关键表diffviewer_diffsethistory、diffviewer_diffset。查询一下这些表 的数据,我很容易来解决第一个问题,通过一条SQL语句即可将通过评审的ReviewRequest关闭,这样就不需要查看到这些评审请求了。

3.2 学习Python并修改代码

接下来的第二个问题,需要修改代码,我花了3个小时学习了Python,通过在线的英文帮助,直接在控制台做实验,学习基本语法结构、学习类与对象编程、学习了Python模块与编译,对Python初步熟悉之后,我决定开始来进行代码修改。

因此,我先通过相似列“Submitter”对所有文件进行查询,通过它很快找到了 columns.py和grids.py这两个文件,看了代码之后,发现ReviewBoard的评审请求通过这两个文件来实现,进行再次查询名称没有再 发现关联文件了。通过这两个文件,可以发现,columns.py用于定义要显示的所有的列,grids.py进行列的组合和显示。那么接下来可以简单的 进行模仿添加一个列了。该列定义为Diffs。

1

2

3

4

5

6

7

8

9

10

11

12

13

14

15

16

17

18

19

20

21

22

23

24

25

26

27

28

29

class DiffSetCountColumn(Column):

    """Shows the diff set count of published reviews for a review request."""

    def __init__(self, *args, **kwargs):

        super(DiffSetCountColumn, self).__init__(

            label=_('Diffs'),

            detailed_label=_('Number of Diffs'),

            shrink=True,

            sortable=True,

            link=False,

            *kwargs, **kwargs)

    def render_data(self, state, review_request):

        if review_request.mydiffsetcount > 1:

            return ('<span class="issue-count">'

                    ' <span class="issue-icon">!</span> %s'

                    '</span>'

                    % review_request.mydiffsetcount)

        else:

            return ''

    def augment_queryset(self, state, queryset):

        return queryset.extra(select={

            'mydiffsetcount': """

                SELECT COUNT(*)

                  FROM diffviewer_diffset

                  WHERE diffviewer_diffset.history_id =

                        reviews_reviewrequest.diffset_history_id

            """

        })

简单说明一下,我定 义了DiffSetCountColumn类,继承于Column类,类初始化方法为__init__,相当于构造器。这里的self是以前经常使用的 this,定义了两个方法,一个是augment_queryset,用于当前列的数据显示,定义名称为mydiffsetcount,另外定义一个方法 render_data,即在表格中展现数据。这里参考了其它列定义。

接下来在grids.py中声明对该列的使用,非常简单。

25行新增:

DiffSetCountColumn,

99行新增:

diffset_count = DiffSetCountColumn()

接着重启Apache服务器,可以简单测试,惊讶发现可以工作了。

3.3 再进一步添加功能

有了这次尝试,我接下来想再进一步来定制。目前各个小组提交到不同的SVN地址,通过SVN可以区分各个组提交情况。但是ReviewBoard没有针对SVN地址的过滤。

首先修改columns.py,修改Repository列。 

1

2

3

4

5

6

7

8

9

10

11

12

13

14

15

16

17

18

19

20

21

22

23

24

class RepositoryColumn(Column):

    """Shows the name of the repository the review request's change is on."""

    def __init__(self, *args, **kwargs):

        super(RepositoryColumn, self).__init__(

            label=_('Repository'),

            db_field='repository__name',

            shrink=True,

            sortable=True,

            link=True,

            link_func=self.link_to_object,

            css_class='repository-column',

            *args, **kwargs)

    def augment_queryset(self, state, queryset):

        return queryset.select_related('repository')

    def render_data(self, state, obj):

        return super(RepositoryColumn, self).render_data(state, obj) or ''

    def link_to_object(self, state, obj, value):

        import urllib

        return local_site_reverse('all-review-requests',

                                  request=state.datagrid.request) + '?' +

               urllib.urlencode({'repository':obj.repository_id})

在这里更改该列支持Link,接着定义 link_to_object函数,这里找了一段时间,发现local_site_reverse可以用于获取需要的url,然后再与url组合,一开始 没有使用urlencode,一直无法正常呈现。由于原来在ASP.NET中,使用过urlencode,知道这里面有坑,就网上查询了Python的 urlencode,然后解决之,发现可以工作了。不过,下一个问题来了,即链接后,如何进行过滤?这里发现其“show closed”过滤功能,通过代码查询,知道在grids.py中,如何工作,那么,添加过滤也就不难了。如下。

grids.py 第116行

self.repository_query = ‘'

grids.py 第133行

      self.repository_query = self.request.GET.get('repository', '')        

      if len(self.repository_query) > 0:

           self.queryset = self.queryset.filter(repository_id=int(self.repository_query))

4 ReviewBoard令人震惊的优秀代码基因

顺利修改之后,我大为震惊,为ReviewBoard这款优秀的软件说叹服。为什么能够 在1天时间里面,从Python学习到对一个完全黑盒子的软件进行修改?答案就是ReviewBoard具有非常整洁的代码。也就是 说,ReviewBoard拥有非常优秀的内外部质量。

数据库设计,简单清晰,结构非常清晰,可读性很强,根本不需要任何数据库文档。文档是多余的!!!

代码风格非常一致,我们发现ReviewBoard的注释非常少,Bob大叔的《代码整 洁之道》强调了注释是代码意图表现失败的补充,最好的代码是一行注释都不要。ReviewBoard的命名,从前到后都非常一致,大小写、下划线、类名、 方法名规则统一,命名准确无误,没有不专业的命名,没有不专业的缩写。ReviewBoard的类都非常简单,在columns.py这个文件,你可以发 现每一个Column基本都在50行以内,每一个方法也非常简单。ReviewBoard非常完美的践行了单一职责,清晰的告诉我,要完成什么任务可以通 过哪些类、哪些方法、写什么样代码来完成。ReviewBoard代码风格非常优秀,没有丑陋的缩紧,没有丑陋的拥挤,在该有空格的时候空格,在该有空行 的时候空行,在该缩紧的时候缩紧。这里,我看不出任何代码腐败的味道。跟着这种优秀的软件“混”,我也很难写出差的代码。

在写代码这件小事上,可能很多人眼里只有架构、框架这类东西,可是实际上,很多工作多年 的程序员,代码依然是狗屎一样,没有规范的风格,没有良好的编码习惯。很多人能够去完成一件工作,但是,不漂亮。在没有意识的情况下,随着时间推移,只是 做的丑事更多而已。从我的编码经验,可以总结出一个程序员现状,1~3年的开发人员,处于混乱状态,他们能完成功能就不错了;3~5年的开发人员,有些抽 象意识,但是,一种是走向黑客,代码抽象的只有自己和上帝看得懂,一种是变得专业,倾向编写一些让团队其他人员看得懂的代码,还有一种就是保持现状;5年 之后,开发人员又是三类,一类走向管理,也是不错选择,一类走向更高的技术路线如架构师或者专家,还有一类,就是变成废物,随着时间的推移,他们越来越没 有空间,越来越没有价值。一般而言,通过培养,年纪越轻的人,成长越快,越有前途,要提升他们的水平,代码评审是最有效的方法,通过评审和培训让他们知道 什么是美、什么是丑,什么是优秀、什么是低劣。

5 致敬

再次致敬ReviewBoard!ReviewBoard既是提升我们整体实力的有效工具,也是一款值得学习的软件,感谢ReviewBoard!(我激动的想给它捐款时,竟然找不到按钮~~)

另外,我们使用了TaoReviewBoard、FindBugs、JSHint这些工具,感谢你们~~。

在代码评审,我大量引用Bob大叔的两本著名书籍《代码整洁之道》、《程序员的职业素养》,非常好的书,感谢Bob大叔!

返回顶部
学到老代码浏览 关闭浏览