Spring Boot starter-mail メールが2回送信されてしまう

Question

前提・実現したいこと

いつもお世話になっております。
実現したいこと
Spring Bootのstarter-mailを使って複数人に同じ内容のメールを1通のみ送信したい

メールは送信されますが、メソッドが2回繰り返されてしまい、二重にメールが
送信されてしまいます。

2回送信される原因が分かりません。

発生している問題・エラーメッセージ

メールが二重送信されてしまう

該当のソースコード

MailController

package jp.co.itc.mbo.controller;

import java.util.List;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.mail.SimpleMailMessage;
import org.springframework.mail.javamail.JavaMailSender;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;

import jp.co.itc.mbo.entity.UserMaster;
import jp.co.itc.mbo.service.UserMasterService;

@Controller
@RequestMapping("/mailer")
public class MailController {

	@Autowired
	UserMasterService usermasterservice;

	    final String charset = "UTF-8";
	    private final JavaMailSender javaMailSender ;

		@Autowired
		MailController(JavaMailSender javaMailSender) {
			this.javaMailSender = javaMailSender;
		}

		@RequestMapping(value = "/send", method = RequestMethod.GET)
		public String send(){
			List <UserMaster> users=usermasterservice.findAllValidUserId();
			//ここから↓が二週目にはいってループ
			int cnt=users.size();
			String[] mailList= new String[cnt];
			
           int i=0;
			for(UserMaster user:users) {
				String mailaddress=user.getMailaddress();
				mailList[i]=mailaddress;
				i++;
			}

			  SimpleMailMessage msg = new SimpleMailMessage();

		        msg.setFrom(“送信元");
		        msg.setTo(mailList);
		        msg.setSubject("テストメール");//タイトルの設定
		        msg.setText("Spring Boot より本文送信"); //本文の設定

		        javaMailSender.send(msg);

		        //一周目でreturnが処理されず　int cnt=users.size();に戻ってしまう
		        //二週目でreturmが処理されメールが2通送信される
		        
				return "redirect:/home";
		}

}

試したこと

・デバックをしたところ、for文を書いていない部分が2回回っていました（詳しくはコメントアウトに記載）
・msg.setTo(mailList);の()の中に1つのメールアドレスを直に打つと1回送信されます。
・mailListの中身を2つにすると1回送信、3つ以上だと2回送信されます。

どうやら配列型で送信しようとするとおかしくなるみたいです(?)

補足情報（FW/ツールのバージョンなど）

MailControllerのみでしかメールに関係する操作をしていないので1つしかクラスを載せていませんが、SpringMVCで関連するクラスがほかにありましたらご教授いただきたいです。
初心者で至らない点が多いと思いますがよろしくお願いします。

Comments

[退会済みユーザー]

とりあえずコンパイルエラーだからそのコードじゃないのでは?

Answer 1

インデントやスペースがやや読みにくいので整理しました。

送信先のメールアドレスをループして取り出しているだけで、メールの送信処理は1回だけです。
今回の例では、1回のメール送信で2つのアドレスに同じ内容のメールが1度飛んでいるだけです。

あとは行コメントに書いている内容をご覧ください。

	@RequestMapping(value = "/send", method = RequestMethod.GET)
    public String send(){
        List<UserMaster> users = Arrays.asList(new UserMaster("aaa"), new UserMaster("bbb"));
        //ここから↓が二週目にはいってループ（ここではない。配列のサイズを決めている）
        int cnt=users.size();
        String[] mailList= new String[cnt];

        int i=0;
        for(UserMaster user:users) {
            String mailaddress = user.getAddress();
            mailList[i] = mailaddress;
            i++;
        } // このループでは、送信先のメールアドレスをループの数だけ取り出しているだけ

        SimpleMailMessage msg = new SimpleMailMessage();

        msg.setFrom("送信元");
        msg.setTo(mailList);  // 複数のメールアドレスを設定している
        msg.setSubject("テストメール");
        msg.setText("Spring Boot より本文送信"); //本文の設定

        javaMailSender.send(msg);  // メールの送信処理はこの1回だけ！

        return "success";
    }

Comments

[chocolate_pie]

今回もご回答いただき誠にありがとうございます。
コードを見直しましたが、書き方がA-pZ様の回答と同じ書き方であったので、
再度デバックをしたところ、1回のみ送信の処理がされました。
前回デバックした際に2回送信されたことが謎ですが、無事解決しました。
ありがとうございました。