面试题，下面代码有什么不妥之处

1. if(username.equals(“zxx”){}

	2.	int  x = 1;

		return x==1?true:false;

------解决方案--------------------
第一个 username可能为null 会空指针异常 应该把zxx放在前面
第二个 三目运算是多余的，直接返回x==1就行了
------解决方案--------------------
 一楼都讲完了 
1 常量和变量比较时，常量放前面，避免变量为空引起的空指针  
2 return x == 1 就够了，后面的 ? : 纯属多余，没有意义