multipart/form-data로 보낼 때 순서 배열 원소값 0을 못 찾겠네요

Question

포스트맨으로 테스트 통신해보고 있었는데,현재 컨트롤러의 파라미터는Req(JSON), List, List를 받고,List == null일 경우, List.size() - 1만큼의 배열을 만들어서 order를 새로 구성하고,for(int i = 0, i < files.size(); i++) 돌면서 files[i], order[i]를 ENTITYRepository.save로 저장하는 로직을 구성중입니다.이 때, 동시요청을 몇회 해본 결과, order[i]에서 0값이 없는 데이터셋이 발견되었습니다.포스트맨 RequestBody - form-data로는,req, 파일들을 보낸 경우에도 0이 없는 경우가 발생했고, req, 파일들, List값을 전송했을 때에도 0이 없는 경우가 발생했습니다. 이는 어떤 것이 문제일까요? 객체 정적 변수는 사용하지 않았는데, 마치 쓰레드에서 공유 변수 참조처럼 느껴집니다. 물론, DB나 리스너 등으로 order값을 수정하는 코드도 전혀 없었습니다. 반복문 안에, S3에 업로드하는 코드가 있긴 한데, 이것 역시 쓰레드마다 개별적인 메모리 공간에서 작업을 수행하고 있다고 생각합니다. 전혀 짐작되는 곳이 없네요죄송합니다 코드 올리겠습니다. 공부 시작한지 얼마 안되어서 설명이 많이 부족하네요. 양해 부탁드립니다.컨트롤러@PostMapping(consumes = MediaType.MULTIPART_FORM_DATA_VALUE) public ResponseEntity> createClass( @RequestPart("req") CreateClassOpenReq req, // JSON 형태의 클래스 정보 @RequestPart(value = "images", required = true) List images, @RequestPart(value = "orders", required = false) List orders, // 사진 순서 @AuthenticationPrincipal CustomUserDetails user // 인증된 사용자 정보 ) { Long userId = user.getUser().getUserId(); CreateClassOpenRes res = classOpenService.createClassOpen(userId, req, images, orders); return ResponseEntity.status(HttpStatus.OK).body(SuccessResponse.ok(res)); }서비스@Transactional @Override public CreateClassOpenRes createClassOpen(Long userId, CreateClassOpenReq req, List images, List orders) { @Transactional @Override public ForFlaskReq createClassOpen(Long userId, CreateClassOpenReq req, List images, List orders) { validateBasic(req); ClassInfo info = resolveInfo(userId, req); MentorPlace mentorPlace = null; PlaceReservation govReservation = null; if (req.getMentorPlaceId() != null && req.getGovReservationId() != null) { throw new ClassOpenPlaceBothSelectedException(); } if (req.getMentorPlaceId() == null && req.getGovReservationId() == null) { throw new ClassOpenPlaceNotSelectedException(); } if (req.getMentorPlaceId() != null) { mentorPlace = mentorPlaceRepository.findById(req.getMentorPlaceId()) .orElseThrow(MentorPlaceNotFoundException::new); if (!mentorPlace.getMentor().getUserId().equals(userId)) { throw new MentorPlaceForbiddenException(); } } else { govReservation = placeReservationRepository.findById(req.getGovReservationId()) .orElseThrow(GovReservationNotFoundException::new); validateNoOverlapWithGovReservation(govReservation, req.getOpenAt(), req.getStartTime(), req.getEndTime()); boolean alreadyExists = classOpenRepository.existsByGovReservationAndOpenAtAndTimeOverlap( govReservation.getReservationId(), req.getOpenAt(), req.getStartTime(), req.getEndTime() ); if (alreadyExists) { throw new DuplicateClassOpenException(); } } User user = userRepository.getReferenceById(userId); ClassOpen open = ClassOpen.builder() .user(user) .info(info) .mentorPlace(mentorPlace) .govReservation(govReservation) .openAt(req.getOpenAt()) .startTime(req.getStartTime()) .endTime(req.getEndTime()) .capacity(req.getCapacity()) .price(req.getPrice()) .isFull(false) .build(); open = classOpenRepository.save(open); List imageUrls = new ArrayList<>(); List files = (images == null) ? List.of() : images.stream() .filter(f -> f != null && !f.isEmpty()) .toList(); if (!files.isEmpty()) { if (files.size() > MAX_IMAGES) throw new TooManyImagesException(); List ords = (orders != null && orders.size() == files.size()) ? orders : IntStream.range(0, files.size()).boxed().toList(); String dir = "class/" + open.getId(); // S3Uploader로 class/{openId} 경로에 업로드 for (int i = 0; i < files.size(); i++) { MultipartFile file = files.get(i); int sortOrder = ords.get(i); String url = s3Uploader.upload(file, dir); imageUrls.add(url); classImageRepository.save( // URL 저장 + ClassImage 엔티티 저장 ClassImage.builder() .classOpen(open) .imageUrl(url) .sortOrder(sortOrder) .build() ); } } String thumbnailUrl = classImageRepository .findFirstByClassOpen_IdOrderBySortOrderAscIdAsc(open.getId()) .map(ClassImage::getImageUrl) .orElse(null); return new CreateClassOpenRes( open.getId(), info.getId(), info.getTitle(), mentorPlace != null ? mentorPlace.getId() : null, govReservation != null ? govReservation.getReservationId() : null, open.getOpenAt().format(DATE_FMT), open.getStartTime().format(TIME_FMT), open.getEndTime().format(TIME_FMT), open.getCapacity(), open.getPrice(), open.isFull(), thumbnailUrl ); }DB에 저장은 잘 되고 있습니다. 관련 없다고 생각이 드는 코드 부분은 조금 제거해서, 보시기에 뭔가 어색한 부분도 있을 것 같습니다.

HJOW · Answer

이거 다른사람이 이해 못할 겁니다.우선 문제가,스프링/스프링부트를 쓰더라도, 파일 업로드는 여러가지 프레임워크로 구현할 수 있습니다.고전 방식 중 하나인 cos.jar 를 쓰는 곳도 여전히 많죠. (얘 여전히 유지보수도 되고있는 살아있는 녀석임)프레임워크에 따라 클래스명도 달라지고 당연히 메소드도 다 달라집니다.List<Integer> 로 대체 뭘 받으셨는데요? 정수 타입 컬렉션이, 파일 업로드 구현할 때 항상 발생하는 녀석이 아닙니다.즉 그건 님이 구현하는 로직에서만 필요한 녀석이란 뜻입니다.마찬가지로 order[i] 얘가 대체 뭔가요. 다른 사이트 파일 업로드에서는 필요 없는 녀석입니다.대강 추정하기로는 파일 업로드 순서를 같이 전달받으려 한 거 같은데 뭐 여하튼이건 님의 코드를 확인하지 않으면 절대 알 수가 없어요.혹시, 그 코드 남이 짠 거고, 그걸 유지보수하는 입장인데, 코드 제대로 이해 못하고 계신 건 아닌가요?수정하셔서 코드 첨부 하셨는데아래쪽에 다른분이 답변을 이미 하셔서 ~~

Lzlzlzzz · Answer

코드를 봐야죠.

캐티 · Answer

자바스크립트에서 0의 분기문은 처리되지 아니하는 수가 있기에 .. 내 실수에 여지가 크빈다.위에 백엔드 images, orders 는 일반적 절대적으로 수신한 내용을 보여주는 거이므로 일반적으로 의심할 여지는 없는거 이기에 .. 프론트엔드를 의심하여야 하게쩌.

고등어통조림 · Answer

얼마전에 회사에서 파일첨부 구현했는데 자꾸 파일이 누락된다는 얘길 들었었는데..

고등어통조림 · Answer

코드를 다시 자세히 보니 몇 가지 의심스러운 부분들이 있네요.## 문제 원인 분석### 1. *가장 유력한 원인: open.getId() 사용 시점 문제*```javaopen = classOpenRepository.save(open); // 이때 ID가 생성됨String dir = "class/" + open.getId(); // S3 업로드 경로에 사용for (int i = 0; i < files.size(); i++) {// ...String url = s3Uploader.upload(file, dir); // ← 여기서 동시성 문제 발생 가능// ...}```*S3 업로드가 동시에 실행되면서 같은 경로(class/{id})에 업로드할 때 파일명 충돌이나 업로드 순서가 꼬일 수 있습니다.*### 2. *IntStream.range() 동시 실행 문제*```javaList<Integer> ords = IntStream.range(0, files.size()).boxed().toList();```여러 스레드가 동시에 이 코드를 실행하면서 *JVM 내부적으로 순서가 보장되지 않을 수 있습니다.*

multipart/form-data로 보낼 때 순서 배열 원소값 0을 못 찾겠네요

5개의 답변

기술 목록