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

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

 一楼都讲完了 
1 常量和变量比较时，常量放前面，避免变量为空引起的空指针  
2 return x == 1 就够了，后面的 ? : 纯属多余，没有意义