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

今天看到网上一面试题

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

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

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

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

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

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

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

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

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

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

回复的人都注意了 这是Java基础面试题

总结：
第一题：缺少“）”
第二题：没错误

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

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

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

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

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

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