Skip to content

Comments

fix: SonarCloud 정적 분석 이슈 정리 (logger, dead code, utility class 개선)#76

Merged
zzmnxn merged 11 commits intodevfrom
refactor/75-medium-issuee
Jan 26, 2026
Merged

fix: SonarCloud 정적 분석 이슈 정리 (logger, dead code, utility class 개선)#76
zzmnxn merged 11 commits intodevfrom
refactor/75-medium-issuee

Conversation

@chwwwon
Copy link
Contributor

@chwwwon chwwwon commented Jan 25, 2026

🔗 Related Issue

📝 Description

무엇을(What), 왜(Why) 변경했는지 설명합니다.
SonarCloud 정적 분석 결과, 로그 처리 방식·불필요한 코드·유틸리티 클래스 구조와 관련된 여러 코드 스멜이 확인되어 이를 일괄 수정했습니다.
로직 변경 없이 코드 품질 및 유지보수성 개선을 목표로 합니다.

🛠 Changes

핵심 변경 사항을 요약합니다.

  1. 표준 출력 제거 및 Logger 사용
    System.out.println, System.err.println 사용 제거
    SLF4J 기반 log.info, log.error로 전환
    예외 로깅 시 stack trace 포함하도록 수정
  2. 주석 처리된 dead code 제거
    실행되지 않는 주석 코드(//System.out.println, 미사용 변수 선언 등) 삭제
    SonarCloud “commented-out code” 이슈 해결
  3. 유틸리티 클래스 구조 개선
    CountryValidator, EmojiValidator 등에 private 생성자 추가
    암묵적 public 생성자 제거로 인스턴스화 방지
  4. 미사용 필드 제거
    사용되지 않는 EMOJI_PATTERN 정규식 필드 삭제
    Uncovered / unused code 경고 해결
  5. 보안 및 안정성 개선
    getAuthorities()에서 null 반환 제거 → Collections.emptyList() 반환
    NPE 가능성 제거

영향 범위

기능 로직 변경 없음
기존 API 동작 및 비즈니스 로직 영향 없음
로그 출력 방식만 변경

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@zzmnxn
Copy link
Contributor

zzmnxn commented Jan 26, 2026

확인했습니다! 머지할게요

@zzmnxn zzmnxn merged commit 3dcc0c8 into dev Jan 26, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants