让人头疼的新手

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

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

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


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

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

评论

119 楼 rjzou2006 2008-05-16  

这个应该给他们一些时间吧,没人一生下来就会生很好的.

118 楼 vicksong 2008-05-16  

抛出异常的爱 写道

vicksong 写道

hotjava 写道

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

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

你以为你工作4,5就很牛是吧，跟我比工龄和项目经验，我应该不输给你吧。
说名凭良心的话，我们这里这么多做IT的，有多少是做一些对代码质量或性能要求特别特别高的系统？大多人还是做一些中小型系统吧，功能相对来说，还算简单吧，如果不是做产品的话，有必要为了那20%的东西，浪费公司80%的成本吗？在满足客户功能的前提下，有必要让公司配一个小秘给你写文档吗，中国软件，可不象国外，有那么多小秘......

楼上我经验不如你但是：
能看的明白的代码
这个要求并不高吧

我也不理解你们做项目管理的是怎么做的，其实应界生，也只能做做界面什么的，一些核心的代码他们根本不需要理解，在一些简单的界面等非关键地方，代码只要没有错误也就行了，如果说你的系统核心只有SSH，那肯定完了，系统下来后，100%五花八门了，对于应界生，不能要求太高，关键是项目管理要可行......

117 楼 ddandyy 2008-05-15  

实际上null == sql 是c时代留下来的习惯
只是习惯问题   在JAVA里 sql == null的影响不是那么明显的

但是 text.equals("0")  就肯定不可以了.....

116 楼 ball_cao 2008-05-15  

yueye 写道

luyunjian 写道

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 有怎样的区别？

第一种写法比较安全，如果你不小心写了一个=就惨了，莫名其妙还不知道怎么死的。。。

另 在大多数编译执行的语言中null==sql比 sql==null执行快 两个值比较将常量放在前面比将变量放在前面速度更快

115 楼 aboutibm 2008-05-15  

面对旧得发黄的电脑,我很难有写程序的冲动

114 楼 抛出异常的爱 2008-05-15  

vicksong 写道

hotjava 写道

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

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

你以为你工作4,5就很牛是吧，跟我比工龄和项目经验，我应该不输给你吧。
说名凭良心的话，我们这里这么多做IT的，有多少是做一些对代码质量或性能要求特别特别高的系统？大多人还是做一些中小型系统吧，功能相对来说，还算简单吧，如果不是做产品的话，有必要为了那20%的东西，浪费公司80%的成本吗？在满足客户功能的前提下，有必要让公司配一个小秘给你写文档吗，中国软件，可不象国外，有那么多小秘......

楼上我经验不如你但是：
能看的明白的代码
这个要求并不高吧

113 楼 vicksong 2008-05-15  

hotjava 写道

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

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

你以为你工作4,5就很牛是吧，跟我比工龄和项目经验，我应该不输给你吧。
说名凭良心的话，我们这里这么多做IT的，有多少是做一些对代码质量或性能要求特别特别高的系统？大多人还是做一些中小型系统吧，功能相对来说，还算简单吧，如果不是做产品的话，有必要为了那20%的东西，浪费公司80%的成本吗？在满足客户功能的前提下，有必要让公司配一个小秘给你写文档吗，中国软件，可不象国外，有那么多小秘......

112 楼 yueye 2008-05-15  

luyunjian 写道

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 有怎样的区别？

第一种写法比较安全，如果你不小心写了一个=就惨了，莫名其妙还不知道怎么死的。。。

111 楼 luckaway 2008-05-15  

跟应届没关系。跟公司的环境、规模、态度有关系；
就算有几年工作经验！
写的代码也不考虑性能问题，怎么能实现就怎么写。
这些设计都没经过大脑思考的

110 楼 Jissbon 2008-05-15  

楼主的话有道理，干什么都得首先端正自己的态度！
  态度决定一切！

109 楼 抛出异常的爱 2008-05-15  

luyunjian 写道

是我说的不清楚，我的问题引申自
"aaa".equals(userName)和
userName.equals("aaa")的区别；
同事告诉我道理类似于
null==userName和
userName==null的区别；
我也百度google搜索过，一直没有答案，放在心中快半年了

平时不用这种裸函数了

apache.common这个包下面好像有个stringutil类常用这个。。。。少写N多代码。

108 楼 luyunjian 2008-05-15  

是我说的不清楚，我的问题引申自
"aaa".equals(userName)和
userName.equals("aaa")的区别；
同事告诉我道理类似于
null==userName和
userName==null的区别；
我也百度google搜索过，一直没有答案，放在心中快半年了

107 楼 抛出异常的爱 2008-05-15  

niwei 写道

dingji 写道

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

------------------------------------------------------
假设，你不小心将"=="写成了"="，如果是sql==null就变成了sql=null,对于编译器来说，语法没有问题，如果是if判断则条件永真。。反过来如果是null==sql就变成null=sql，语法错误，不能给一个常量赋值，报错。。所以我一般将这种句式的变量写后面，自从采用这样写法，腰不酸了，背不疼了，生活更和谐了

终于解决了我很长时间的疑问。

很多人一眼看不明白了。
在写代码时
有很多两可的选择。。。。比如大扩号，缩进，都是两可的。

你看看代码大全这本书很多想法会在你的代码中体现出来
比如：
你写的 sql==null
在一般用法中是

if(sql!=null){
//nothing to do 
}else{
//do something 
 return ;
}

这种形式的工作

106 楼 niwei 2008-05-15  

dingji 写道

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

------------------------------------------------------
假设，你不小心将"=="写成了"="，如果是sql==null就变成了sql=null,对于编译器来说，语法没有问题，如果是if判断则条件永真。。反过来如果是null==sql就变成null=sql，语法错误，不能给一个常量赋值，报错。。所以我一般将这种句式的变量写后面，自从采用这样写法，腰不酸了，背不疼了，生活更和谐了

终于解决了我很长时间的疑问。

105 楼 抛出异常的爱 2008-05-15  

sai619 写道

spiritfrog 写道

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));  
没有想通为什么要写where 1=1 恒等的条件， 而且这里的参数处理明显麻烦了点，只要检测参数是否为空，再拼接where好了

确实有点诡异 有点乱

--我是说我的脑子 呵呵

a =" select XXX ,XXX ,XXX ";
b =" form XXX where 1=1  ";
c =" and tt =?";
d =" and mm =?";

这样子写if时简单一点

104 楼 sai619 2008-05-15  

spiritfrog 写道

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));  
没有想通为什么要写where 1=1 恒等的条件， 而且这里的参数处理明显麻烦了点，只要检测参数是否为空，再拼接where好了

确实有点诡异 有点乱

--我是说我的脑子 呵呵

103 楼 sai619 2008-05-15  

还好 我在刚开始学习java的时候就找了个规范看了看 至少代码风格还凑合啊 呵呵
至于字符串用+连接 并不只是新手的专利 老手也会出现的

感觉应该严于律己 宽以待人

建议看看重构 嘿嘿 很不错的书 专门买了一本当枕头呢 哈哈

102 楼 spiritfrog 2008-05-14  

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));  
没有想通为什么要写where 1=1 恒等的条件， 而且这里的参数处理明显麻烦了点，只要检测参数是否为空，再拼接where好了

101 楼 dingji 2008-05-14  

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

------------------------------------------------------
假设，你不小心将"=="写成了"="，如果是sql==null就变成了sql=null,对于编译器来说，语法没有问题，如果是if判断则条件永真。。反过来如果是null==sql就变成null=sql，语法错误，不能给一个常量赋值，报错。。所以我一般将这种句式的变量写后面，自从采用这样写法，腰不酸了，背不疼了，生活更和谐了

100 楼 miracle9i 2008-05-14  

晕 晒代码的就那水平，还不让别人说几句....

说真的，我要晒代码，至少重构一次再拿出来，第一次写出来的过程性语言，就不要出来丢人了