CLEANUP: Refactored to remove unnecessary codes or redundancies.

naver / arcus-spring

ARCUS as a caching provider for the Spring Cache Abstraction

Apache License 2.0

26 stars 16 forks source link

CLEANUP: Refactored to remove unnecessary codes or redundancies. #91

Closed uhm0311 closed 3 months ago

uhm0311 commented 3 months ago

🔗 Related Issue

⌨️ What I did

릴리즈 전 마지막 커밋으로, 아래의 사항을 반영합니다.

불필요한 import문 제거
불필요한 @SuppressWarnings 어노테이션 제거
필요한 @SuppressWarnings의 경우 메소드 단위로 적용되어 있으면 클래스 단위로 통합
String#toString() 메소드 호출 제거

jhpark816 commented 3 months ago

필요한 @SuppressWarnings의 경우 메소드 단위로 적용되어 있으면 클래스 단위로 통합

@oliviarla 메소드 단위 또는 클래스 단위 중 어떤 설정이 좋은 지를 리뷰 바랍니다.

oliviarla commented 3 months ago

@jhpark816 메서드 단위로 SuppressWarnings 어노테이션을 달면 어느 메서드에서 warning이 발생하는지 파악하기 쉽습니다.

제 생각에 ArcusCache 클래스에서는 기존대로 메서드마다 붙이는 것이 어느 메서드에서 warning이 발생하는지 파악하기 쉬워보입니다. ArcusCacheTest 클래스에서는 모든 테스트 메서드에 설정되어있는 어노테이션을 클래스 단위로 명시하는 것이 오히려 가독성이 좋아보일 것 같습니다.

jhpark816 commented 3 months ago

메서드 단위로 SuppressWarnings 어노테이션을 달면 어느 메서드에서 warning이 발생하는지 파악하기 쉽습니다.

제 생각에 ArcusCache 클래스에서는 기존대로 메서드마다 붙이는 것이 어느 메서드에서 warning이 발생하는지 파악하기 쉬워보입니다.

ArcusCacheTest 클래스에서는 모든 테스트 메서드에 설정되어있는 어노테이션을 클래스 단위로 명시하는 것이 오히려 가독성이 좋아보일 것 같습니다.

@uhm0311 내 생각도 위와 같은 생각입니다. 가능하면 위와 같이 하시죠.

uhm0311 commented 3 months ago

@oliviarla

반영했습니다.