“过度设计”之真实例子

　　我刚到了一家新公司，公司给我的感觉很不错，不过当开始做第一个项目时便有过度设计的嫌疑，项目不大，基本就实现CURD的功能，用struts2+spring+ibatis+extjs。拿我开发的一个简单的功能来讲，就花了大概一周，如果采用简化的技术，实际上可能只需要一两天。

　　设计太多的分层，以及偶和性太高，添加或修改一个模块太困难了，而且还不知道会不会影响到其它模块。按照项目定义的规范做法，写一个Hello world，创建的代码文件个数必须达到8个！！！！

　　过度分层已经成为过度设计的一个典型。项目经理说，这是另一个项目的架构直接搬过来用的，我们做一下架构上的优化。

　　其实本来就没有什么优化，原来本就是简单的东西，为了显示技术，搞得特复杂。现在所谓的优化，实际上回归质朴而已。

　　记得OSI7层模型就有过度设计嫌疑，可以去看一下现在的教材，都没有用OSI作为例子，尤其没有专门讨论会话层。

　　如何防止过度设计，最好办法就是使用敏捷编程，他的思路就是刚刚好就行，如果有问题，再重构。另外我自己目前的实践方法就是多编程，少设计，好像也能避免。因为当你的任务主要是写代码的时候，你绝不会去写8个类来完成一个helloworld。

　　其实，我所说的就是开发原型，也是《敏捷开发方法》里提到过的方法。

评论

50 楼 IcedCoffee 2010-09-29  

lpn520 写道

jasph77 写道

构架师 自然有架构师考虑。。
要不然人家怎么坐上这个位置，你还是一个普通的程序员。
架构师也是个人，他的架构必然也有不完美地方。

最讨厌那种，整天说这个不好、那个行，自己又没解决方案。

给你看看我之前做的解决方案，同样“添加删除修改查询”，同样是Struts2+Spring+iBATIS+ExtJs

前端
helloWorld.jsp       //用过Ext的都知道，页面是不需要写HTML代码的，所以helloWorld.js直接可以省掉，直接把js写在jsp文件里，还有用EXT直接使用原生态的，封装一层只的很恶心

业务层
HelloWorldAction.java     //struts2的Action，本身就是动作的意思，为什么它不叫Controllor呢，因为控制类已经在它内部封装好了，你实现了Action接口，只要在类里写业务就可以了，再加上struts2每个请求可以对应到Action的一个个方法上，所以万恶的HelloWorldServer.java和HelloWorldServerImp.java可以去掉了

DAO层
再讨论一下DAO层，我们用了ibatis框架，他本身就是个DAO层的实现，你每个SQL调起来就像调用DAO里的每个方法一样，所以HelloWordDao.java就可以去掉了，像HelloWordDao.java里的每个方法就一条代码，有意思吗
所以只要下面两个文件就可以了
HelloWord.java          //HelloWord表的ORM的对象
HelloWord.xml           //HelloWord表操作的SQL语句，ibatis的sqlmap

无语了..那干脆dao也去掉，放进action算了...

49 楼 chenlixun 2010-09-29  

xhdwell 写道

jasph77 写道

构架师 自然有架构师考虑。。
要不然人家怎么坐上这个位置，你还是一个普通的程序员。
架构师也是个人，他的架构必然也有不完美地方。

最讨厌那种，整天说这个不好、那个行，自己又没解决方案。

确实，以前javaeye上有篇文章说的就是别老骂别人的代码烂，说的很在理。在你没完全理解别人设计的思路之前千万不要妄下结论。别人这么设计是有他的道理的，只是有些原因你还不能理解。你在没有完全理解前就说他过度设计只能说你学的还不够。再过2年你就不会这么说了。

同意楼上的, 只要不是像楼主说的紧耦合, 合理的分层是必要的, 也许今天理解不了, 不代表人家的的设计没道理.

48 楼 bonny 2010-09-29  

呵呵 这个8个文件的设计是比较正常的老式的设计。最近的设计一般都会简化。

helloworld.js这个文件可省，可以写在jsp中，这样减少很多文件，减少很多代码查找和切换快捷键，也减少了http请求数。

service接口可省，直接用实现类即可。这和面向接口设计比较违背，不过多数情况下的确是个累赘。
serviceimpl多数情况下（增删改查）也可以省略掉，action直接调用dao即可。除非有比较复杂的业务操作和多dao关联。


dao可以用泛型简化通用操作，并用代码生成器生成。




这样看来，仅仅需要些的就是jsp  action （serviceimpl） 代码已经非常简化了。

47 楼 lpn520 2010-09-29  

jasph77 写道

构架师 自然有架构师考虑。。
要不然人家怎么坐上这个位置，你还是一个普通的程序员。
架构师也是个人，他的架构必然也有不完美地方。

最讨厌那种，整天说这个不好、那个行，自己又没解决方案。

给你看看我之前做的解决方案，同样“添加删除修改查询”，同样是Struts2+Spring+iBATIS+ExtJs

前端
helloWorld.jsp       //用过Ext的都知道，页面是不需要写HTML代码的，所以helloWorld.js直接可以省掉，直接把js写在jsp文件里，还有用EXT直接使用原生态的，封装一层只的很恶心

业务层
HelloWorldAction.java     //struts2的Action，本身就是动作的意思，为什么它不叫Controllor呢，因为控制类已经在它内部封装好了，你实现了Action接口，只要在类里写业务就可以了，再加上struts2每个请求可以对应到Action的一个个方法上，所以万恶的HelloWorldServer.java和HelloWorldServerImp.java可以去掉了

DAO层
再讨论一下DAO层，我们用了ibatis框架，他本身就是个DAO层的实现，你每个SQL调起来就像调用DAO里的每个方法一样，所以HelloWordDao.java就可以去掉了，像HelloWordDao.java里的每个方法就一条代码，有意思吗
所以只要下面两个文件就可以了
HelloWord.java          //HelloWord表的ORM的对象
HelloWord.xml           //HelloWord表操作的SQL语句，ibatis的sqlmap

46 楼 Crusader 2010-09-29  

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWorld.jsp                //页面文件，不用说了
helloWorld.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWorldAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWorldService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWorldServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWorldDao.java            //数据操作类，关于HelloWorld业务的数据操作都在这里
HelloWorld.java               //HelloWorld表的ORM的对象
HelloWorld.xml                //HelloWorld表操作的SQL语句，ibatis的sqlmap

看了这个分层，我怎么觉得是你的想法有问题

45 楼 xhdwell 2010-09-29  

jasph77 写道

构架师 自然有架构师考虑。。
要不然人家怎么坐上这个位置，你还是一个普通的程序员。
架构师也是个人，他的架构必然也有不完美地方。

最讨厌那种，整天说这个不好、那个行，自己又没解决方案。

确实，以前javaeye上有篇文章说的就是别老骂别人的代码烂，说的很在理。在你没完全理解别人设计的思路之前千万不要妄下结论。别人这么设计是有他的道理的，只是有些原因你还不能理解。你在没有完全理解前就说他过度设计只能说你学的还不够。再过2年你就不会这么说了。

44 楼 navy0168 2010-09-29  

rocketball 写道

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

这不是很正常的嘛。

+1

43 楼 IcedCoffee 2010-09-29  

ggpp224 写道

传说中的应届毕业生！

42 楼 WWQQ 2010-09-29  

我们的还好 后台CTO一人搞定 ， 你想要什么数据 他用JSONArray 返回给你， 然后页面的设计 数据的填充 效果的展示 想怎么弄就怎么弄。 玩转JS 呵呵

41 楼 ggpp224 2010-09-29  

传说中的应届毕业生！

40 楼 rocketball 2010-09-29  

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

这不是很正常的嘛。

39 楼 xiaoming_320 2010-09-29  

这样的分层一般应该是针对一个功能模块或业务模块，而不是针对末个类或文件的。

38 楼 ggpp224 2010-09-29  

jasph77 写道

构架师 自然有架构师考虑。。
要不然人家怎么坐上这个位置，你还是一个普通的程序员。
架构师也是个人，他的架构必然也有不完美地方。

最讨厌那种，整天说这个不好、那个行，自己又没解决方案。

很合理啊，看不出来什么问题。

37 楼 rocketball 2010-09-29  

itcrown2005 写道

这算什么，我见过一个浙大毕业的所谓的架构师，设计了一套架构，直接把一个上亿的项目搞死了，服务端好久都起不来，客户端15分钟才起来，还每天嚣张的吹他的架构

可不可以不加上浙大两个字，浙大每年毕业几千人，总有菜的

36 楼 xhdwell 2010-09-29  

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

看了这个结构觉得已经很简单了啊。是最正常的分层啊。兄弟，建议你多学点设计思想。就不会问这样的问题了。

35 楼 lhlpyl 2010-09-29  

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

这样的设计中小型项目应该不会有什么问题啊 ！

34 楼 skycray 2010-09-29  

IcedCoffee 写道

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

这个真的很正常...

+1.

33 楼 xhdwell 2010-09-29  

lpn520 写道

kakaluyi 写道

过度设计导致耦合性太高？
那就是设计的有问题了，真正的设计，包括工厂方法，策略，门面模式等等都是为了降低耦合，修改一个模块只要更改很小一部分代码或者配置就好了，你这种情况属于瞎设计

设计多个层，每个层都想着某个功能能共用，也就是说一个业务，经常给多个模块共用，本来就很简单的模块，就是添加删除修改查询的东西，明明可以独立开写的，现在搞得每个模块都融合在一起，修改了一个业务，都不知道会影响哪个模块，你说这算不算耦合性高啊？

估计都是些纸上谈兵的人在瞎设计。用着面向过程的思想在做面向对象的事。

32 楼 xiaoming_320 2010-09-29  

IcedCoffee 写道

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

这个真的很正常...

恩 我也觉的很正常。

31 楼 IcedCoffee 2010-09-29  

lpn520 写道

wangking717 写道

真想看看那个HELLOWORD分8个的代码，让我见识一下。

让你见识一下传说中的8个文件：

前端
helloWord.jsp                //页面文件，不用说了
helloWord.js                 //javascript文件

Struts的Action，就叫动作层吧
HelloWordAction.java         //Action类文件，客户端请求到这个类上

业务层
HelloWordService.java        //业务接口， 还用了一套传说中的“面向接口编程”
HelloWordServiceImp.java     //业务实现类，实现上面定义的接口

DAO层
HelloWordDao.java            //数据操作类，关于HelloWord业务的数据操作都在这里
HelloWord.java               //HelloWord表的ORM的对象
HelloWord.xml                //HelloWord表操作的SQL语句，ibatis的sqlmap

这个真的很正常...