Question

我听说过处理运行时异常并不总是一个好习惯。我已经实现了一个将产品评级作为参数的方法，如果当前产品对象具有此评级，则返回ProductDTO对象。我发现使方法按预期工作的唯一方法是使用运行时异常。但根据Joshua Bloch的说法，使用控制流异常是一个非常糟糕的主意。

有没有办法改善方法的逻辑？

public ProductDTO findByRating(int productRating) {
    ProductDTO productDTO = new ProductDTO();
    if (productRating == this.avgRating()) {
        productDTO.setProductName(productName);
        productDTO.setProductsLeftForSale(productsLeftForSale());
        productDTO.setAvgRating(avgRating());
        productDTO.setTotalVotes(reviews.size());
        return productDTO;
    } else {
        throw new RuntimeException(String.format("No Product found with the rating: %s", productRating));
    }
}

List<ProductDTO> productDTOList = new ArrayList<>();    
        int rating = 5;
        for (BaseProduct product : products) {
            try {
                ProductDTO productDTO = product.findByRating(rating);
                if (productDTO != null) {
                    productDTOList.add(productDTO);
                }
            } catch (RuntimeException e) {
                e.printStackTrace();
            }
        }

Answer 1

也许看起来好一点：

import java.util.List;
import java.util.Optional;
import java.util.stream.Collectors;

...

public Optional<ProductDTO> findByRating(int rating) {
    return this.avgRating() == rating ? Optional.of(createDTO()) : Optional.empty();
}

private ProductDTO createDTO() {
    ProductDTO productDTO = new ProductDTO();
    productDTO.setProductName(productName);
    productDTO.setProductsLeftForSale(productsLeftForSale());
    productDTO.setAvgRating(avgRating());
    productDTO.setTotalVotes(reviews.size());
    return productDTO;
}

public List<ProductDTO> findProductsByRating(int rating) {
    return products.stream()
            .map(product -> product.findByRating(rating))
            .filter(Optional::isPresent)
            .map(Optional::get)
            .collect(Collectors.toList());
}

在方法中处理RuntimeException的更好方法是什么？

1 个答案: