让人头疼的新手

刚进公司没多久时，领导让我带两个新人(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));
	}

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

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


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

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