spring boot2 aop対象になるとnullになるServiceクラスがあります。

Question

前提・実現したいこと

只今、spring boot2にてaop機能を利用したメソッドのログ出力を実施しています。
ただ、MenuControllerをaop対象にした場合、menuService変数がnullになってしまいます。
aop対象外にするとオブジェクトが正常に参照されます。
できれば、MenuControllerをaop対象にしてログ出力したいと考えております。

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

2021/11/09 16:03:16 WARN  [http-nio-9210-exec-5] - Resolved [java.lang.NullPointerException]

該当のソースコード

package jp.co.toraya.aop;

import org.aspectj.lang.JoinPoint;
import org.aspectj.lang.annotation.After;
import org.aspectj.lang.annotation.AfterThrowing;
import org.aspectj.lang.annotation.Aspect;
import org.aspectj.lang.annotation.Before;
import org.aspectj.lang.annotation.Pointcut;
import org.slf4j.Logger;
import org.springframework.stereotype.Component;


@Aspect
@Component
public class LogAspct {
	Logger logger = org.slf4j.LoggerFactory.getLogger(LogAspct.class);

	// ～Controllerクラスとメソッドを対象にする
	@Pointcut("execution(* *..*.*Controller.*(..))")
	boolean loggingPattern() {
		return true;
	}

	// 除外クラスを指定。
	// aop対象にMenuControllerを指定すると52行目menuServiceがnullになってしまう。
	// 原因不明。
//	@Pointcut("execution(* *..*.MenuController.*(..)) " + "|| execution(* *..*.ForwardController.*(..) )" + "")
	@Pointcut("execution(* *..*.ForwardController.*(..) ) " )
	boolean ignorePattern() {
		return true;
	}

	// AOP実装
	@Before("loggingPattern() && !ignorePattern()")
	public void startLog(JoinPoint jp) {
		logger.info("メソッド開始 : " + jp.getSignature());
	}

	// AOP実装
	@After("loggingPattern() && ! ignorePattern()")
	public void endLog(JoinPoint jp) {
		logger.info("メソッド終了 : " + jp.getSignature());
	}
	@AfterThrowing(value = "loggingPattern() && ! ignorePattern()", throwing = "e")
	public void afterThrowing(Throwable e){
		// logger.info(e.getMessage());
		logger.warn("Caught Exception.", e);
	}



}

package jp.co.toraya.controller;




import java.util.Enumeration;
import java.util.List;

import javax.servlet.http.HttpSession;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.security.core.Authentication;
import org.springframework.security.core.context.SecurityContextHolder;
import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.bind.annotation.RequestParam;

import jp.co.toraya.constants.TzzConstants;
import jp.co.toraya.domain.KoyuMenuJoin;
import jp.co.toraya.service.MenuService;
import jp.co.toraya.service.MenuSub;
import jp.co.toraya.service.Pankuzu;

/**
 * メニュー画面のコントローラクラス
 * @author hogehoge
 *
 */
@Controller
public class MenuController {

	@Autowired
	private MenuService menuService;  // !!ここのmenuServiceがaop対象になるとnullになります。
	@Autowired
	HttpSession session; 	

	/**
	 * ログイン成功時に呼び出されるメソッド。ホームリンク押下時呼ばれるメソッド。
	 * SecurityContextHolderから認証済みユーザの情報を取得しモデルへ追加する
	 * @param model リクエストスコープ上にオブジェクトを載せるためのmap
	 * @return menuページのViewName
	 */
	@RequestMapping("/menu")
	private String init(Model model) {
		Authentication auth = SecurityContextHolder.getContext().getAuthentication();
		//Principalからログインユーザの情報を取得
		String shainCd = auth.getName();

		// sidebarMenuのデータセット
		List<KoyuMenuJoin> koyuMenuJoinList = menuService.setKoyuMenuJoinList(shainCd);
		model.addAttribute("koyuMenuJoinList", koyuMenuJoinList);

		// パンくずの削除
		session.removeAttribute("pankuzuList");
		// springや、パンくず以外のセッションをメニューに戻った際は消す。
		clearSession();
		return "menu";
		
	}

	/**
	 * sidebarMenuのボタンが押された時の処理。
	 * @param model リクエストスコープ上にオブジェクトを載せるためのmap
	 * @return menuページのViewName
	 */
	@RequestMapping(value = "menu", params = "gyomuBunruiOyaNo", method = RequestMethod.GET)
	private String clickedSidebarMenu(@RequestParam("gyomuBunruiOyaNo") String gyomuBunruiOyaNo
			,Model model) {
		Authentication auth = SecurityContextHolder.getContext().getAuthentication();
		//Principalからログインユーザの情報を取得
		String shainCd = auth.getName();

		// sidebarMenuのデータセット
		List<KoyuMenuJoin> koyuMenuJoinList = menuService.setKoyuMenuJoinList(shainCd);
		model.addAttribute("koyuMenuJoinList", koyuMenuJoinList);

		// メニューのmainのデータセット
		List<MenuSub> menuSubList = menuService.setMenuSubList(shainCd, gyomuBunruiOyaNo);
		model.addAttribute("menuSubList", menuSubList);

		// パンくずリストの作成
		List<Pankuzu> pankuzuList = menuService.setPankuzuList(gyomuBunruiOyaNo);
		// パンくずをセッションスコープへ保持
		session.setAttribute("pankuzuList", pankuzuList);
		// springや、パンくず以外のセッションをメニューに戻った際は消す。
		 clearSession();
		// str
		//  以下の処理を書いても、別のControllerのセッションを消してくれない。
		// sessionStatus.setComplete();
		


		
		
		// end
		return "menu";
		
	}
	// springや、パンくず以外のセッションをメニューに戻った際は消す。
	private void clearSession() {
		for(Enumeration<String> enumeration = session.getAttributeNames();enumeration.hasMoreElements();) {
			String attributeName = (String)enumeration.nextElement();
			// System.out.println("attributeName:" + attributeName);
				if(!TzzConstants.NO_CLEAR_SESSION_ATTRIBUTE_NAME.contains(attributeName)) {
					session.removeAttribute(attributeName);
					System.out.println("削除する属性:" + attributeName);
				}
		}
	}
	
	

}

package jp.co.toraya.service;

import java.util.ArrayList;
import java.util.List;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;

import jp.co.toraya.domain.KoyuMenuJoin;
import jp.co.toraya.domain.KoyuMenuJoinPgm;
import jp.co.toraya.domain.KoyuMenuJoinSub;
import jp.co.toraya.domain.MenuOya;
import jp.co.toraya.domain.Shain;
import jp.co.toraya.mapper.KoyuMenuJoinMapper;
import jp.co.toraya.mapper.MenuOyaMapper;
import jp.co.toraya.mapper.ShainMapper;

@Service
public class MenuService {
	@Autowired
	KoyuMenuJoinMapper koyuMenuJoinMapper;
	@Autowired
	ShainMapper shainMapper;
	@Autowired
	MenuOyaMapper menuOyaMapper;
	/**
	 * sidebarMenuのデータセット
	 * @param shainCd	社員コード
	 * @param gyomuBunruiOyaNo	業務分類親NO
	 * @return sidebarMenuのデータ
	 */
	public List<KoyuMenuJoin> setKoyuMenuJoinList(String shainCd) {
		KoyuMenuJoin koyuMenuJoin = new KoyuMenuJoin();
		koyuMenuJoin.setGyomuBunruiOyaName("ホーム");
		Shain shain = shainMapper.findByShainCd(shainCd);
		List<KoyuMenuJoin> koyuMenuJoinList = koyuMenuJoinMapper.findByKoyuMenuCd(shain.getKoyuMenuCd());
		koyuMenuJoinList.add(0, koyuMenuJoin);
		return koyuMenuJoinList;
	}

	/**
	 * メニューのmainのデータをセット
	 * @param shainCd	社員コード
	 * @param gyomuBunruiOyaNo	業務分類親NO
	 * @return メニューのmainのデータ
	 */
	public List<MenuSub> setMenuSubList(String shainCd, String gyomuBunruiOyaNo) {
		List<KoyuMenuJoinSub> koyuMenuJoinSubList = koyuMenuJoinMapper.findByKoyuMenuCdGyomuBunruiOyaNo(
				shainMapper.findByShainCd(shainCd).getKoyuMenuCd(), 
				gyomuBunruiOyaNo
				);
		MenuSub menuSub = null;
		List<MenuSub> menuSubList = new ArrayList<MenuSub>();
		MenuProgram menuProgram = null;
		List<MenuProgram> menuProgramList = null;
		for (KoyuMenuJoinSub koyuMenuJoinSub : koyuMenuJoinSubList) {
			menuSub = new MenuSub();
			menuSub.setGyomuBunruiKoCd(koyuMenuJoinSub.getGyomuBunruiKoCd());
			menuSub.setGyomuBunruiKoName(koyuMenuJoinSub.getGyomuBunruiKoName());

			menuProgramList = new ArrayList<MenuProgram>();
			List<KoyuMenuJoinPgm> koyuMenuJoinPgmList = 
					koyuMenuJoinMapper.findByKoyuMenuCdGyomuBunruiOyaNoGyomuBunruiKoNo(
							shainMapper.findByShainCd(shainCd).getKoyuMenuCd(), 
							gyomuBunruiOyaNo, 
							koyuMenuJoinSub.getGyomuBunruiKoCd()
							);
			for (KoyuMenuJoinPgm koyuMenuJoinPgm : koyuMenuJoinPgmList) {
				menuProgram = new MenuProgram();
				menuProgram.setPrgCd(koyuMenuJoinPgm.getPgmId());
				menuProgram.setMenuHyojiName(koyuMenuJoinPgm.getPgmName());
				menuProgram.setUrl(koyuMenuJoinPgm.getUrl());
				menuProgram.setHikiCnt(koyuMenuJoinPgm.getHikiCnt());
				menuProgramList.add(menuProgram);
			}
			menuSub.setProgramList(menuProgramList);
			menuSubList.add(menuSub);
		}
		return menuSubList;
	}

	/**
	 * パンくずリストの作成
	 * @param userId	ユーザーID
	 * @param gyomuBunruiOyaNo	業務分類親NO
	 * @return メニューのmainのデータ
	 */
	public List<Pankuzu> setPankuzuList(String gyomuBunruiOyaNo) {
		// パンくずの作成（セッションスコープへ保持）
		ArrayList<Pankuzu> pankuzuList = new ArrayList<Pankuzu>();
		Pankuzu pankuzu = new Pankuzu(); 
		pankuzu.setUrl("menu"); 
		pankuzu.setHikiCnt(null); 
		pankuzu.setName("ホーム"); 
		pankuzuList.add(pankuzu);  

		pankuzu = new Pankuzu(); 
		pankuzu.setUrl("menu"); 
		pankuzu.setHikiCnt("?gyomuBunruiOyaNo=" + gyomuBunruiOyaNo); 
		MenuOya menuOya = menuOyaMapper.findByPrimaryKey(gyomuBunruiOyaNo);
		pankuzu.setName(menuOya.getGyomuBunruiOyaName()); 
		pankuzuList.add(pankuzu);
		return pankuzuList; 
	
	}

}

試したこと

色々自身で調査した結果では、
・該当Serviceクラスをインターフェースにする。
・finalメソッドをやめる。
・privateでの参照から、publicへ変更する。
などの情報より色々試してみましたが、改善しませんでした。

お気づきの点がありましたらご指摘頂けます出ようか。
宜しくお願い致します。

Answer 1

MenuControllerのAOP対象になっているメソッドをpublicにして試してみてください。
自分も同じような問題を抱えていましたが、それで解決しました。