让人头疼的新手

刚进公司没多久时，领导让我带两个新人(07年7月份毕业的)。他们两个是试用期3个月都过了之后才参与到我们现在的项目中来的，算起来他们也工作了快一年了，可是今天无意中修改一个他们写过的一个类文件时，看到他们写的一些代码，让我又好气又好笑。可能也是我以前只注意给他们测试功能了，没怎么注意代码方面的问题吧！

晒晒他们的部分代码！

/**
	 * 查询方案金额总和
	 * 
	 * @param lottype
	 *            Long
	 * @return double
	 */
	public Double getsummation(Long lottype)
	{
		double sum = 0.00;
		List ls = null;
		String sql = "";
		if (sql != null)
		{
			sql = " select sum(amount) from OccurProj a where a.lottype = '" + lottype + "' ";
		}
		ls = find(sql);
		if (ls != null && ls.isEmpty())
		{
			for (int i = 0; i < ls.size(); i++)
			{
				OccurProj occurproj = (OccurProj) ls.get(i);
				sum = sum + occurproj.getAmount();
			}
		}
		return sum;
	}

/**
	 * 根据主键查询明细信息
	 * 
	 * @param occurproj
	 *            OccurProj
	 * @return String
	 */
	public List queryById(Long id)
	{
		String sql = "";
		if (sql != null)
		{
			sql = " from OccurProj a where a.id=" + id;
		}
		return translateCode2Str(find(sql));
	}

/**
	 * 参数查询
	 * 
	 * @param occurproj
	 *            OccurProj
	 * @param lottype
	 *            Long
	 * @return List
	 */
	public List queryByPara(String username, Long lottype)
	{
		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 + "' ";
		return translateCode2Str(find(sql));
	}

我现在也体会到当初我找工作的时候为什么那么难了，同时也提醒一下即将毕业的朋友，想找到不错的工作，你的工作态度确实得端正！

以前没带过新人，大家也介绍一下，带新人的经验吧！


我需要说明一下，我发贴的目的是想提醒一下临近毕业的朋友，不是公司不需要应届毕业生，应届毕业生有优点也有缺点。但是你进入公司以后，自己的发展和进步还是要靠自己的努力。工作了半年还能写出这样令人佩服的代码，那就只能是你自己的问题了，当然也有我自己的责任，毕竟是我带他们。

有则改之，无则加勉，别无他意，仅此而已！

评论

159 楼 movomoto 2008-08-29  

没有逻辑。。明显粘贴复制忘记改了

158 楼 kill 2008-08-29  

[align=left]人,夸夸其谈,人,他们只会把自己的才能展现出来,分享给别人,造福IT行业,不管是新手还是有多年经验的!带新人,是最快乐的事,"想一想以后行业里面又多了个高手,那我谁带出来的,谁这么厉害呢?"

[/align]

157 楼 kimmking 2008-08-13  

ql = " from OccurProj a where a.id=" + id;

>>> id = 1 and 1=1

156 楼 kimmking 2008-08-13  

sql = " select sum(amount) from OccurProj a where a.lottype = '" + lottype + "' ";  

>>>   lottype = 1’ and '1' = '1

155 楼 javaxy 2008-08-13  

这样的新人，为什么会转正，当初怎么过试用期的！这才是应该反思的！

154 楼 daoger 2008-08-12  

moneyworship 写道

太恐怖了吧~~这..这..太刺激了~~

???

153 楼 moneyworship 2008-08-10  

太恐怖了吧~~这..这..太刺激了~~

152 楼 wei841118z 2008-07-23  

你们都是强人，在笑话别人的时候请想一下你们刚出来的时候是什么样子

151 楼 yongfan_420 2008-07-18  

   我觉得每个人在写代码的时候都会在某些点上都会认为这样是没问题的。例如
   ArrayList arr = new ArrayList();
   ArrayList arr1 = new ArrayList(10);
   它两者到底有多大区别，效率和空间到底差多少，可能有待于研究。
    所以要不断的学习别人写得代码，也许现在的一代不注重这个不是更多了，而是怎么样交活。

150 楼 xiejiangbo 2008-07-18  

应该用stringbuffer

149 楼 zhaowei_520 2008-07-09  

JAVA基础太差了!

148 楼 shishi11 2008-06-25  

我们也遇到过这样的问题，想想解决的办法：
一，起步的时候，一定要有模板，当然如果能采用如继承的方式更好，像SS3那样，SCUD都继承。当然，于他们的提高有限，但可以提高代码质量，也有利于开发。
二，这也是经验问题，比如“魔术数字”问题，新手不太明白，得让他知道好处。LJ的例子里也有，为什么不写成'xxx='+id的方式，因为有种可能性被SQL注入。得让新手明白为什么。当然这个工作应该新手自己完成，但本着带新人的原则，应该适当指导一下。
三，工具，人都不可靠，机器好些。我最近在研究checkstyle，由机器保证代码。
四，代码生成，也见过类似的东东，虽然有些灵活性上的问题，但一般普通地方还是可以的，新手就做点简单的工作吧。

当然，我碰到过超XX的，让她看一下别人的代码写自己的，结果连包名类名都一样，把别人的给覆盖了，她自己的过了，别人不灵了还不知为什么。

147 楼 shishi11 2008-06-25  

我们也遇到过这样的问题，想想解决的办法：
一，起步的时候，一定要有模板，当然如果能采用如继承的方式更好，像SS3那样，SCUD都继承。当然，于他们的提高有限，但可以提高代码质量，也有利于开发。
二，这也是经验问题，比如“魔术数字”问题，新手不太明白，得让他知道好处。LJ的例子里也有，为什么不写成'xxx='+id的方式，因为有种可能性被SQL注入。得让新手明白为什么。当然这个工作应该新手自己完成，但本着带新人的原则，应该适当指导一下。
三，工具，人都不可靠，机器好些。我最近在研究checkstyle，由机器保证代码。
四，代码生成，也见过类似的东东，虽然有些灵活性上的问题，但一般普通地方还是可以的，新手就做点简单的工作吧。

当然，我碰到过超XX的，让她看一下别人的代码写自己的，结果连包名类名都一样，把别人的给覆盖了，她自己的过了，别人不灵了还不知为什么。

146 楼 open2ye 2008-06-25  

mylifestyle1225 写道

armorking 写道

mylifestyle1225 写道

不过还是谢谢指点,我会改成英文的.然后把名字改得尽量意义明确些.

如果不是对美外包的项目的话，为什么非要用英语写注释

恩，是不是对美外包的,这是个金融报文的解析和生成,所以自然没什么SQL,JDBC,SPRING等东东了.
有道理,注释的作用本来就是为了让人看懂.

你觉得以下怎么样?


// وقع الكتاب المقدس

145 楼 jian'shang 2008-06-07  

我觉得那两个人应该是受了误导，可能在试用期间看了公司的“规范代码”，他也许是想验证传入的参数，但又没加异常处理，呵呵，这事只能怨公司

144 楼 sanfeng_chow 2008-05-30  

终于看完了。受益颇深。
07年毕业的人还不少哦。呵呵。

143 楼 dongle2001 2008-05-27  

我也经常写这样的代码。

工作两年多了，最近才发现，这么写法真的是害人害己。

142 楼 CherryRemind 2008-05-23  

我们TEAM里面也这样的同事, 受不了, 尽做些不必要的判断.

<代码大全2>一本介绍CODE STYLE的书, 对新手很有帮助

141 楼 tjxjj 2008-05-20  

唉.看了下,自己代码也没新手高多少

只不过不会出现逻辑错误而已

140 楼 muffings 2008-05-20  

daoger 写道

刚进公司没多久时，领导让我带两个新人(07年7月份毕业的)。他们两个是试用期3个月都过了之后才参与到我们现在的项目中来的，算起来他们也工作了快一年了，可是今天无意中修改一个他们写过的一个类文件时，看到他们写的一些代码，让我又好气又好笑。可能也是我以前只注意给他们测试功能了，没怎么注意代码方面的问题吧！

晒晒他们的部分代码！


我现在也体会到当初我找工作的时候为什么那么难了，同时也提醒一下即将毕业的朋友，想找到不错的工作，你的工作态度确实得端正！

以前没带过新人，大家也介绍一下，带新人的经验吧！


我需要说明一下，我发贴的目的是想提醒一下临近毕业的朋友，不是公司不需要应届毕业生，应届毕业生有优点也有缺点。但是你进入公司以后，自己的发展和进步还是要靠自己的努力。工作了半年还能写出这样令人佩服的代码，那就只能是你自己的问题了，当然也有我自己的责任，毕竟是我带他们。

有则改之，无则加勉，别无他意，仅此而已！

只能说明两点：
1. 你们没有能力培养起程序员
2. 你们的项目管理混乱

在我们项目组，新人的招收是很严格的，但是招收进来， 就会给予很多的培养和指导。 一个新人，前几个月的试用，会有专人给予全方位的指导，如果有代码风格，代码质量的问题，很快就能够发现，并给他指出来， 新人会很快能够改进。 不至于会有人说， 某某写的代码有问题一年以后才发现。

难道你们没有code review 机制？ 我们的项目中，每一行代码的提交，都会有code review， 这种可笑的代码质量问题很难在cvs 代码中找到。 每个project的build 出去以后会做design 的review， 提出design的改进方法， review 的过程， 也是对整个团队提高的过程。

建议楼主也从自己项目组的管理上找找问题。