维护有感 -- 代码恶习千奇百怪
xfei6868
这个作者很懒,什么都没留下…
展开
-
维护有感1--字符串拼接老问题
上个月拿自己的一份大学时候的代码报名javaeye的招聘,但是被robbin指出了string直接相加的毛病。当然现在我一直都不是用直接用+号了,大学的时候没有考虑这么多问题。最近在维护一个项目,修改里面的功能和代码,发现问题多多,千奇百怪。 以后每天写出点供自己警惕,同时有没发现其“精髓”的地方大家讨论。 其中就象我以前一样 的错误 sql 语句用 + 号 大行其道 [code...2009-12-03 14:15:15 · 80 阅读 · 0 评论 -
维护有感2--pl/sql这样写有点繁
先贴出来这样一段代码 [code="oracle:pl/sql"] SELECT decode(length(substr(max(to_number(CGBH)), 5) + 1), null, null, 1, ...2009-12-04 09:18:47 · 77 阅读 · 0 评论 -
维护有感3 -- 被滥用的synchronized
同样要先看一段代码,这是一段方法内部的代码: [code="java"] StringBuffer temp = new StringBuffer(64); synchronized (temp) { String cgbh = Decoder.getParameter(request, "cgbh"); temp.append("INSERT INTO LWZZW...2009-12-07 09:19:58 · 131 阅读 · 0 评论 -
维护有感4 — try 里的代码不需要这么多
写数据库代码时候经常会用到try catch 。 但是一下子都把所有的东西写在try里,很显然在错误定位的时候, 很难发现真正错误的原因,而且性能上来说也不是个很好的表率。 看看下面这个: [code="java"] Transaction trans = db.getTransaction(); try { trans.begin(); String s_cgb...2009-12-09 10:50:54 · 147 阅读 · 0 评论 -
struts1 acton 中大量的 dao实例变量
项目是struts1.x的项目,而在action中有大量的类似代码: [code="java"] public class XXXAction extends DispatchAction { private XXXManager manager = null public XXXAction() { manager = new X...2009-12-25 13:22:53 · 114 阅读 · 0 评论