糟糕的代码设计真的让人很心烦..

刚进公司不到一个月, 现在正在进行一个项目开发,项目里使用 Hibernate2 + Struts1.2,正常情况下这是一个相当好的组

 

合,但是现在由于DAO层代码几乎全由MyExclipse 生成,不知是否由于MyExclipse 版本过低,生成的代码中大量使用了继

 

承,而且最主要的,项目管理人几乎没有进行重写的想法,他们的意思是:这些我们都用了很久,都很熟悉了;但是项目项目进度的

 

发展,这些问题也越来越突出,由于采用MyEclipse 自动生成的代码,在调用DAO对象前要先执行初始化,如:

try {
	_BaseRootDAO.initialize();
	dao = XXXDAO.getInstance();
} catch (HibernateException e) {
	e.printStackTrace();
}

 

然后才能调用,我想对它做个封装,可是对它这个初始化实在束手无测,现在我只能简单的把这些业务单独做个类来封闭,让它

 

不与WEB层混淆在一起,但是这样并不能彻底解决问题,我封闭的这些类中重复的代码实在太多,我也试过对这些代码进行重

 

构,可是由于现在项目也经进行到一半了,项目开发由多个人一同进行,我如果更改了原始代码结构,必然会引起别人代码不能

 

正常运行,可是每次自己读到这些代码,特别是自己还要参与到其中,真的很心烦.....

 

我应该如何做呢,难道企业里开发都这样吗? 大家能否给点建议..

评论

15 楼 wcleye 2008-08-12  

leonhe_cn 写道

公司规模太小了吧，我上次到新公司报道，一看项目和楼主一样连spring都没用，马上走人了呵呵

spring 不是每个公司都用吧,有些公司招聘上要求要会这个,可真去了一年半载用不到是常事,再说我现在的公司规模也不算小了,只不过业务很单一,都做些公安的系统,真正应了前人铺路后人乘凉,连点改进意识都没了..

14 楼 lucumu 2008-08-12  

<div class='quote_title'>liuqiang 写道</div>
<div class='quote_div'>
<div class='quote_title'>leonhe_cn 写道</div>
<div class='quote_div'>公司规模太小了吧，我上次到新公司报道，一看项目和楼主一样连spring都没用，马上走人了呵呵</div>
<p> </p>
<p> 憨，你也不看看用没用EJB，会EJB的现在可是个奥运金蛋</p>
<p> </p>
<p>我上次到接触一个新公司，一看项目用spring，马上走人了，呵呵</p>
</div>
<p>奥运会金蛋??</p>

13 楼 liuqiang 2008-08-12  

<div class='quote_title'>leonhe_cn 写道</div>
<div class='quote_div'>公司规模太小了吧，我上次到新公司报道，一看项目和楼主一样连spring都没用，马上走人了呵呵</div>
<p> </p>
<p> 憨，你也不看看用没用EJB，会EJB的现在可是个奥运金蛋</p>
<p> </p>
<p>我上次到接触一个新公司，一看项目用spring，马上走人了，呵呵</p>

12 楼 leonhe_cn 2008-08-12  

公司规模太小了吧，我上次到新公司报道，一看项目和楼主一样连spring都没用，马上走人了呵呵

11 楼 supttkl 2008-08-12  

那就别自动生成，手写算了。

10 楼 Else 2008-08-12  

最好不要动，卖力不讨好

9 楼 wcleye 2008-08-12  

daquan198163 写道

看来面试的时候，很有必要提出观摩该公司代码

刚进公司时谁知道呢,公司规模也不算小了,就是开发人员写的代码太差劲..

8 楼 wcleye 2008-08-12  

<div class='quote_title'>抛出异常的爱 写道</div>
<div class='quote_div'>
<p>AOP前装配</p>
</div>
<p><br/>很精辟,但是公司似乎没人会用Spring .... </p>

7 楼 抛出异常的爱 2008-08-12  

<div class='quote_title'>wcleye 写道</div>
<div class='quote_div'>
<p>刚进公司不到一个月, 现在正在进行一个项目开发,项目里使用 Hibernate2 + Struts1.2,正常情况下这是一个相当好的组</p>
<p> </p>
<p>合,但是现在由于DAO层代码几乎全由MyExclipse 生成,不知是否由于MyExclipse 版本过低,生成的代码中大量使用了继</p>
<p> </p>
<p>承,而且最主要的,项目管理人几乎没有进行重写的想法,他们的意思是:这些我们都用了很久,都很熟悉了;但是项目项目进度的</p>
<p> </p>
<p>发展,这些问题也越来越突出,由于采用MyEclipse 自动生成的代码,在调用DAO对象前要先执行初始化,如:</p>
<pre name='code' class='java'>try {
_BaseRootDAO.initialize();
dao = XXXDAO.getInstance();
} catch (HibernateException e) {
e.printStackTrace();
}
</pre>
<p> </p>
<p>然后才能调用,<span style='color: #ff0000;'>我想对它做个封装</span>,可是对它这个初始化实在束手无测,现在我只能简单的把这些业务单独做个类来封闭,让它</p>
<p> </p>
<p>不与WEB层混淆在一起,但是这样并不能彻底解决问题,我封闭的这些类中重复的代码实在太多,我也试过对这些代码进行重</p>
<p> </p>
<p>构,可是由于现在项目也经进行到一半了,项目开发由多个人一同进行,我如果更改了原始代码结构,必然会引起别人代码不能</p>
<p> </p>
<p>正常运行,可是每次自己读到这些代码,特别是自己还要参与到其中,真的很心烦.....</p>
<p> </p>
<p>我应该如何做呢,难道企业里开发都这样吗? 大家能否给点建议..</p>
</div>
<p>AOP前装配</p>

6 楼 Joo 2008-08-11  

呵呵 主考官对比你的代码和他们本公司代码 发现不是一个档次 心想你小子来了岂不是又是一更新派 干脆不要了 于是你被告知你写的代码太烂跟他们公司人员的代码不是一个档次...

5 楼 daquan198163 2008-08-11  

看来面试的时候，很有必要提出观摩该公司代码

4 楼 sg552 2008-08-11  

继续撞钟…… 呵呵。。。。liuqiang很逗啊。

LZ,我认为你要重构他们原有系统的代码，首先要保证有单元测试。

如果他们原来的程序员没有写单元测试的话，你就应该比较慎重。因为你写的不一定全面，而且这个工作应该本应由原来的程序员来完成的。

如果要进行某些封装，建议你多看看《设计模式》，相信你会找到合适的。

3 楼 liuqiang 2008-08-11  

<div class='quote_title'>wcleye 写道</div>
<div class='quote_div'>
<div class='quote_title'>liuqiang 写道</div>
<div class='quote_div'>项目经理都不急，你急什么？ <br/>这个现象很普遍啊，不要太在意，建议继续撞钟，机会来了跳到开发规范点的公司</div>
<br/><br/>当一天和尚撞一天钟吗?如果我去了下个公司还是这样呢?然后继续跳吗?</div>
<p>很多事情是你一个人改变不了的，是你去适应公司，而不是公司适应你，下次找工作时最好调研一下该公司的各个方面，比如说印客网就不会有你说的那些情况，因为人家的人才梯队做的比较好。当然如果下次你还是那么随意找个工作的话，撞不撞钟只有听天由命了……</p>
<p> </p>

2 楼 wcleye 2008-08-11  

liuqiang 写道

项目经理都不急，你急什么？
这个现象很普遍啊，不要太在意，建议继续撞钟，机会来了跳到开发规范点的公司

当一天和尚撞一天钟吗?如果我去了下个公司还是这样呢?然后继续跳吗?

1 楼 liuqiang 2008-08-11  

项目经理都不急，你急什么？
这个现象很普遍啊，不要太在意，建议继续撞钟，机会来了跳到开发规范点的公司