黑马程序员技术交流社区

标题: 看到一段代码说有不妥的地方能帮我看看是哪嘛 [打印本页]

---

作者: 赵国刚    时间: 2013-8-10 07:48
标题: 看到一段代码说有不妥的地方能帮我看看是哪嘛
下面的代码有什么不妥之处?
1. if(username.equals(“zxx”){
        2.        int  x = 1;
                return x==1?true:false;
}

---

作者: Friendy89    时间: 2013-8-10 12:31


你的代码return 语句写在if里面，就可以直接返回结果了，而你已经定义x=1了就可以直接返回结果了而不需要再继续判断，应该这样写

1. if(username.equals("zxx"){
   
2.         int x = 1 ;
   
3.         return true ;
   
4. }else{
   
5.         return false ;
   
6. }

复制代码

---

作者: 张禄宇    时间: 2013-8-10 15:14
   uesername 如果是String类型   在没有赋值的情况 是 NULL  你这样写 会报 空指针异常  建议写成  "zxx".equals(username);

---

作者: 付建超    时间: 2013-8-10 15:19
1.在里面定义的那个x有什么作用？？判断if语句以后返回对错情况即可，况且是个局部变量，出了大括号也就不认识了。
2.单纯只是要返回ture或false。
3.if(username.equals("zxx"))
        return true;
return false;

---

作者: 付建超    时间: 2013-8-10 15:20
"zxx".equals(username);确实是这样好

---

欢迎光临 黑马程序员技术交流社区 (http://bbs.itheima.com/)

黑马程序员IT技术论坛 X3.2