“过度设计”之真实例子

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

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

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

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

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

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

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

评论

30 楼 csc365kl 2010-09-29  

公司给你们 过度设计 的成本 何乐而不为

29 楼 jasph77 2010-09-29  

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

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

28 楼 lpn520 2010-09-29  

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

27 楼 ggpp224 2010-09-29  

楼主可能心态问题吧，我觉得不能单单的看过度设计之说，再说即使过度设计总比不设计要好的多。之于说耦合高了，那只能说是设计的缺陷，分层本身的一个目的就是实现“松耦合高内聚”。之于设计要看你项目的大小，准备运行的时间，参于人数的多少来说。

26 楼 qiren83 2010-09-29  

lpn520 写道

nothink 写道

没代码没真相。

兄弟，这个算不算真相







实际的业务代码只有这些

没看出过分设计呀 不过项目挺小的吧

25 楼 lpn520 2010-09-29  

xiaoming_320 写道

呵呵 这种公司管理上应该是很混乱 团队之间缺乏默契。表达能力和人际关系起决定作用。只要能说，就能得到领导的赏识，你说的那个设计构架的，应该是学历高，而且特能说的人吧

学历应该蛮高的，可能是硕士，技术应该也蛮牛的，就是不会往简单的想

24 楼 lpn520 2010-09-29  

nothink 写道

没代码没真相。

兄弟，这个算不算真相

23 楼 xiaoming_320 2010-09-29  

呵呵 这种公司管理上应该是很混乱 团队之间缺乏默契。表达能力和人际关系起决定作用。只要能说，就能得到领导的赏识，你说的那个设计构架的，应该是学历高，而且特能说的人吧

22 楼 IcedCoffee 2010-09-29  

国内一般没有过度设计这回事，一般就是某个架构师总是使用着自己的一套思想..
所有项目都用同一套架构...导致了一个简单的项目搞的负责了而已..
国内还没有人能做出过度设计的东西出来..

21 楼 itcrown2005 2010-09-29  

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

20 楼 wangking717 2010-09-29  

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

19 楼 nothink 2010-09-29  

没代码没真相。

18 楼 lpn520 2010-09-29  

kakaluyi 写道

lpn520 写道

kakaluyi 写道

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

真的是被架构师瞎设计，他写的东西，完全就是把一个人的开发能力死死的套住，自己又不去写业务代码，然后把自己当专家一样，再拿什么规范来压你（认为自己就是规范）。
像ExtJs，他也要封装一层，我们平时写Ext好好的，现在又一定要按他的封装的方式来写，我真的很无语了。。。

EXT已经封装的够多了，还封装一层。。。
在你们公司干下去，你们也和你们公司耦合了
估计出去干不了什么事了

现在做的最多的就是天天调试他的架构，问题超多，因为不是原型开发，只要他的架构有问题，整个项目就有问题 Orz...

17 楼 kakaluyi 2010-09-29  

lpn520 写道

kakaluyi 写道

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

真的是被架构师瞎设计，他写的东西，完全就是把一个人的开发能力死死的套住，自己又不去写业务代码，然后把自己当专家一样，再拿什么规范来压你（认为自己就是规范）。
像ExtJs，他也要封装一层，我们平时写Ext好好的，现在又一定要按他的封装的方式来写，我真的很无语了。。。

EXT已经封装的够多了，还封装一层。。。
在你们公司干下去，你们也和你们公司耦合了
估计出去干不了什么事了

16 楼 LSQ6063 2010-09-29  

架构本来就没有十全十美，合适就行，是需要根据具体的业务场景，应用场景等各种不同场景来综合考虑的，我也看过很多架构师本来就很简单的动不动就架着一个高射炮打蚊子，一个开发可以完成的，最后搞十个开发进行，累的要死要活。。。杯具

15 楼 bonny 2010-09-29  

Durian 写道

这事儿遇到过，一个大博士做的设计，繁冗的厉害。
这个架构师的表达能力还很强，理论很深，所以，基本上把最简单的问题用当今最先进高深的技术实现了。不复杂不足以表达他的雄心壮志。

架构设计应该和问题域密切相关，抛开问题谈设计无异于闭着眼睛瞄准。许多初级“架构师”仅仅从技术上，或者从表达自己想法层面上设计架构，不是愚蠢，而是因为经验欠缺或者是自视甚高从不参与具体项目开发和代码编写所致。

架构师必须了解业务，必须参与代码编写，必须考虑程序员和业务人员日常碰到的问题和烦恼。

14 楼 Durian 2010-09-29  

徐风子 写道

设计：一定要写过代码的人做。

----------
顶你这话。

13 楼 lpn520 2010-09-29  

高中生 写道

lpn520 写道

kakaluyi 写道

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

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

我觉得既然是多个层，那么层与层之间应该是单向关联。如果多个层同时依赖一个东西，那就是分层的问题。

架构师的设想就是让更多的代码共用起来

12 楼 徐风子 2010-09-29  

设计：一定要写过代码的人做。

11 楼 高中生 2010-09-29  

lpn520 写道

kakaluyi 写道

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

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

我觉得既然是多个层，那么层与层之间应该是单向关联。如果多个层同时依赖一个东西，那就是分层的问题。