Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🐛 limit length of characters on recipe description #356

Open
wants to merge 3 commits into
base: be/dev
Choose a base branch
from
Open

🐛 limit length of characters on recipe description #356

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
353e882
Create PR for #355
github-actions[bot] Sep 12, 2024
5082d25
:wrench: add docker compose name
geoje Sep 12, 2024
2ec4548
:recycle: limit recipe title max length to 255
geoje Sep 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion backend/src/main/java/net/pengcook/recipe/dto/RecipeRequest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,12 @@
import jakarta.validation.constraints.Min;
import jakarta.validation.constraints.NotBlank;
import jakarta.validation.constraints.NotEmpty;
import jakarta.validation.constraints.Size;
import java.util.List;
import net.pengcook.ingredient.dto.IngredientCreateRequest;

public record RecipeRequest(
@NotBlank String title,
@NotBlank @Size(max = 255) String title,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

255로 설정한 기준을 함께 공유해주세요!
description 수정이나... comment 등으로...

Copy link

@HaiSeong HaiSeong Sep 22, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

새양은 아니지만 제목 칼럼이 데이터베이스에 255자로 제한이 걸려있어서 255자 이상인 제목 넣은 경우에 데이터베이스쪽애서 익셉션 발생해 500에러 발생하는걸 막으려는 목적인것 같습니다.

255자를 칼럼 제한으로 두는건 VARCHAR의 데이터 저장 특성 때문입니다. VARCHAR는 가변길이이기 때문에 문자열의 길이를 추가로 저장해야 합니다. 이때 문자 길이에 따라 1바이트 또는 2바이트의 공간을 차지하는데 1바이트에 최대 저장할 수 있는 수는 255입니다. VARCHAAR(256)부터는 한바이트에 길이를 저장할 수 없어서 2바이트를 써야해서 길이 저장에 한바이트를 써서 효율적이면서 가장 크게 설정 할 수 있는게 255자 입니다.

Copy link
Contributor

@hyxrxn hyxrxn Sep 22, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HaiSeong
이 제한 설정은 저장의 효율의 문제에서 시작된 것이 아닌 뷰에서 표시할 수 없는 제목의 길이를 설정하기 위해 시작된 것으로 기억합니다.
그래서 안드로이드 담당자(케이엠)와 함께 길이를 어느정도로 제한해야할지 이야기 나누고 설정해두는 것으로 정리했던 것 같은데 아니었을까요???

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저장의 효율 주제 보다는 앞쪽에 말씀드린 너무 긴 문자가 왔을 경우 막는것을 강조했어야 하는데 뒷쪽 내용이 너무 길어 오해가 생겼네요. 죄송합니다.

그렇지만 화면에 제목 출력 문제를 해결하는 길이 제한 설정은 request dto에서 처리할만한 내용이 아니라고 상각합니다.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이제 보니 pr 재목은 description 길이설정인데 막상 추가된건 title 관련 설정이네요. 🤔

@NotBlank String cookingTime,
@NotBlank String thumbnail,
@Min(0) @Max(10) int difficulty,
Expand Down
1 change: 1 addition & 0 deletions backend/src/main/resources/.env
View file
Open in desktop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

갑자기 추가한 이유가 뭔가요?!

Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
COMPOSE_PROJECT_NAME=pengcook
Loading