下面的代码有什么不妥之处

今天看到网上一面试题

说下面的代码有什么不妥之处：

 

[java]  view plain  copy

1. 1. if(username.equals(“zxx”){}  
2.     2.  int  x = 1;  
3.         return x==1?true:false;  

第一题明白，比较的时候应该把可能为null的最好放到后面 否则可能报错。

第二题刚开始还真有点没看明白 就网上搜了下。

 

还真有在网上讨论这两题的。

看到有些人的回答简直吐血了...

第一题说少了个")"   仔细一看，靠，还真少了个")" 我估计人家的本意肯定不是让你说出少了一个")"

接下来就是越来越多的 鄙视，乱说连篇

[html]  view plain  copy

1. 这真是极品代码啊，你公司是招聘校对员的吗？第一个开发环境会给你校对，第二个是没有什么问题的。看来你公司的技术主管水平不杂地啊。  

 

[html]  view plain  copy

1. 楼主还是管好自己吧，我如果是你主管，你写出第二种代码，非骂死你不可。并且你在方法内部定义什么跟方法的返回值是没有关系的。第二个没错，代码有点多余罢了。看你的解答，让我觉的你学java的时间连一年都没有。  

 

[html]  view plain  copy

1. 回复的人都注意了 这是Java基础面试题  
2.   
3. 总结：  
4. 第一题：缺少“）”  
5. 第二题：没错误  

 

[html]  view plain  copy

1. 你第2个里面哪里说明返回值要是int类型 这么2的问题也拿来问  

 

[html]  view plain  copy

1. 哎，看了这么多楼的回复，都没找到重点，第一道题是字符串的比较，字符串的比较要用到equals吗？不是说用了会错，这是基础问题。用==和compare会怎么样呢？  

 

[html]  view plain  copy

1. 三元表达式的x==1是不是要加个括号，我记得书上这么写的…  

 

人家楼主也说了 只是在网上找的一个面试题而已，请大家来讨论指点指点。

哎，一群无知者，不懂就别说，说了就请说到重点子上 好吧。

 

x==1本来就是boolean类型的 故而后面是多余的。

转载出处：http://blog.csdn.net/itmyhome1990/article/details/7671926