Feature/chung step4 #11

kochungcheon · 2024-02-28T06:42:29Z

구현 기능

폴더 요약

폴더 요약에는 다음 정보가 담겨 있습니다

폴더 명
- 사용자가 설정한 폴더 명을 폴더 요약 API 요청 시 보냅니다
생성 날짜
- 생성 날짜는 최초 생성 시 반영되며 변경 되지 않습니다
수정 날짜
- 수정 날짜의 경우, 하위 폴더의 과업이 발생하면 바뀌게 됩니다

루트 폴더 요약

루트 폴더 요약에는 다음 정보가 담겨 있습니다

루트 폴더 pk
폴더 이름
- 사용자가 설정한 폴더 명을 API 요청 시 보냅니다
가용 용량
- 폴더 총 용량으로 현재 프로젝트에서는 2GB 로 고정입니다
사용 용량
- 스토리지 사용 용량이고 전달 단위는 GB (소수점 6째 자리에서 반올림)로 클라이언트에 전달합니다.
여유 용량
- API 요청 시 마다 계산(가용 용량 - 사용 용량)해서 클라이언트에게 전달합니다.

루트 폴더를 만든 이유

루트 폴더는 최상위 폴더이자 상품으로 가용 용량에 대한 정보를 지니고 있습니다. 가용 용량은 하위 파일의 수정 때마다, 변경이 되어야합니다. 그렇기에 일반 폴더와 테이블을 분리하여 관리하고자 하였습니다. 루트 폴더 테이블에서만 찾으면 되서 불필요한 데이터를 찾지 않아도 되기 때문입니다.

루트 폴더 메타데이터

필드 설명
id : pk
origianlFolderName : 사용자가 루트 폴더에 설정한 이름입니다
storedFolderName : 서버에 저장될 루트 폴더의 이름입니다 UUID + 날짜로 만들어집니다.
availableSpace : 가용 공간입니다. 현재 프로젝트에선, 2GB로 고정되어있습니다.
usedSpace : 폴더에 업로드된 파일 총 용량입니다
메서드 설명
storedName : 서버에 저장될 폴더 명을 생성합니다. UUID + 날짜로 생성합니다
increaseUsedSpace : 사용 용량을 증가 시킵니다. 루트 폴더 하위에 파일 업로드 과업이 발생할 시 호출 됩니다. 가용 용량을 초과하는 업로드가 발생하게 될 시, EXCEEDED_CAPACITY(HttpStatus.INSUFFICIENT_STORAGE, "더 이상 업로드 할 수 없습니다") 를 던집니다
decreaseUsedSpace : 사용 용량을 줄입니다. 파일 및 폴더 삭제 과업이 발생할 시 호출됩니다. 사용 용량 보다 더 많이 줄이게 될 경우 INVALID_OPERATION(HttpStatus.BAD_REQUEST, "사용 중인 공간보다 많은 공간은 해제할 수 없습니다")를 던집니다
calRemainSpace : 잔여 용량을 계산 합니다. 루트 폴더 요약 API 요청 시 호출됩니다.
인덱스
조회 로직에 인덱스를 걸었습니다

@Table(indexes = {
    @Index(name = "index_id_ownerId", columnList = "id, ownerId"),
    @Index(name = "index_storedFolderName", columnList = "storedFolderName"),
    @Index(name = "index_ownerId_originalFolderName", columnList = "ownerId, originalFolderName")
})

기존 코드 변경 사항

MetadataType 제거

파일, 폴더를 구분 하던 MetadataType을 제거 했습니다. DBMS 에서 파일과 폴더는 서로 다른 테이블에서 관리되기 때문에, 별도로 구분을 더 할 필요는 없다는 판단이였습니다.

MINE 타입 도입 (FileType)

파일을 비디오, 오디오, 도큐먼트, 이미지, 그 외로 분류 하였습니다.

- add @RequestBody

- 루트 폴더 사용 용량 업데이트 메서드와 파일 메타데이터 저장 메서드를 하나의 트랜잭션으로 묶기 위함

- Cast one of the operands of this multiplication operation to a long

>> - Folder 에 있던 Type 을 삭제하였습니다. 테> 블을 통해 폴더와 파일이 식별 되기에 Type으로 한번 더 구분하는 것은 메모리 낭비라 판단 되서 입니다. >> - File 은 기존 File 로 통칭 했던 것에서 이미지, 비디오, 오디오, 도큐먼트 그리고 그외로 분류하였습니다

VSFe

논리 자체는 되게 괜찮은 논리였던 것 같습니다.
온라인으로 설명했던 부분 반영해서 수정하시고, 다음 Step 진행하시면 됩니다~

VSFe · 2024-05-09T13:58:17Z

src/main/java/com/c4cometrue/mystorage/file/FileController.java

+    }
+
+    @GetMapping
+    @ResponseStatus(HttpStatus.NO_CONTENT)


statusCode 좀 이상하네요

VSFe · 2024-05-09T14:00:05Z

src/main/java/com/c4cometrue/mystorage/file/FileMetadata.java

+    @Column(nullable = false)
+    private Long uploaderId;
+    private Long parentId;
+    private BigDecimal sizeInBytes;


VSFe · 2024-05-09T14:01:33Z

src/main/java/com/c4cometrue/mystorage/file/FileService.java

-	public void uploadFile(MultipartFile file, Long userId, Long parentId) {
-		String basePath = folderService.findPathBy();
+    @Value("${file.buffer}")
+    private int bufferSize;


VSFe · 2024-05-09T14:02:44Z

src/main/java/com/c4cometrue/mystorage/file/dto/FileDeleteRequest.java

-	public static FileDeleteRequest of(long fileId, long userId) {
-		return new FileDeleteRequest(fileId, userId);
-	}
+        @NotNull(message = "파일 id는 null 될 수 없습니다") long fileId,


Validation을 넣을거면 @Positive도 넣는게 좋지 않나 싶어요

- - NotNull -> Positive 로 교체 - 오탈자 수정 mine -> mime

sonarcloud · 2024-05-12T14:59:59Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
91.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

kochungcheon added 14 commits February 16, 2024 23:34

feat: 파일 요약

add90fd

feat: rootFolder 생성

9ad3ddd

style : LF 정리

9fd41ee

refactor : 파일 업로드, 폴더 삭제 시 루트 폴더 사용 용량 변화

996a328

feat : 루트 폴더 요약

fab9b39

refactor : 폴더 삭제 로직 시 사용 용량 변화 반영

b57b305

chore: jpa method name 수정

bce2a77

refactor : form data -> json

6ec329b

- add @RequestBody

bug : 파일 바이트 저장 누락 및 폴더 업로드 경로 변경

55f207e

bug : 타인 폴더에 접근 문제 해결

238a78b

chore : 메서드 위치 수정

72981f5

refactor : private uploadFile 생성

c98af22

- 루트 폴더 사용 용량 업데이트 메서드와 파일 메타데이터 저장 메서드를 하나의 트랜잭션으로 묶기 위함

refactor : index 생성

ad9a5a1

test : 폴더 요약 테스트 코드 추가

dcf34c1

kochungcheon added the enhancement New feature or request label Feb 28, 2024

kochungcheon requested review from VSFe, semi-cloud, pear96 and MilanoBeer February 28, 2024 06:42

kochungcheon self-assigned this Feb 28, 2024

kochungcheon added 3 commits February 28, 2024 15:53

refactor : sonar 반영

16c0daa

- Cast one of the operands of this multiplication operation to a long

chore : 테스트 시나리오 삭제

55d791e

VSFe approved these changes May 9, 2024

View reviewed changes

kochungcheon added 4 commits May 12, 2024 21:03

chore: 코드 스타일 정리

da8b928

chore: 코드 스타일 정리

4591a3c

chore: 코드 사용하지 않은 import 정리

4127034

refactor: PR 반영

1689962

- - NotNull -> Positive 로 교체 - 오탈자 수정 mine -> mime

kochungcheon merged commit 0415c16 into base/chung May 12, 2024
2 checks passed

kochungcheon deleted the feature/chung_step4 branch May 12, 2024 15:13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/chung step4 #11

Feature/chung step4 #11

kochungcheon commented Feb 28, 2024 •

edited

Loading

VSFe left a comment

VSFe May 9, 2024

VSFe May 9, 2024

VSFe May 9, 2024

VSFe May 9, 2024

sonarcloud bot commented May 12, 2024

Feature/chung step4 #11

Feature/chung step4 #11

Conversation

kochungcheon commented Feb 28, 2024 • edited Loading

구현 기능

폴더 요약

루트 폴더 요약

루트 폴더를 만든 이유

루트 폴더 메타데이터

기존 코드 변경 사항

MetadataType 제거

MINE 타입 도입 (FileType)

VSFe left a comment

Choose a reason for hiding this comment

VSFe May 9, 2024

Choose a reason for hiding this comment

VSFe May 9, 2024

Choose a reason for hiding this comment

VSFe May 9, 2024

Choose a reason for hiding this comment

VSFe May 9, 2024

Choose a reason for hiding this comment

sonarcloud bot commented May 12, 2024

Quality Gate passed

kochungcheon commented Feb 28, 2024 •

edited

Loading