refactor: sonarCloud를 참고한 코드 리팩토링 #274

leeeeeyeon · 2023-10-17T08:25:34Z

🛰️ Issue Number

🪐 작업 내용

sonarCloud를 참고하여 코드 리팩토링을 진행했습니다.
우선 코드 변경이 적어 side effect가 적을 것 같은 부분들에 대해 1차적으로 리팩토링을 진행했습니다. 코드 변경이 많은 부분은 추후 새로운 PR로 2차 리팩토링을 진행하겠습니다.

Agora 관련 코드는 Agora 공식 코드를 참고하여 구현했기 때문에 코드를 임의로 수정할 경우 기능이 제대로 동작하지 않을 수 있어 우선 리팩토링을 진행하지 않았습니다.
printStackTrace 메서드가 사용되던 부분을 error log로 변경하였습니다.
사용하지 않는 import문을 제거하였습니다.
switch문을 if문으로 변경하였습니다.

👋 리뷰어가 중점적으로 확인해야 할 것 (Optional)

X

🔬 테스트 코드 결과

✅ Check List

코드가 정상적으로 컴파일되나요?
테스트 코드를 통과했나요?
merge할 브랜치의 위치를 확인했나요?
Reviewer를 지정했나요?
Assignee를 지정했나요?
Label을 지정했나요?

codecov · 2023-10-17T08:28:25Z

Codecov Report

Merging #274 (9cf84e9) into develop (d2866ba) will increase coverage by 0.02%.
Report is 2 commits behind head on develop.
The diff coverage is 12.00%.

@@              Coverage Diff              @@
##             develop     #274      +/-   ##
=============================================
+ Coverage      47.59%   47.62%   +0.02%     
- Complexity       367      369       +2     
=============================================
  Files            100      100              
  Lines           1975     1976       +1     
  Branches          52       54       +2     
=============================================
+ Hits             940      941       +1     
  Misses          1017     1017              
  Partials          18       18

Files	Coverage Δ
...com/gsm/blabla/auth/application/GoogleService.java	`9.09% <ø> (+0.75%)`	⬆️
...la/content/dto/MemberContentDetailResponseDto.java	`0.00% <ø> (ø)`
...n/java/com/gsm/blabla/crew/api/CrewController.java	`28.57% <ø> (ø)`
...ain/java/com/gsm/blabla/crew/domain/VoiceFile.java	`47.05% <ø> (ø)`
...rc/main/java/com/gsm/blabla/jwt/TokenProvider.java	`10.63% <ø> (ø)`
...a/com/gsm/blabla/admin/application/FcmService.java	`3.12% <0.00%> (-2.94%)`	⬇️
src/main/java/com/gsm/blabla/agora/Utils.java	`30.15% <50.00%> (+1.12%)`	⬆️
...bla/content/dto/AIPracticeFeedbackResponseDto.java	`0.00% <0.00%> (ø)`
...a/com/gsm/blabla/crew/application/CrewService.java	`25.80% <0.00%> (ø)`
...in/java/com/gsm/blabla/agora/RtcTokenBuilder2.java	`51.51% <33.33%> (+1.51%)`	⬆️
... and 2 more

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be1308e...9cf84e9. Read the comment docs.