Question

我已经做完所有数据并将其从object转换为objectDto。但是，以某种方式，我觉得我的代码还不好。在这里，我需要您的帮助，我需要任何参考/建议以使我的代码更好（在性能上）。这是我的工作代码：

    @Override
    public List<BookDto> findAll() throws Exception {
        try {
            List<Book> list = bookDao.findAll();
            List<BookDto> listDto = new ArrayList<>();

            for (Book book : list) {
                BookDto bookDto = new BookDto();
                Set<AuthorDto> listAuthorDto = new HashSet<AuthorDto>();
                Set<Author> dataAuthor = new HashSet<Author>();
                book.getAuthor().iterator().forEachRemaining(dataAuthor::add);

                BeanUtils.copyProperties(book, bookDto, "author", "category");
                bookDto.setCategory(book.getCategory().getCategory());

                for (Author author : dataAuthor) {
                    AuthorDto authorDto = new AuthorDto();
                    BeanUtils.copyProperties(author, authorDto);
                    listAuthorDto.add(authorDto);
                }

                bookDto.setAuthor(listAuthorDto);
                listDto.add(bookDto);
            }
            return listDto;
        } catch (Exception e) {
            throw new Exception(e);
        }
    }

这是我需要的输出（已经通过上面的代码实现）：

[
    {
        "title": "book1",
        "year": "2013",
        "author": [
            {
                "name": "john",
                "address": "NY"
            },
            {
                "name": "angel",
                "address": "LA"
            }
        ],
        "category": "science"
    },
    {
        "title": "book2",
        "year": "2014",
        "author": [
            {
                "name": "john",
                "address": "NY"
            }
        ],
        "category": "science"
    },
    {
        "title": "book3",
        "year": "2009",
        "author": [
            {
                "name": "angel",
                "address": "LA"
            }
        ],
        "category": "comedy"
    }
]

Answer 1

与其重新发明轮子并自己编写轮子，不如使用为此目的而专门设计的现有库/工具。

我建议使用ModelMapper，它是DTO /实体映射的绝佳库，您只需要一行即可将您的实体转换为DTO，例如：

ModelMapper modelMapper = new ModelMapper();
BookDto bookDTO = modelMapper.map(book, BookDto.class);

您可以检查library's Examples page或Entity To DTO Conversion for a Spring REST API tutorial来更深入地了解该库。

Answer 2

首先，此代码：

} catch (Exception e) {
   throw new Exception(e);
}

太荒谬了，我建议不要向任何人展示此代码。

第二个-

List<Book> list = bookDao.findAll();
List<BookDto> listDto = new ArrayList<>();

您可以在此处分配具有确切大小的listDto，因为您使用list.size（）== listDto.size（），并且在其他任何位置，如果您知道确切的大小或目标集合，则可能必须使用它。（对于哈希集，也要注意负载系数）。

BeanUtils.copyProperties(

在这里使用不必要的反射，直接调用方法将提高性能。

如果您想要更多-您可以考虑使用哈希集来替换以查找唯一作者，那么可能会有更有效的方法。

作为一般建议，您可能希望使用http://mapstruct.org/documentation/stable/reference/html/将实体映射到dto，以避免编写样板代码。

所有这些问题看起来像是代码审查的任务。不是吗？

将列表对象转换为列表对象的最佳实践

2 个答案: