我想知道，有没有更有效的方法来做到这一点

Question

有一个方法saveCountryIslands带有String数组参数。 有一个实体是国家和岛屿。

我一直在尝试使用jpa .save()保存提供的岛屿名称列表。

现有的方法主体已经在工作（您可以在下面的代码中看到）。

有更有效的方法吗？

private void saveCountryIslands(String[] islandNames){

    List<Islands> listOfIsland = new ArrayList<>();
    Country country=new Country();

    Stream.of(islandNames).distinct().forEach(islandNamesFromArray -> {
      Islands islands = new Islands();
      islands.setIslandName(islandNamesFromArray);
      listOfIsland.add(islands);
    });

    country.setCountryIslands(listOfIsland);
    countryRepository.save(country);

}

Answer 1

您可以通过以下方式使用构造函数来带来可读性的一个小变化：

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

然后将现有代码重构为：

    List<Islands> listOfIsland = Stream.of(islandNames)
            .distinct()
            .map(Islands::new)
            .collect(Collectors.toList()); // collect 'toSet' depending on compatibility