🐛 limit length of characters on recipe description #356
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
| import java.util.List; | ||
| import net.pengcook.ingredient.dto.IngredientCreateRequest; | ||
|
|
||
| public record RecipeRequest( | ||
| @NotBlank String title, | ||
| @NotBlank @Size(max = 255) String title, |
There was a problem hiding this comment.
255로 설정한 기준을 함께 공유해주세요!
description 수정이나... comment 등으로...
There was a problem hiding this comment.
새양은 아니지만 제목 칼럼이 데이터베이스에 255자로 제한이 걸려있어서 255자 이상인 제목 넣은 경우에 데이터베이스쪽애서 익셉션 발생해 500에러 발생하는걸 막으려는 목적인것 같습니다.
255자를 칼럼 제한으로 두는건 VARCHAR의 데이터 저장 특성 때문입니다. VARCHAR는 가변길이이기 때문에 문자열의 길이를 추가로 저장해야 합니다. 이때 문자 길이에 따라 1바이트 또는 2바이트의 공간을 차지하는데 1바이트에 최대 저장할 수 있는 수는 255입니다. VARCHAAR(256)부터는 한바이트에 길이를 저장할 수 없어서 2바이트를 써야해서 길이 저장에 한바이트를 써서 효율적이면서 가장 크게 설정 할 수 있는게 255자 입니다.
There was a problem hiding this comment.
@HaiSeong
이 제한 설정은 저장의 효율의 문제에서 시작된 것이 아닌 뷰에서 표시할 수 없는 제목의 길이를 설정하기 위해 시작된 것으로 기억합니다.
그래서 안드로이드 담당자(케이엠)와 함께 길이를 어느정도로 제한해야할지 이야기 나누고 설정해두는 것으로 정리했던 것 같은데 아니었을까요???
There was a problem hiding this comment.
저장의 효율 주제 보다는 앞쪽에 말씀드린 너무 긴 문자가 왔을 경우 막는것을 강조했어야 하는데 뒷쪽 내용이 너무 길어 오해가 생겼네요. 죄송합니다.
그렇지만 화면에 제목 출력 문제를 해결하는 길이 제한 설정은 request dto에서 처리할만한 내용이 아니라고 상각합니다.
There was a problem hiding this comment.
이제 보니 pr 재목은 description 길이설정인데 막상 추가된건 title 관련 설정이네요. 🤔
closes #355