Good Code - 규약과 구조

@Data
@Builder
@JsonIgnoreProperties(ignoreUnknown = true)
public class RequestDto {
 
    @Schema(description = "파일 URL", requiredMode = Schema.RequiredMode.NOT_REQUIRED)
    private String url;
 
    @Schema(description = "업로드 할 파일의 메타 정보", requiredMode = Schema.RequiredMode.NOT_REQUIRED)
    private FileMetadata metadata;
}

/**
 * 메타데이터가 비어있는지 확인.
 * 클라이언트에서 이미지 추출에 실패하면, 기본값을 가진 메타데이터 전송.
 */
@JsonIgnore
public boolean isEmptyMetadata() {
    return width == 0 && height == 0 && filesize == 0;
}

@Service
public class ClientApiService {
 
    public ResponseDto execute(RequestDto requestDto) {
        var metadata = requestDto.getMetadata();
        if (metadata.isEmptyMetadata()) {
            log.info("이미지 메타데이터 정보가 없어 이미지 바이트로부터 크기 계산 수행");
            ...
        }
        // 메타데이터를 꺼내서 사용
    }
}

@RestController
public class ClientApiController {
 
    public ResponseEntity<ResponseDto> request(RequestDto dto, ClientRequestDto clientRequestDto) {
        log.info("신규 요청 {} | {}", dto, clientRequestDto);
        var responseDto = clientApiService.execute(dto);
        log.info("요청 결과 : {}", responseDto);
        return ResponseEntity.ok(responseDto);
    }
}

@RestController
public class InternalRequestApiController {
 
    public ResponseEntity<ResponseDto> request(InternalRequestDto dto) {
        log.info("내부 sdk 통한 요청: {}", dto);
        var responseDto = clientApiService.execute(dto.toRequestDto());
        log.info("요청 결과 : {}", responseDto);
        return ResponseEntity.ok(responseDto);
    }
}

public class RequestDto {
 
    /**
     * 사용할 수 있는 메타데이터를 가지고 있는지 확인.
     * 클라이언트가 아예 실어 보내지 않으면 null, 추출에 실패하면 기본값(0)을 가진 객체가 온다.
     */
    @JsonIgnore
    public boolean hasFileMetadata() {
        return metadata != null && metadata.isUsable();
    }
}

if (!requestDto.hasFileMetadata()) {
    log.info("이미지 메타데이터 정보가 없어 이미지 바이트로부터 크기 계산 수행");
    ...
}
var metadata = requestDto.getMetadata();

public class RequestDto {
 
	@Nullable  
	@Schema(description = "업로드 할 파일의 메타 정보", requiredMode = Schema.RequiredMode.NOT_REQUIRED)  
	@Getter(AccessLevel.NONE)  
	private FileMetadata metadata;
 
    @JsonIgnore
    public Optional<FileMetadata> findUsableMetadata() {
        return Optional.ofNullable(metadata).filter(FileMetadata::isUsable);
    }
}

public class FileMetadata {
    private int width;
    private int height;
    private long filesize;
 
    /**
    *크기 계산에 쓸 수 있는 값인지. 하나라도 비면 신뢰하지 않는다.
    */
    @JsonIgnore
    public boolean isUsable() {
        return width > 0 && height > 0 && filesize > 0;
    }
}

ImageSize size = requestDto.findUsableMetadata()
        .map(ImageSize::from)
        .orElseGet(() -> {
            log.info("이미지 메타데이터 정보가 없어 이미지 바이트로부터 크기 계산 수행");
            return ImageSize.from(bytes);
        });

// InternalRequestDto.toRequestDto()
return RequestDto.builder()
        .url(url)
        .metadata(metadata != null && metadata.isUsable() ? metadata : null)
        .build();