|
精华帖 (0) :: 良好帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
时间：2008-05-17
armorking 写道 代码1问题多多
这里jdk已经有BigDecimal.ZERO,你这是浪费内存。 private static BigDecimal BIGDECIMAL_ZERO = new BigDecimal("0"); /** * 查询方案金额总和 * * @param lottype * Long * @return BigDecimal */ 方法名不骆驼 public BigDecimal getSummary(Long lottype) { if (lottype == null ){ return BIGDECIMAL_ZERO; } String sql = sql = " select sum(amount) from OccurProj a where a.lottype = " + lottype; List ls = find(sql); &&改为|| if (ls == null && ls.size() == 0) { return BIGDECIMAL_ZERO; } OccurProj occurproj = (OccurProj) ls.get(0); return occurproj.getAmount(); } 你别又误导新人哦 |
| 返回顶楼 | |
|
时间：2008-05-17
哈哈 没必要强调新手。有的老手也犯这样的错误，如果可能加大代码审查力度吧，这些小错误如果有可能可以通过代码审查避免。
|
| 返回顶楼 | |
|
时间：2008-05-17
还是不错的，我看过写几年代码都出现这种问题，甚至是方法名首字母大写的问题
|
| 返回顶楼 | |
|
时间：2008-05-17
mylifestyle1225 写道 回LS的,拿出来晒就是为了请老手指点的.
有事说事,没事少插嘴,不说话没人把你当哑巴.每个人都有菜鸟的过程,你也不是这么过来的么?你是一个新人的时候有胆量晒代码让人指点吗? 典型的中国人的挖苦讽刺,自持清高的心理,不知道你自己达到了什么境界. 是不是这么说一句,很有满足感?那就成就你的满足感吧.但是我到了你这个年龄,肯定能超过你这类货色. 小孩子说大话了不是？ |
| 返回顶楼 | |
|
时间：2008-05-18
抛出异常的爱 写道 ddbird 写道 第3段 sql的拼装能倒着写也蛮强的，肯定copy过来以后看都没看，就看了Eclipse报的编译错误，改了变量名字，就算完事了。
很多人都这么干活，不知道他们的脑子长的是什么。。。。 完成任务呢。 |
| 返回顶楼 | |
|
时间：2008-05-20
String sql = "";
if (sql != null) { sql = " from OccurProj a where a.id=" + id; } String sql = ""; if (username != null && username.length() > 0) { sql += " and a.username like ? "; } if (sql.length() > 1) { sql = " from OccurProj a where 1=1 and a.lottype = '" + lottype + "' " + sql; } else sql = " from OccurProj where lottype = '" + lottype + "' "; 工作有一年,怎么还会这样的代码,有些东东,String sql = ""; if (sql != null) { sql = " from OccurProj a where a.id=" + id; } 他们头脑里面怎么想的,是逻辑不懂呢,还是没有认真去学. |
| 返回顶楼 | |
|
时间：2008-05-20
重点的是这种人怎么找到工作的！
|
| 返回顶楼 | |
|
时间：2008-05-20
不错，至少知道判断空。。。
|
| 返回顶楼 | |
|
时间：2008-05-20
我最后一次回这个帖子，我并不是什么高手，白干几年，除了加班把身体熬差了，没什么大的收获。我只是希望刚刚工作或者是工作n久都不知道怎么写代码的人，好好看看，代码大全，重构这样的书。也算是自己的基本功把。
就像一个网球运动员，如果弯腰低头去拣求，必然观众觉得他不是高手，不专业。 至少应该拿拍子一抄，就把球弄起来了，这样才帅嘛。 |
| 返回顶楼 | |
|
时间：2008-05-20
都够浮躁的；不过写这样代码的人本身态度有点问题，不只是能力的问题。
只是新手的话没必要BS的，都是从这个阶段过来的（其实我现在还是新手，做2年java了，才发现刚入门。。这里很多人也算不上多老手） |
| 返回顶楼 | |
