来自java面试题基础部分的一道题
分析下面代码的不妥之处：
1.if(username.equals(“zxx”)){}
2.int x = 1;
return x == 1 ? true : false;

1.if(username.equals(“zxx”){}，这个不是少了右边的一个括号吗？还是楼主手误了？
  如果没有手误，那应该会报错，NullPointerException。
2.intx=1;
returnx==1?true:false;
x==1这个表达式本来就返回boolean。不用再?true:false;

if(username.equals(“zxx”){}
首先是命名，标示符都是遵循驼峰式规范，其次就是“zxx”应该与username位置对换一下，要是username再前面的话，若果出现username为空值，会出现异常，不能得到结果；“zxx”在前面的话，无论username是否为空，都能得到比较的结果！
returnx==1?true:false;
x==1得到的本来就是一个布尔值，这样用三目运算符显然显得有点多余，问号后面的不要就可以了！

lcjasas 发表于 2013-4-3 20:31
1.if(username.equals(“zxx”){}，这个不是少了右边的一个括号吗？还是楼主手误了？
  如果没有手误，那应 ...

知道了。谢谢

黑马-郑玉元 发表于 2013-4-3 20:34
if(username.equals(“zxx”){}
首先是命名，标示符都是遵循驼峰式规范，其次就是“zxx”应该与username位 ...

谢谢。。。