【一道面试题】下面的代码有什么不妥之处?

这样两道题,猛的一看，好像没什么问题，但仔细一推敲就发现有问题

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

第一题 如果username 为null ，那么会报 空指针异常，我们应该这样来做

    if("zxx".equals(username)）{}

这样就不会报错了！

再来看第二题

    //2
    int  x = 1;
    return x==1?true:false;

我们先思考下 x == 1 返回的是什么类型？ boolean类型 对吧？ 那么直接return x==1；不就好了 ，干嘛要写三目运算符？