Контейнер Spring не вводит зависимости после введения аспекта - PullRequest
Новая
       14

Контейнер Spring не вводит зависимости после введения аспекта

1 голос
/ 20 июня 2019

Проект хорошо работал до использования AOP, но после добавления класса LoggingAspect появился NullPointerException.Я выполнил отладку проекта, и в тот момент, когда появляется NullPointerException, нет объекта, у которого есть метод для вызова.Объект defaultLogger встроен в конструктор объекта Runner.Почему так происходит? Ссылка на проект 

@Component
@Aspect
public class LoggingAspect {

    @Pointcut("execution(* *.logEvent(..))")
    public void allLogEventMethods() {}

    @Before("allLogEventMethods()")
    public void logBefore(JoinPoint joinPoint) {
        System.out.println("BEFORE: " +
                joinPoint.getTarget().getClass().getSimpleName() + " " +
                joinPoint.getSignature().getName());
    }

    @AfterReturning(pointcut="allLogEventMethods()",
            returning="retVal")
    public void logAfter(Object retVal) {
        System.out.println("AFTER_RET: " + retVal);
    }

    @AfterThrowing(pointcut="allLogEventMethods()",
            throwing="ex")
    public void logAfterThrow(Throwable ex) {
        System.out.println("AFTER_THR: " + ex);
    }
}

@Component
public class Runner {
    private Client client;
    private EventLogger defaultLogger;
    private Map<EventType, EventLogger> loggers;

    @Autowired
    public Runner(
            Client client,
            EventLogger defaultLogger,
            Map<EventType, EventLogger> loggerMap) {
        this.client = client;
        this.defaultLogger = defaultLogger;
        this.loggers = loggerMap;
    }

    public static void main(String[] args) {
        ConfigurableApplicationContext ac = 
            new ClassPathXmlApplicationContext("context.xml");
        Runner app = ac.getBean(Runner.class);
        Event event;
        int i = 0;
        while (i < 10) {
            event = ac.getBean(Event.class);
            event.setMessage("Some event for " + i++);
            app.logEvent(null, event);
        }
        while (i < 20) {
            event = ac.getBean(Event.class);
            event.setMessage("Some event for " + i++);
            app.logEvent(EventType.INFO, event);
        }
        while (i < 30) {
            event = ac.getBean(Event.class);
            event.setMessage("Some event for " + i++);
            app.logEvent(EventType.ERROR, event);
        }

        ac.close();
    }

    private void logEvent(EventType type, Event message) {
        EventLogger eventLogger;
        if (type == null) eventLogger = defaultLogger;
        else eventLogger = loggers.get(type);
        eventLogger.logEvent(message);                //NullPointerException
    }
}

Соответствующий объект помечен @Component ("defaultLogger"). 

@Component("defaultLogger")
public class CacheFileLogger extends FileLogger {
    private int cacheSize;
    private List<Event> events;

    public CacheFileLogger(
            @Value("${filename}") String filename,
            @Value("${cacheSize}") int cacheSize
    ) {
        super(filename);
        this.cacheSize = cacheSize;
        events = new ArrayList<>(cacheSize);
    }

    @Override
    public void logEvent(Event event) {
        events.add(event);
        if (cacheSize == events.size()) {
            writeAndClearCache();
        }
    }

    private void writeAndClearCache() {
        for (Event event1 : events) {
            super.logEvent(event1);
        }
        events.clear();
    }

    @PreDestroy
    private void destroy() {
        if (!events.isEmpty()) {
            writeAndClearCache();
        }
    }
}

Файл context.xml: 

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns="http://www.springframework.org/schema/beans"
       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
       xmlns:context="http://www.springframework.org/schema/context"
       xmlns:aop="http://www.springframework.org/schema/aop"
       xsi:schemaLocation="http://www.springframework.org/schema/beans
        https://www.springframework.org/schema/beans/spring-beans.xsd http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context.xsd http://www.springframework.org/schema/aop http://www.springframework.org/schema/aop/spring-aop.xsd">
    <context:annotation-config/>
    <context:component-scan base-package="by.babanin"/>
    <aop:aspectj-autoproxy/>
</beans>

Изображение отладки

Ответы [ 2 ]

1 голос
/ 20 июня 2019

Ваш объект eventLogger становится нулевым, когда вы вызываете метод logEvent. Обязательно передавайте объекты без нулевых ссылок в конструктор класса runner через ваш context.xml. То, что я здесь написал, предназначено только для передачи объекта eventLogger в конструктор класса Runner. Вы должны сделать то же самое для двух других параметров конструктора. Обязательно поделитесь своими результатами с нами. 

<bean id="Runner" class="com.xyz.abc.Runner">
    <property name="EventLogger ">
      <bean class="com.xyz.abc.EventLogger ">
        <property name="param1" value="value1" />
        <property name="param2" value="value2" />
        <property name="param3" value="value3" />
      </bean>
    </property>
</bean>
0 голосов
/ 20 июня 2019

Ошибка исправлена, благодаря комментариям по этому вопросу.Я изменил приватный метод logEvent класса Runner, на публичный!Я бы объяснил, почему это сработало, но я не совсем понимаю, почему.Если я знаю, я оставлю объяснение в комментариях к этому ответу. 

public void logEvent(EventType type, Event message) {
        EventLogger eventLogger;
        if (type == null) eventLogger = defaultLogger;
        else eventLogger = loggers.get(type);
        eventLogger.logEvent(message);
    }
...