Problema com o ForEach que esta pegando somente o último

Ola pessoal!

Estou com o seguinte problema :

tenho essa interface com esse método :

@Repository
public interface EmailRepository extends JpaRepository<Email, Long> {

    @Query("select  e from Email e join fetch e.attachments join fetch e.recipients where e.emailStatus in ?1")
    public List<Email> getUnsentEmails(List<EmailStatus> emailStatus);
}

e a classe que executa essa método esta da seguinte forma :

public void sendEmails() throws MessagingException {

        MimeMessage mimeMessage = javaMailSender.createMimeMessage();

        MimeMessageHelper helper = new MimeMessageHelper(mimeMessage, true);

        List<Email> emails = this.emailRepository.getUnsentEmails(Arrays.asList(EmailStatus.NEW, EmailStatus.ERROR));

        emails.forEach(s -> {
            String[] recipients = s.getRecipients()
                    .stream()
                    .map(String::new)
                    .toArray(String[]::new);

            try {
                helper.setText(s.getMessage());
                helper.setFrom(sender);
                helper.setTo(recipients);
                helper.setSubject(s.getTitle());
                helper.setReplyTo(replyTo);
                
                for (Attachment attachment : s.getAttachments()) {
                    helper.addAttachment(attachment.getFilename(), new ByteArrayResource(attachment.getData()));
                }
            } catch (MessagingException e) {
                logger.error(e.getMessage(), e);
            }
        });
        
        javaMailSender.send(mimeMessage);
    }

Porem na hora de enviar os emails,mesmo tendo por exemplo 5 emails com status NEW, somente o último é enviado, estou achando que é problema nesses forEach aqui :

for (Attachment attachment : s.getAttachments()) {
                        helper.addAttachment(attachment.getFilename(), new ByteArrayResource(attachment.getData()));

porem não conseguir resolver o mesmo de outra forma. Aqui o getAttchments é Set que foi declarado na classe Email e o getRecipients é um List tambem declarado na classe Email, que no techo é convertida em um Array de String por causa do setTo do JavaMail.

Algum amigo saberia o que poderia esta ocorrendo e me aponta uma solução.

Obrigado

2 Respostas

Solucao aceita

AbelBueno 18 de mai. de 2017

O seu problema é que você cria um objeto mimeMessage, depois percorre a lista de emails que você tem, sobreescrevendo as propriedades nesse objeto mimeMessage e por último, fora do loop, vc manda enviar o email (com o método send do objeto javaMailSender).

A soluçao simples seria colocar dentro do seu emails.forEach, a parte que cria o mimeMessage, a parte que cria o helper e a parte que envia o email.

Uma outra coisa, dá pra transformar seu recipientes em um array com o método toArray da classe list.

PaoloSilva 18 de mai. de 2017

Obrigado. Funcionou perfeitamente.

2 Respostas

Topicos relacionados