Java 类问题（一对多 - 多对一）？

Question

我正在尝试制作一个程序狗注册，您可以在其中注册用户和狗。 我遇到的问题是让不同的类一起工作。

我想让它让用户可以有很多狗，但狗只能有一个用户。

我希望能够在类的 toString 中检索狗和主人。

当我为我的 User 类调用 toString 时，它可以工作，但当我从 Dog 类中尝试时，它不起作用。

更新我不明白的是，当它存储在 User 类中时，我将如何从 dog 类访问所有者？

我究竟做错了什么？

添加我的狗类

    public class Dog{

    private String name, breed;
    private int age, weight;
    private User owner;

    public Dog(String n, String b, int a, int w){

        name = n;
        breed = b;
        age = a;
        weight = w;
    }


    public void setName(String n){
        name = n;
    }

    public String getName(){
        return name;
    }

    public String getBreed(){
        return breed;
    }

    public int getAge(){
        return age;
    }

    public int getWeight(){
        return weight;
    }

    public double getTailLength(){
        if(getBreed().equalsIgnoreCase("Tax") || getBreed().equalsIgnoreCase("Dachshund") ){
            return 3.7;
        }else{
            double d = (getAge()*getWeight()/10.0);
            return d;
        }
    }

    public void increaseAge(int a){
        if(a<getAge()){
            this.age = age;
        }else{
            this.age += a;
        }
    }

    public int testAge(){
        return age += 1;
    }

    public void setOwner(User owner){
        User oldOwner = this.owner;
        this.owner = owner;

        if(oldOwner != null && oldOwner != owner){
            oldOwner.removeDog(this);
        }

        if(owner!=null){
            owner.addDog(this);
        }
    }

    public User getOwner(){
        return owner;
    }


    public String toString(){
        return getName() + " " + getBreed() + " " + getAge() + " år " + getWeight() + " kg svans=" + getTailLength() + getOwner();
    }
}

用户类

public class User{

    private String name;
    private ArrayList<Dog> owner = new ArrayList<>();

        public void addDog(Dog d1){
        if(!owner.contains(d1)){
            owner.add(d1);
            d1.setOwner(this);
        }
    }

    public void removeDog(Dog d1){
        if(owner.remove(d1)){
            d1.setOwner(null);
        }
    }

    public ArrayList<Dog> getDogs(){
        return new ArrayList<Dog>(owner);
    }

    public User(String n){
        this.name = n;
    }

    public String getName(){
        return name;
    }

    public String toString(){

        return getName() + " " + getDogs();

    }

}

Answer 1

这里：

 private ArrayList<Dog> owner = new ArrayList<>();

那应该只是：

 private ArrayList<Dog> ownedDogs = new ArrayList<>();

然后，您可以添加（或删除）属于该用户的狗。

最后：

public ArrayList<Dog> getDogs(){
    return new ArrayList<Dog

变成：

public ArrayList<Dog> getDogs(){
    return ownedDogs;
}

现在，每次调用getDogs()返回一个新的空列表。 这是没有意义的，因为你已经有了“自有”狗的名单。 如果有的话，您可以考虑创建新列表（包含所有拥有的狗），然后分发出去。 但是返回一个空列表是没有意义的！

关于stackoverflow：你必须确保那里没有递归！ 当向用户添加狗时……将用户添加为所有者，然后添加狗……您只需打破该链。

有趣的是，这是 OOP 处理状态中的一个非常基本的问题：这里有“双重簿记”，主人认识他的狗，所有的狗都认识他们的主人。 因此，以有意义的方式更新该信息实际上很难。

真正的答案在这里：根据现实构建您的模型！ 我的狗没有内置财产，上面写着“由 Ghostcat 拥有”。 我知道“这是我的狗”，某处有一些文件说“狗 X 归鬼猫所有”。 换句话说：考虑删除狗类的“所有者”字段。

Answer 2

这是我认为你应该如何解决这个问题的方法，首先是 Dog 类，为了清楚起见，我跳过了一些属性。 toString只包含狗的名字（以及我跳过的其他狗的特定属性），而不是我有一个特定的方法来获取所有者的名字进行打印

public class Dog {
    String name;
    User owner;

    public Dog(String name) { 
        this.name = name; 
    }

    public String getName() { return name; }

    public void setName(String name) { this.name = name; }

    public User getOwner() { return owner; }

    public void setOwner(User owner) { this.owner = owner; }

    public String ownerAsString() {
        if (getOwner() != null) {
            return getOwner().toString();
        } else {
            return "<No Owner>";
        }
    }

    @Override
    public String toString() {
        return name;
    }
}

User 类有一个狗列表， toString方法只返回用户名，而不是我有一个特定的方法来获取拥有的狗作为字符串。 这是为了在打印时不会在 User 和 Dog 之间获得循环引用。

public class User {
    String name;
    List<Dog> ownedDogs;

    public User(String name) {
        this.name = name;
        ownedDogs = new ArrayList<>();
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public List<Dog> getOwnedDogs() {
        return ownedDogs;
    }

    public void setOwnedDogs(List<Dog> ownedDogs) {
        this.ownedDogs = ownedDogs;
    }

    public void addDog(Dog dog) {
        ownedDogs.add(dog);
        dog.setOwner(this);
    }

    public String ownedDogsAsString() {
        return ownedDogs.toString();
    }

    @Override
    public String toString() {
        return name;
    }
}

然后可以像这样使用

User u = new User("abc");
Dog d1 = new Dog("D1");
Dog d2 = new Dog("D2");
u.addDog(d1);
u.addDog(d2);
System.out.println(u + " ownes " + u.getOwnedDogs());
System.out.print(d1 + " is owned by " + d1.ownerAsString());

输出：

abc 拥有 [D1, D2]
D1归abc所有