V1 개선사항 정리

1. 중복 코드 제거 및 공통화

1-1. 트랜잭션 커밋 후 처리 로직 공통화

• 대상

  • BlockSyncService.java
  • WalkWebSocketService.java

• 내용

  • 중복 구현된 sendAfterCommit() 유틸을 공통 유틸로 통합

  private void sendAfterCommit(Runnable action) {
  		if (TransactionSynchronizationManager.isSynchronizationActive()) {
  			TransactionSynchronizationManager.registerSynchronization(new TransactionSynchronization() {
  				@Override
  				public void afterCommit() {
  					action.run();
  				}
  			});
  			return;
  		}
  		action.run();
  }
  // 해당 로직이 서비스에서 중복으로 구현되어 있음
  

1-2. 응답 매핑 로직 공통화

• 대상

  • WalkService.java
  • BlockService.java

• 내용

  • BlockIdUtil 기반 blockId → response 변환 로직 중복 제거

  private NearbyBlockResponse toNearbyBlock(BlockOwnership ownership) {
  		Block block = ownership.getBlock();
  		String blockId = BlockIdUtil.toBlockId(block.getX(), block.getY());
  		return NearbyBlockResponse.from(
  				blockId,
  				ownership.getDog().getId(),
  				ownership.getAcquiredAt()
  		);
  }
  

  private OccupiedBlockResponse toOccupiedBlock(BlockOwnership ownership) {
  		Block block = ownership.getBlock();
  		String blockId = BlockIdUtil.toBlockId(block.getX(), block.getY());
  		return OccupiedBlockResponse.from(
  				blockId,
  				ownership.getDog().getId(),
  				ownership.getAcquiredAt()
  		);
  }
  

  • 공통 클래스로 통합

1-3. 입력 검증 로직 공통화

• 대상

  • BlockService.java
  • WalkWebSocketService.java

• 내용

  • 좌표 유효성 검증 로직을 공통 validator로 분리

  private boolean isValidInput(Double lat, Double lng, Integer radiusMeters) {
  		if (lat == null || lng == null || radiusMeters == null) {
  			return false;
  		}
  		if (!Double.isFinite(lat) || !Double.isFinite(lng)) {
  			return false;
  		}
  		return radiusMeters > 0;
  	}
  

  private boolean isValidCoordinate(double lat, double lng) {
  		if (!Double.isFinite(lat) || !Double.isFinite(lng)) {
  			return false;
  		}
  		return lat >= -90.0 && lat <= 90.0 && lng >= -180.0 && lng <= 180.0;
  }
  

• 기대 효과

  • 검증 정책 변경 시 누락 방지
  • 서비스 코드 단순화

2. N+1 및 불필요한 엔티티 로딩 제거

2-1. 소유 블록 count 방식 개선

• 대상
  • WalkService.java
• 기존