让人头疼的新手

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

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

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


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

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

评论

99 楼 yujianqiu 2008-05-14  

我觉得应该和他谈谈，深入了解一下他这样写的动机，再给他分析一下这样做为什么是不好的。尽管某些新人的水平差，可是既然带了就要努力去把人家带好。

98 楼 luyunjian 2008-05-14  

armorking 写道

代码1问题多多

     public Double getsummation(Long lottype)
    {
        //1、既然是算钱，就应该用BigDecimal
        //2、sum和ls变量不需要这么早声明
        double sum = 0.00;
         List ls = null;

        String sql = "";

        //3、这个条件是无效（永真）的
        if (sql != null)
        {
            //4、既然lottype的类型是Long，那么在SQL语句中作为参数的时候，就不应该括上单引号
            sql = " select sum(amount) from OccurProj a where a.lottype = '" + lottype + "' ";
        }

借人气，想问一下，null==sql 和 sql==null 有怎样的区别？

97 楼 hotjava 2008-05-14  

vicksong 写道
能满足客户功能的代码就是好代码

－－－－－－－－－－－－－－－－－－－－－－
4，5年以前我也信奉过这样的话，当时我还说过，我啥都不懂，就是能实现功能，这就足够了。 其实这就象一个医生告诉你，反正这药吃不死人，但是说不定有啥副作用一样。

96 楼 hotjava 2008-05-14  

怎么说呢，我想给你提个醒，我以前也是忙的四脚朝天的，根本没有时间给开发人员做review，但是当项目上线的时候，你会发现问题接踵而至，性能问题，OOM问题，heapdump，到时候忙死你，整天各种日志看死你，我盯着gc,javacore,dump看了半个月，发现有人拼sql的时候， 一个if语句判断的不严密，一个错误数据进来，整个服务就崩溃了，所以还是开发阶段做好review吧。

95 楼 hotjava 2008-05-14  

至少sql = "XXXXX"+id 会经过语法分析，语意分析，貌似还有授权分析，反正就是降低sql的执行效率了。

94 楼 hotjava 2008-05-14  

sql = "XXXXX"+id

回降低缓存的命中率，导致硬解析过多，还可能大量占用缓存，造成IO过多，还可能导致CPUwait比较多，当然是不是由于他导致wait太多，我还没确认。请大家指点一下。

93 楼 yuan 2008-05-14  

土匪一份子 写道

楼主   坦白说：你真的很不负责任。。。。。。
我也是07年毕业的   看来我很幸运  现在看来以前带我的那个组长真的不错，刚开始做项目时，每完成一个模块他都会抽一点时间看我的代码，然后指导哪里的代码不规范，哪里的代码应该怎么来写效率更高，刚开始修改的地方还是有些，慢慢的，修改的越来越少，后来有闲时直接是让我看我自己以前写的代码，让我重构自己的代码，再后来是让我重构别人的代码。。。。。。   

厦门哪家公司？有这么好的待遇？大公司吧？羡煞我也……

92 楼 java9981 2008-05-14  

qsrock 写道

林杰杰 写道

抛出异常的爱 写道

刑天战士 写道

#         String sql = ""; 
#         if (sql != null) 

哈哈哈哈哈哈哈……

lottype ！＝null

if (username.length() > 1) 

这种代码改时间长了自然会明白他想写什么
不过很恶心用
sql = "XXXXX"+id
或这类的其它写法。

不是很懂java，想向抛抛请教几个问题：
1. sql = "XXXX" + id;为什么恶心？
是怕SQL注入还是String相加时候引起的性能问题？

2. if (username.length() > 1)
这有不对吗？

一般都不会这样写的，比如你可以用PreparedStatement.
username!=""就可以直接代替username.length() > 1,为什么要写那么麻烦呢

java中不能这要比较，相比!username.equals("")，也没什么问题啊，虽然两种看起来都不爽

91 楼 qsrock 2008-05-14  

林杰杰 写道

抛出异常的爱 写道

刑天战士 写道

#         String sql = ""; 
#         if (sql != null) 

哈哈哈哈哈哈哈……

lottype ！＝null

if (username.length() > 1) 

这种代码改时间长了自然会明白他想写什么
不过很恶心用
sql = "XXXXX"+id
或这类的其它写法。

不是很懂java，想向抛抛请教几个问题：
1. sql = "XXXX" + id;为什么恶心？
是怕SQL注入还是String相加时候引起的性能问题？

2. if (username.length() > 1)
这有不对吗？

一般都不会这样写的，比如你可以用PreparedStatement.
username!=""就可以直接代替username.length() > 1,为什么要写那么麻烦呢

90 楼 qsrock 2008-05-14  

呵呵，可能lz那些是例外的。我也是07的,我想大部分都不会那样写的吧

89 楼 yashen 2008-05-14  

从功能上看没有任何问题，对于初学者来说，判断对象是否为空也是好习惯，虽然有些多余

88 楼 拎着板砖上轮坛 2008-05-14  

土匪一份子 写道

刑天战士 写道

土匪一份子 写道

楼主   坦白说：你真的很不负责任。。。。。。
我也是07年毕业的   看来我很幸运  现在看来以前带我的那个组长真的不错，刚开始做项目时，每完成一个模块他都会抽一点时间看我的代码，然后指导哪里的代码不规范，哪里的代码应该怎么来写效率更高，刚开始修改的地方还是有些，慢慢的，修改的越来越少，后来有闲时直接是让我看我自己以前写的代码，让我重构自己的代码，再后来是让我重构别人的代码。。。。。。   

为什么自己的事情要别人负责？

刑天大老，我想对于刚毕业的人来说，各方面经验都还比较欠缺，公司或项目经理让你带新人，你总不应该把他们扔在一边，让他们自由发挥吧，这对公司或那个人或那个项目不是很不负责吗。。

他才1年半，而已，你自己对自己不负责，不对项目负责，却反过来怪罪别人不对你负责，很好，很强大，宝宝，奶吃太多了，影响X能力的。

87 楼 土匪一份子 2008-05-14  

刑天战士 写道

土匪一份子 写道

楼主   坦白说：你真的很不负责任。。。。。。
我也是07年毕业的   看来我很幸运  现在看来以前带我的那个组长真的不错，刚开始做项目时，每完成一个模块他都会抽一点时间看我的代码，然后指导哪里的代码不规范，哪里的代码应该怎么来写效率更高，刚开始修改的地方还是有些，慢慢的，修改的越来越少，后来有闲时直接是让我看我自己以前写的代码，让我重构自己的代码，再后来是让我重构别人的代码。。。。。。   

为什么自己的事情要别人负责？

刑天大老，我想对于刚毕业的人来说，各方面经验都还比较欠缺，公司或项目经理让你带新人，你总不应该把他们扔在一边，让他们自由发挥吧，这对公司或那个人或那个项目不是很不负责吗。。

86 楼 土匪一份子 2008-05-14  

murainwood 写道

太忙顾不了怎么办啊?

忙不是借口，有心的话再忙也会挤出点时间。。。呵呵，这是在帮你自己，不是帮别人，如果你现在多抽点时间，可能目前你会更忙点，但想想他们进步的快，对你是很有好处的，以后你更可以放心的把东西交给他们做，你的压力自然就少了。。。。。。（这是体会，我组长后来干脆都不怎么动了，什么东西都交给别人，他只监督一下。。或专心做一些需要攻关的东西）

85 楼 天才阿昭 2008-05-14  

其实我倒是觉得JAVA应该支持这种写法，哈哈哈，反正很多人会错

84 楼 刑天战士 2008-05-14  

土匪一份子 写道

楼主   坦白说：你真的很不负责任。。。。。。
我也是07年毕业的   看来我很幸运  现在看来以前带我的那个组长真的不错，刚开始做项目时，每完成一个模块他都会抽一点时间看我的代码，然后指导哪里的代码不规范，哪里的代码应该怎么来写效率更高，刚开始修改的地方还是有些，慢慢的，修改的越来越少，后来有闲时直接是让我看我自己以前写的代码，让我重构自己的代码，再后来是让我重构别人的代码。。。。。。   

为什么自己的事情要别人负责？

83 楼 murainwood 2008-05-14  

太忙顾不了怎么办啊?

82 楼 土匪一份子 2008-05-14  

楼主   坦白说：你真的很不负责任。。。。。。
我也是07年毕业的   看来我很幸运  现在看来以前带我的那个组长真的不错，刚开始做项目时，每完成一个模块他都会抽一点时间看我的代码，然后指导哪里的代码不规范，哪里的代码应该怎么来写效率更高，刚开始修改的地方还是有些，慢慢的，修改的越来越少，后来有闲时直接是让我看我自己以前写的代码，让我重构自己的代码，再后来是让我重构别人的代码。。。。。。   

81 楼 raojl 2008-05-14  

林杰杰 写道

抛出异常的爱 写道

刑天战士 写道

#         String sql = ""; 
#         if (sql != null) 

哈哈哈哈哈哈哈……

lottype ！＝null

if (username.length() > 1) 

这种代码改时间长了自然会明白他想写什么
不过很恶心用
sql = "XXXXX"+id
或这类的其它写法。

不是很懂java，想向抛抛请教几个问题：
1. sql = "XXXX" + id;为什么恶心？
是怕SQL注入还是String相加时候引起的性能问题？

2. if (username.length() > 1)
这有不对吗？

这种东西我也用，简单嘛，象帮学校做罗，除非安全特别要，采用解码方式！人是活的啊

80 楼 xiaoyuonline 2008-05-14  

小题大作,工作中做了点错事有什么大不了的,是人就会犯错,我们都在是犯错中成熟起来的,不要成熟起来后就看不起还不成熟的人,这是部分人的劣根!