Как правильно оформить код?

Question

[федор unknown]

Есть объект, в котором завернуть другой объект, у которого есть лист объектов.
мне надо достать лист объектов и обогащать данными.
Вопрос: как правильно проверить лист на пустоту?
так как до листа надо обойти большую цепочку объектов.

код:

spoiler

public List<Receipt> buildTitle(List<ComServiceResponse> comServiceResponses) {

        List<Receipt> receipts = new ArrayList<>();

        for (ComServiceResponse comService : comServiceResponses) {
            
            // if list not empty
            if (comService != null && comService.getComment() != null && !comService.getComment().getComs().isEmpty()) {

                String date = calendarService.formatDate(comService.getPeriod());

                boolean paymentStatus = isPayed(comService.getComment().getComs());
                double totalAmount = calculateComServiceTotalAmount(comService.getComment().getComs());

                Receipt receipt = new Receipt();

                receipt.setPeriod(date);
                receipt.setPayed(paymentStatus);
                receipt.setTotalAmount(totalAmount);
                receipt.setAccount(comService.getAccount());
                receipt.setServiceIcon(properties.getMyVillageIcon());
                receipt.setName(comService.getComment().getComs().get(0).getOwner());
                receipt.setAddress(comService.getComment().getComs().get(0).getAddress());

                receipts.add(receipt);
            }
        }
        return receipts;
    }

Answer 1

[Dmitry Roo]

Примерно так:

public List<Receipt> buildTitle(List<ComServiceResponse> comServiceResponses) {
return comServiceResponses.stream()
 .map(ComServiceResponse::getComment)
 .map(Comment::getComs)
 .filter(Objects::nonNull)
 .flatMap(List::stream) // Collection::stream ?
 .map(comService -> {   // todo:: extract to mapping method
                var date = calendarService.formatDate(comService.getPeriod()); // todo :: check for null here and below

                var paymentStatus = isPayed(comService.getComment().getComs()); 
                var totalAmount = calculateComServiceTotalAmount(comService.getComment().getComs());

                var receipt = new Receipt();  // todo:: beter use fluent accessors or builder

                receipt.setPeriod(date);
                receipt.setPayed(paymentStatus);
                receipt.setTotalAmount(totalAmount);
                receipt.setAccount(comService.getAccount());
                receipt.setServiceIcon(properties.getMyVillageIcon());
                receipt.setName(comService.getComment().getComs().get(0).getOwner());
                receipt.setAddress(comService.getComment().getComs().get(0).getAddress());
                
                return receipt;
 })
 .toList();
}

Современный язык Java. Лямбда-выражения, потоки и ...

Comments

[федор unknown]

// todo:: beter use fluent accessors or builder

а чем тут билдер хорош?
спасибо!

[Dmitry Roo]

федор unknown, мне лично не нравится такая череда сеттеров.
Так, кмк, выглядит симпатичнее:
https://projectlombok.org/features/experimental/Ac...

return new Receipt()
                .setPeriod(date)
                .setPayed(paymentStatus)
                .setTotalAmount(totalAmount)
                .setAccount(comService.getAccount())
                .setServiceIcon(properties.getMyVillageIcon())
                .setName(comService.getComment().getComs().get(0).getOwner())
                .setAddress(comService.getComment().getComs().get(0).getAddress());

https://projectlombok.org/features/Builder

return Receipt.builder()
                .period(date)
                .payed(paymentStatus)
                .totalAmount(totalAmount)
                .account(comService.getAccount())
                .serviceIcon(properties.getMyVillageIcon())
                .name(comService.getComment().getComs().get(0).getOwner())
                .address(comService.getComment().getComs().get(0).getAddress())
                .build();

[mayton2019]

Как вы там exceptions обрабатываете?

[Dmitry Roo]

mayton2019, никак. Это нарушение srp.