CLEANUP: Refactored to remove unnecessary codes or redundancies.

[uhm0311]

🔗 Related Issue

⌨️ What I did

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

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

[jhpark816]

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

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

[oliviarla]

매개변수 명을 value -> v 로 변경하신 이유가 무엇인가요? 제생각에는 그대로 둬도 될 것 같아서요!

[uhm0311]

value라는 변수명이 어딘가에서 겹치기 때문에 @SuppressWarnings("SameParameterValue")가 붙은 걸로 생각했습니다.
@SuppressWarnings("SameParameterValue")를 지우면서 겹치지 않도록 변수명을 변경했는데, 그대로 둬도 워닝이 나오지 않네요.

[oliviarla]

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

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

[jhpark816]

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

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

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

[uhm0311]

@oliviarla

반영했습니다.