dwbin 发表于 2013-2-3 13:58:01

java

作为一个并不是资深的java开发者,妄断的谈论设计多多少少有些不自量力,但是有些东西还是想说出来。

首先,什么才是好的代码?

好的代码并没有一个绝对的标准,我们可以看一些比较不错的开源软件的代码,比如struts,spring等等,他们的代码的模块化和层次感都很分明,每个方法的意思从名称上面就可以理解,这样的代码想必维护和扩展起来会比较简单,可以称之为设计良好的代码,但是同时我们看jdk里面的代码,很多底层的封装无法做到模块化和层次分明,但是就效率和代码组织来说感觉也是不错的,当然这些代码大都不利于阅读。但是并不能说这些代码就不是好的代码。

我觉得好代码的标准是这样的:

首先是易读,只要代码读起来轻松,这样的代码就可以算是好的代码,比如下面:

boolean isLogin = isUserLogin(User);

if(isLogin){}

和if(isUserLogin(user)){}

这两种写法,我相对推荐第二种写法,因为第二种写法更加接近于我们的语言,而不是计算机逻辑,第一种需要这样理解,首先判断是否已经登录,然后再根据登录状态进行相应的处理,但是第二种就可以理解为,如果已经登陆了就怎样怎样。符合生活的思考习惯。当然第二种写法需要阅读者相对深一些的技术基础。

除了易读,剩下的就是易维护,什么样的代码才算作是易维护的代码呢?私人认为横向扩展的代码较纵向扩展易于维护,拿我以前做过的一个文件类型验证作为例子:

一个前人的思路:

if(fileName.indexOf("txt")){}

if(fileName.indexOf("gif")){}

这个就是一种纵向的扩展,随着种类的增多,代码的逻辑变多,如果牵扯到改动也会比较麻烦。

我做的一个思路:

首先定义一个接口:

Interface Validate{

    boolean doValidate(fileName);

}

然后为一系列的接口进行分发,根据名称进行分发:

Stirng className = fileName.subString(len1,len)+"Validater";

instance = (Validate)Class.forName(className);

instance.doValidate(fileName);

以后每增加一种类型的鉴定就为该类型增加一个实现该接口的类如下:

TxtValidater,GifValidater,

当然这里写的只是伪代码,用来说明一下情况的。这样的鉴定就有了一个横向的无限扩展性,当然也有个坏处就是类型特别多的时候会出现类爆炸的情况。要避免这种情况就需要对实现类进行更加进一步的抽象。当然类爆炸出现的情况还是比较少的。

除了可读,可扩展,好的代码还应该具有一个明确的描述名称,比如isLogin(User)明显比check(User)的意义要明朗的多。

当然好的代码的结构组织也非常重要。记住格式化代码就是了。

差点忘记了,要体现良好的代码就一定要学会分层,mvc是最基本的,mvc之后是service层和dao的组合等等,不过我的一个意见就是一般的bs程序最少要分为三层,这三层分别是这样:

mvc用作前端控制,controller中仅仅包含界面的跳转逻辑,具体的业务流程需要实现一个service层,service层需要不涉及任何的数据库直接操作和对于http类接口的调用,然后是dao层,该层次将不考虑任何的业务逻辑,只负责数据库逻辑,比如要更新某两个相互关联的表,但是数据库逻辑上尽可能的拆分成不同的dao进行操作,这样再辅以明确的命名和注释,以后维护该部分代码的时候将会省力的多。

一些陋见,欢迎大家拍砖!
页: [1]
查看完整版本: java