我想重构我的代码以一次获取所有图像并将其映射到JPA中的ID

Question

我有两个表是图像表，另一个是购物车表。 现在，当我想将购物车检索为列表时，我正在迭代购物车，并通过产品PK查找图像，然后对其进行编码并将其响应。

        for (Cart cart : cartList) {
            itemAndQuantityResponses.add(ItemAndQuantityResponse.builder()
                    .image(fileHandler.getStringImage(imageRepository.findByImageRelationAndForeignId(ImageRelation.ITEM_MAIN, cart.getItem().getId())))
                    .build());
        }

我认为这效率很低，所以我决定对其进行重构以及这样的结果。

        final List<Long> itemIds = cartList.stream().map(Cart::getItem).map(Item::getId).collect(Collectors.toList());
        final List<Image> images = imageRepository.findByImageRelationAndForeignIdIn(ImageRelation.ITEM_MAIN, itemIds);
        for (ItemAndQuantityResponse itemAndQuantityResponse : itemAndQuantityResponses) {
            for (Image image : images) {
                final String stringImage = fileHandler.getStringImage(image);
                if (itemAndQuantityResponse.getItemId().equals(image.getForeignId())) {
                    itemAndQuantityResponse.setImage(stringImage);
                    break;
                }
            }
        }

但是无论哪种方式对我来说都不好。 如果可能的话，我想尽可能避免使用双重循环。

即使是一个微小的提示，对我都会非常有帮助。 感谢您的帮助！

Answer 1

我会在map＆lt; number，image＆gt;中缓存图像（是否image＃getForeignId返回number）：

Map<Number, Image> imageByForeign = images.stream()
    .collect(Collectors.toMap(Image::getForeignId, Function.identity()));

现在您可以通过响应并直接获取图像。如果可以在没有匹配图像的情况下进行任何响应，我会这样这样做：

for (ItemAndQuantityResponse itemAndQuantityResponse : itemAndQuantityResponses) {
    Optional.of(imageByForeign)
        .map(m -> m.get(itemAndQuantityResponse.getItemId()))
        .map(fileHandler::getStringImage)
        .ifPresent(itemAndQuantityResponse::setImage);

如果每个响应的ID是地图中的一个键，则可以摆脱可选。