Question

我遇到JavaMailSender发送双重或三重邮件的问题。

在控制器中：

while(size > 0) //# of emails I want to send, for example 5
{
    Item item= itemRepository.findFirstBySentFalseAndValidTrue(); //getting only emails that are not sent!
    item.getData(); // API call that adds some info to the item, takes between 5-15 seconds per item    
    if(item.isValid())
    {

            item.setHeaderName(dataService.setHeader(item.getStore(), LocaleContextHolder.getLocale()));
            item.setAddress(dataService.getAddress(item.getStore(), LocaleContextHolder.getLocale()) + item.getId());

            String email = dataService.getEmail(item.getStore());
            String footer = dataService.getFooter(item.getStore(), LocaleContextHolder.getLocale());

            if(!item.isSent()) //I check if sent here but still get double messages!
            {
                if(mailSenderService.sendEmail(item, "emails/templates/item", email, footer))
                {
                    item.setSent(true);
                }
            }

    itemRepository.save(item);
    size--;
    }
}

在我的电子邮件发送服务类：

    final MimeMessage mimeMessage = this.mailSender.createMimeMessage();
    final MimeMessageHelper message = new MimeMessageHelper(mimeMessage,true, "UTF-8");

    Context ctx =  new Context(LocaleContextHolder.getLocale());
    message.setFrom(email);
    message.setTo("test@gmail.com");

    ResourceBundle labels = ResourceBundle.getBundle("messages", LocaleContextHolder.getLocale());
    mimeMessage.setHeader("Content-Type", encodingOptions);
    mimeMessage.setSubject(labels.getString("email.itme.title"), "UTF-8");


    ctx.setVariable("data", item);
    ctx.setVariable("footer", footer);

    try{        
        String processedTemplate =  templateEngine.process(template, ctx);
        mimeMessage.setContent(processedTemplate, encodingOptions);
        this.mailSender.send(mimeMessage);

    } catch(Exception e){
      e.printStackTrace();
    }

    return true;
}

我只是假设它在我的电子邮件发送类中存在问题，但是api调用可能太慢，方法执行时间太长并且发送重复的消息？有时，当我只发送几封电子邮件（例如3）时，它确实没有发送重复的电子邮件。当我想发送20或30时，一切都需要大约15分钟，我得到重复（我无法控制API的响应时间，有时候它超级速度有时会更快）。有什么方法可以调试它，看看哪个方法被调用的时间和原因我会得到这些重复项？

编辑：多数民众赞成我的itemRepository实现：

 @Repository
 @Qualifier(value = "itemRepository")
 @Transactional
 public interface itemRepository extends CrudRepository<Item, Long> {

 public Item save(Item item);
 public Item findFirstBySentFalseAndValidTrue();
 }

Answer 1

这部分：

while(size > 0) //# of emails I want to send, for example 5
{
    Item item= itemRepository.findFirstBySentFalseAndValidTrue(); //getting only emails that are not sent!

没有任何意义。您指定了要发送的静态电子邮件数量，然后永不递减此值，然后从尚未发送的存储库中调用一封电子邮件，并使用它执行某些操作。这是糟糕的设计。您可能会收到重复项，因为没有任何东西阻止您的程序使用以下代码行重复检索相同的记录：Item item= itemRepository.findFirstBySentFalseAndValidTrue();之前它有机会更新记录。

您应该做的是查询您的存储库以查找符合某些条件的未发送电子邮件列表，然后在此返回列表上执行循环以发送电子邮件，然后更新存储库中的记录。

Java邮件发件人发送重复的电子邮件

1 个答案: