Question

我有两个班Product和Categorie。如下面的代码所示，当我想用categoryRepository.save(c1)修改类别的产品列表时，会发生此错误：

 java.lang.StackOverflowError: null
    at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:449) ~[na:1.8.0_191]
        at java.lang.StringBuilder.append(StringBuilder.java:136) ~[na:1.8.0_191]
        at org.sid.entities.Product.toString(Product.java:12) ~[classes/:na]
        at java.lang.String.valueOf(String.java:2994) ~[na:1.8.0_191]
        at java.lang.StringBuilder.append(StringBuilder.java:131) ~[na:1.8.0_191]
        at java.util.AbstractCollection.toString(AbstractCollection.java:462) ~[na:1.8.0_191]
        at java.lang.String.valueOf(String.java:2994) ~[na:1.8.0_191]
        at java.lang.StringBuilder.append(StringBuilder.java:131) ~[na:1.8.0_191]
        at org.sid.entities.Categorie.toString(Categorie.java:15) ~[classes/:na]
        at java.lang.String.valueOf(String.java:2994) ~[na:1.8.0_191]

@Document
@Data @AllArgsConstructor @NoArgsConstructor @ToString
public class Categorie {
    @Id
    private String id;
    private String name;
    @DBRef
    @JsonIgnore
    private Collection<Product> products=new ArrayList<>();

@Document
@Data @AllArgsConstructor @NoArgsConstructor @ToString
public class Product {
    @Id
    private String id;
    private String name;
    private double price;
    @DBRef
    private Categorie categorie;

@Bean
    CommandLineRunner start(CategoryRepository categoryRepository, ProductRepository productRepository){

        return args -> {
            categoryRepository.deleteAll();
            Stream.of("c1 Ordinateur","c2 Imprimente").forEach(c->{
                categoryRepository.save(new Categorie(c.split(" ")[0],c.split(" ")[1],new ArrayList<>()));
            });
            categoryRepository.findAll().forEach(System.out::println);

            productRepository.deleteAll();
            Categorie c1=categoryRepository.findById("c1").get();
            Stream.of("P1","P2","P3","P4").forEach(name->{
               Product p= productRepository.save(new Product(null,name,Math.random()*1000,c1));
               c1.getProducts().add(p);
               categoryRepository.save(c1);
            });

productRepository.findAll().forEach(p->{
                System.out.println(p.toString());
            });

        };
    }

有人能解决这个问题吗？谢谢。

Answer 1

我假设@ToString注释告诉您正在使用的某些工具（Lombok？）生成打印所有字段值的toString方法。每个类都相互引用：产品具有一个类别，而类别具有一个产品实例列表。因此，当toString实现打印一个Categorie时，它将在每个Product上调用toString，然后在其Categorie上调用toString，依此类推。由于Product大概是指一个在其产品列表中包含该Product的Categorie，因此toString来回调用直到堆栈溢出。解决方案是避免从toString方法中打印Categorie，products或Product.categorie。如果您使用的是Lombok，请尝试使用@ToString.Exclude注释Categorie.products。

Answer 2

您正在由Lombok生成的toString方法中使用循环引用。

Product引用Categorie上的toString，引用Product，依此类推

您可以使用排除属性@ToString，但很快就会弃用，因此请使用@ToString.Exclude：

@Document
@Data @AllArgsConstructor @NoArgsConstructor @ToString
public class Product {
  ...

  @ToString.Exclude
  private Categorie categorie;

  ...
}

@Document
@Data @AllArgsConstructor @NoArgsConstructor @ToString
public class Categorie {
  ...

  @ToString.Exclude
  private Collection<Product> products=new ArrayList<>();

  ...
}

龙目岛裁判here和here

Answer 3

我在 Spring JPA 实现中遇到了类似的问题，实体使用 Lombok @Data 进行注释，它们一起生成 @ToString、@EqualsAndHashCode、@Getter / @Setter 和 @RequiredArgsConstructor。但是@ToString、@EqualsAndHashCode 注释生成具有实体所有属性的代码，包括集合，这些属性通常引用其他实体和其他一对一关系实体。这会导致循环引用，也会导致加载惰性集合（JPA 默认以惰性方式加载一对多和多对多）。

添加 @ToString.Exclude 或 @EqualsAndHashCode.Exclude 会导致代码繁琐，强烈建议避免在 JPA 实体上使用 @Data 或 @ToString、@EqualsAndHashCode。最好使用 IDE 生成 toString、equals、hashCode 并在使用前验证它们（即非空与空字段等）。

龙目岛-java.lang.StackOverflowError：toString方法上为null

3 个答案: