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

1.

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

username可能为null。如果为null就是null.equals("xyz"),会出现NullPointerException异常。正确的应该是“xyz”.equals（username）

2.

int x = 1;

return x==1?true:fals

如果要返回boolean,?那么可以直接return?x==1,?(x==1这个表达式本来就返回boolean)