一个非常简单的例子,反映了很多开发人员的通病

最近在项目中接到一个任务,我负责后台开发,另一开发人员负责前台开发。任务非常简单,请看下面的类图。

 

A和B有一个单向的关联关系,现在要为A增加一个属性boolean resident,该属性值有如下简单的业务逻辑决定(伪代码):

if(a.x == a.b.x)
   resident = true;
else
   resident = false;

 

前台的查看页面需要显示这个值,当用户修改a.x或者a.b.x时,要触发一个事件,实时的在前台更新resident,后台需要以这个resident来

做某些业务逻辑的判断。

我们知道,由于resident完全可以由a.x与a.b.x决定,属于一个冗余数据,不需要显式的增加该属性。很快,前台的那位开发人员就轻松的

告诉我,这个任务已经写完了,原来他是在页面写了一个JS脚本,当a.x或a.b.x发生改变时,触发一个onChange()事件,里面就实现了上述

的逻辑。我一看,感觉问题来了,然后我告诉他,我已经在后台的服务提供了一个方法,用于判断resident的值,他应该在事件触发的时候

,通过异步方式调用后台方法来获取reisdent的值。他一听觉得非常奇怪,这么简单的逻辑,为什么需要使用异步方式去调用后台这么麻烦

,于是我就从设计的角度跟他解释,resident由 a.x与a.b.x决定,这是一个业务逻辑,从职责划分的设计原则分析,前台只负责显示逻辑,

业务逻辑属于后台的职责。接着,他又提出了疑问,说这里其实只有一行代码,没有必要分得那么清,这样调用不但麻烦,而且性能相对较

低。

这个例子,反映出了很多开发人员的通病,没有真正从事物的本质考虑问题,只从代码的数量去考虑。

上述的做法存在两个问题:

1.开发人员没有从职责划分的角度考虑问题,而只是贪图一时的方便,或者觉得简单的一两行代码不需要进行设计,但实际上,上面的问题

必然导致重复代码。上面的例子,即使只有一行代码,但在系统的前后台却出现了两份相同的逻辑,假设以后某天业务逻辑发生了变化,那

么,必须在两个地方(甚至更多的地方)进行修改、测试,如果漏掉了一个地方,就会导致Bug的出现。作为一个开发人员,如果不注意这些

细节,随意的拷贝和重复代码,长期下来,一个系统就会遍布无数的重复代码,系统越往后期就越难维护,最终陷于崩溃。

2.如果作为后台开发人员,看到前台已经实现了这个逻辑,而在前台往后台传递数据的时候,把resident也传递进来,从而认为后台就不需

要重复这段逻辑,直接拿着前台传过来的这个resident来进行业务判断,那么就可能会给系统带来致命的漏洞,因为数据在传输过程中,很

可能被有意或无意的修改,不在后台进行业务规则校验这个错误是常见的,但也是致命的。


有一个原则开发人员特别容易犯,也一定要切记:

 

在Web应用中,相信客户端提交的数据是正确的,不在业务层进行校验,这是致命的错误。

 笔者会在下一篇博文中,详细说明某国内著名的第三方支付平台,因为犯了这个低级的错误,而导致系统出现致命的漏洞,敬请关注。

posted on 2010-04-27 21:43 Johnny.Liang 阅读(1832) 评论(8)  编辑  收藏 所属分类: 系统设计


Feedback

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-04-28 04:08问楼上的人
我觉得这是此消彼长的,一切照搬设计原则,设计出来的系统并不是最完美的,就像设计数据库,最高范式的并不是最好的。他那样做有一个好处就是减少一次向服务端发送请求,这样减轻服务端压力。1000个这样的并发就可以看出他的性能了。所以在设计上,个人觉得允许一些耦合是必要的  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-04-28 09:07Johnny.Liang
@问楼上的人
我非常同意你的看法,设计是需求权衡的,设计是一种选择,我写这篇博文是发现很多开发人员没有这种思想,并不是一刀切的要求都必须按设计方式生搬硬套。谢谢你的意见。  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-04-28 11:54grass
up.up.up.  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病[未登录]2010-04-29 10:30wpskl
我认为前后台验证都是必须的,尤其后台验证最为重要。我一般都是前后台都验证的。  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-04-29 10:42Johnny.Liang
@wpskl
前台验证主要看需求和用户体验,如果希望用户填写信息时出现错误马上得到提醒,就需要做前台校验,不过现在的软件强调以用户为中心(UCD)所以你的做法我是赞同的。  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-04-30 16:37sample
问题的根结是,对于开发人员,应该按照功能模块划分开发任务,而不是以程序模块划分  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-05-01 02:03leekiang
做后端验证不是浪费时间、增加成本吗,在国内是可以不做的。  回复  更多评论

 

# re: 一个非常简单的例子,反映了很多开发人员的通病2010-05-05 22:04webclerk

@leekiang
Web项目不做后台校验的话,就好比一车不带手刹一样,你敢开吗?  回复  更多评论

posted @ 2010-06-15 01:36  心梦帆影  阅读(338)  评论(0编辑  收藏  举报