麻烦老师看下这样写正确吗？

package com.imooc.user;
 
public class User {
    //定义属性用户名、密码
    private String name;
    private String pwd;
     
    public User(String name,String pwd){
        this.setName(name);
        this.setPwd(pwd);
    }
     
    public void setName(String name){
        this.name=name;
    }
    public String getName(){
        return this.name;
    }
    public void setPwd(String pwd){
        this.pwd=pwd;
    }
    public String getPwd(){
        return this.pwd;
    }
     
    public void userInfo(){
        System.out.println("用户名："+getName());
        System.out.println("密码："+getPwd());
    }
}
 
package com.imooc.user;
 
public class UserManager {
    // 用户信息验证的方法
    String str;
    public String checkUser(User one, User two) {
       // 判断用户名是否为空，是否一致
        if((one.getName()==null||two.getName()==null)){
            System.out.println("用户名不能为空");
        }else if(one.getName()!=two.getPwd()){
            System.out.println("用户名不一致");
        }
    // 判断密码是否为空，是否一致
        if((one.getPwd()==null||two.getPwd()==null)){
            System.out.println("密码不能为空");
        }else if(one.getPwd()!=two.getPwd()){
            System.out.println("密码不一致");
        }
        return str;
    }
}
 
package com.imooc.user;
public class Test {
 
    public static void main(String[] args) {
        // TODO Auto-generated method stub
         //实例化对象，调用相关方法实现运行效果
         User one=new User("Lucy","123456");
         User two=new User("Mike","123456");
         UserManager um=new UserManager();
         one.userInfo();
         two.userInfo();
         System.out.println("====================");
         um.checkUser(one, two);   
    }
 
}

涅槃重生Jayden 2017-08-17

源自：Java封装 1-11 编程练习

收起

1回答

Tender10 2017-08-18 11:28:56

程序运行结果正确，但是针对字符串的判断建议使用equals()方法，更加有效地进行字符串的判断。另外，判断用户名是否一致时，应该是用户名之间的对比，你写成了用户名和密码的对比。继续努力，加油~

收起回答

FLY8 #1

为什么我感觉是错的。if里面的条件是密码和用户名对比。而且并不是用eauals方法

2017-09-13 22:41:47
Tender10 回复 FLY8 #2

嗯嗯，是的。他的最终输出结果是正确的，但是实现的方式中还是有一些细节的问题。正如你说的字符串建议使用equals()方法进行对比，用户名是否一致的判断也有问题。

2017-09-14 09:36:49