为什么我收到 StackOverflowError

Question

public class Category {

    private Category parentCategory;
    private Set<Category> childCategories;
    private String name;

    public Category() {
        childCategories = new HashSet<Category>();
    }

    public Category getParentCategory() {
        return parentCategory;
    }

    public void setParentCategory(Category parentCategory) {
        this.parentCategory = parentCategory;
    }

    public Set<Category> getChildCategories() {
        return childCategories;
    }

    public void setChildCategories(Set<Category> childCategories) {
        this.childCategories = childCategories;
    }

    public String getName() {
        return name;
    }

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

    @Override
    public String toString() {
        return "Category [childCategories=" + childCategories + ", name="
                + name + ", parentCategory=" + parentCategory + "]";
    }

}


public static void main(String[] args) {
        Category books = new Category();
        books.setName("Books");
        books.setParentCategory(null);

        Category novels = new Category();
        novels.setName("Novels");
        novels.setParentCategory(books);

        books.getChildCategories().add(novels);
        //novels.setChildCategories(null);

        System.out.println("Books > " + books);
    }

System.out.println 正在生成 StackOverflowError。

Answer 1

当您执行 toString() 时，您会调用子级的 toString()。除了在这里调用父级的 toString() 之外，这里没有问题。这将调用子级的 toString() 等。

很好的无限循环。

摆脱它的最好方法是将您的 toString() 方法更改为：

@Override
public String toString() {
    return "Category [childCategories=" + childCategories + ", name="
            + name + ", parentCategory=" + parentCategory.getName() + "]";
}

这样您就不会打印parentCategory，而只打印它的名称，没有无限循环，也没有 StackOverflowError。

编辑：正如 Bolo 下面所说，您需要检查parentCategory 是否不为空，如果是，您可能会遇到NullPointerException。

---

资源：

• Javadoc - StackOverflowError< /a>

同一主题：

• java 中的 toString()
• Java postfix 计算器中的 StackOverFlowError

Answer 2

由于错误是 System.out.println，因此问题必定出在 toString() 中。

问题是，toString() 会打印您使用 toString() 方法打印的对象的父和子 Category 对象。因此，当您打印一个类别时，它会在父级上调用 toString() ，在子级上调用 toString() ，在子级上调用 toString() 。父级对子级调用 toString() ，依此类推，直到堆栈耗尽。

Answer 3

因为每次调用 Category#toString() 都会产生大量其他 toString。请注意，打印 childCategories 会导致打印出每个元素（通过 toString），从而重复许多 toString 方法调用的整个过程。

不仅如此，每个子级都会在其父级上调用 toString，而父级又会在其子级上调用 toString，每个子级都会在父级上调用 toString ，它又对其子级调用 toString ，这......

Answer 4

return "Category [childCategories=" + childCategories + ", name="
                + name + ", parentCategory=" + parentCategory + "]";

您正在使用诸如 parentCategory 之类的实例来连接到您的 toString。它将调用这些实例的 toString 方法。

toString 调用的循环永远不会结束。因为子类别将调用父类别，而父类别将再次调用子类别，所以...

不是说如果您输入： System.out.println(myObject); ，它实际上是： System.out .println(myObject.toString());

Answer 5

您的 toString() 正在陷入递归混乱。您需要有两个toString();一份给父母，一份给孩子。您的 toString 可能如下所示：

@Override
public String toString() {
     toStringParent(parent);   // This print only parent
     toStringChildren(children);  // This print only children
}

Answer 6

StackOverflowError 实际上是在构建将传递给 System.out.println 的 String 参数时引发的。

每次连接 String 和 Category 时，都会执行 Category 的 toString() 方法。问题是 Category 中的 toString() 有点过于冗长。解决此问题的一种方法是仅打印父类别的名称（跳过其父类别和子类别）：

@Override
public String toString() {
    return "Category [childCategories=" + childCategories + ", name="
            + name + ", parentCategory="
            + ((parentCategory == null) ? null : parentCategory.getName())
            + "]";
    }