A股上市公司传智教育(股票代码 003032)旗下技术交流社区北京昌平校区

 找回密码
 加入黑马

QQ登录

只需一步,快速开始

下面的代码有什么不妥之处?
1. if(username.equals(“zxx”){
        2.        int  x = 1;
                return x==1?true:false;
}

评分

参与人数 1技术分 +1 收起 理由
神之梦 + 1

查看全部评分

4 个回复

倒序浏览
本帖最后由 Friendy89 于 2013-8-10 12:32 编辑

你的代码return 语句写在if里面,就可以直接返回结果了,而你已经定义x=1了就可以直接返回结果了而不需要再继续判断,应该这样写
  1. if(username.equals("zxx"){
  2.         int x = 1 ;
  3.         return true ;
  4. }else{
  5.         return false ;
  6. }
复制代码

评分

参与人数 1技术分 +1 收起 理由
神之梦 + 1

查看全部评分

回复 使用道具 举报
   uesername 如果是String类型   在没有赋值的情况 是 NULL  你这样写 会报 空指针异常  建议写成  "zxx".equals(username);

评分

参与人数 1技术分 +1 收起 理由
神之梦 + 1

查看全部评分

回复 使用道具 举报
付建超 来自手机 中级黑马 2013-8-10 15:19:00
板凳
1.在里面定义的那个x有什么作用??判断if语句以后返回对错情况即可,况且是个局部变量,出了大括号也就不认识了。
2.单纯只是要返回ture或false。
3.if(username.equals("zxx"))
        return true;
return false;

评分

参与人数 1技术分 +1 收起 理由
神之梦 + 1

查看全部评分

回复 使用道具 举报
付建超 来自手机 中级黑马 2013-8-10 15:20:24
报纸
"zxx".equals(username);确实是这样好
回复 使用道具 举报
您需要登录后才可以回帖 登录 | 加入黑马