[Refactor/#34] MessageLog Annotation SpEl 표현식 적용 - [merged]

[TaeYeongKwak]

Merges refactor/#34 -> dev

📋 구현 내용

작업 내용에 대한 개요

LogAspect class messageBrokerLogAdvisor method 로직 일부 수정

🔍 테스트 케이스

• Annotation

• 결과

📢 코멘트

• 기존 일반 문자열을 넣어 처리했던 exchange 이름과 queue 이름을 SpEL 표현식으로 처리할 수 있도록 변경하였습니다.

@tommysgit @AnTaeWook @muyongKim

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 20, 2023, 13:43

Commented on src/main/java/com/gabia/weat/gcellapiserver/aspect/LogAspect.java line 65

반환 타입을 Object로 선언한 이유가 궁금합니다.

[TaeYeongKwak]

In GitLab by @AnTaeWook on Feb 20, 2023, 13:46

approved this merge request

[TaeYeongKwak]

현재 해당 advice가 적용된 메서드는 return 타입이 void이지만 후에 더 확장되어 return 하는 값이 존재하는 메서드에도 적용할 수 있을 것이라고 생각하여 return 타입을 Object로 변경하였습니다.

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 20, 2023, 13:55

Commented on src/main/java/com/gabia/weat/gcellapiserver/aspect/LogAspect.java line 65

후에 확장되면 return 타입의 추상화를 하나요?

[TaeYeongKwak]

In GitLab by @muyongKim on Feb 20, 2023, 13:57

approved this merge request

[TaeYeongKwak]

joinpoint.proceed() 메서드의 return type이 기본적으로 Object이기 때문에 따로 더 추상화 해줄 필요는 없을 것 같습니다.

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 20, 2023, 14:30

resolved all threads

[TaeYeongKwak]

mentioned in commit 6941b20ee5f64de53a2f57eaaea4613eabb5f8ef