面试题：下面的代码有什么不妥之处？

下面代码有什么不妥之处

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

答案

1，直接调用equals方法可能会抛出NullPointerException(空指针异常)，为了防止抛出异常调用equals方法之前要先判断一下是否为空
即：
if(username!=null&&username.equals(“zxx”){};

2,代码冗余，去掉后面的部分就可以了
int x = 1;
return x==1;
等于1时返回true，不等于1时返回false