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.

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

refactor: 닉네임 띄어쓰기 허용/미허용 #570

Open
2 of 5 tasks
BurningFalls opened this issue Dec 18, 2024 · 3 comments
Open
2 of 5 tasks

refactor: 닉네임 띄어쓰기 허용/미허용 #570

BurningFalls opened this issue Dec 18, 2024 · 3 comments
Assignees
Labels
android We are android>< backend We are backend>< confirm need confirmation! refactor 리팩토링 (변수 및 메서드 네이밍 변경)
Milestone

Comments

@BurningFalls
Copy link
Contributor

BurningFalls commented Dec 18, 2024

🤮 As Is

  • 띄어쓰기를 넣어서 닉네임을 만들고 싶다는 사용자 피드백이 있었습니다.
    닉네임에 띄어쓰기를 허용할지 미허용할지에 대해서 논의해보고 싶습니다.

  • 올바르지 않은 닉네임 형식입니다.라고 메시지가 떠서, 닉네임의 어떤 부분이 잘못된 것인지 알기가 힘듭니다.
    처음에 닉네임에는 띄어쓰기가 포함될 수 없습니다.와 같은 메시지를 생각했으나, 모든 예외에 대해서 일일이 메시지를 만들면 관리 포인트가 너무 많아진다고 느꼈습니다.
    아래 사진의 인스타그램 UI 예시처럼, 닉네임에는 ~~만 사용할 수 있습니다.와 같이 메시지 구체화가 필요해보입니다.
    해당 빨간색 글씨는 처음에는 없다가, 이름 규정을 어긴채로 적어서 '다음' 버튼을 누르면, '다음' 버튼이 살짝 내려가면서 중간에 메시지가 표시됩니다.

  • 아예 닉네임 입력 칸에 띄어쓰기 입력 자체를 불가능하게 만드는 방법도 생각해볼 수 있을 것 같습니다.
    (띄어쓰기를 입력하면 아무 일도 일어나지 않음)
    이 방안을 선택하면, 위에서 언급한 띄어쓰기에 대한 예외 메시지 구체화를 할 필요가 없어집니다.
    그럼에도 불구하고 닉네임의 다른 조건이 있다면, 그 조건들에 대한 예외 메시지 구체화는 필요해보입니다.

🤩 To Be

논의 후 결정

😇 이때까지 끝낼게요!

논의 후 결정

🙇‍♀️이슈 확인했어요:)

@BurningFalls BurningFalls added android We are android>< backend We are backend>< refactor 리팩토링 (변수 및 메서드 네이밍 변경) labels Dec 18, 2024
@BurningFalls BurningFalls added this to the sprint-7 milestone Dec 18, 2024
@BurningFalls BurningFalls self-assigned this Dec 18, 2024
@BurningFalls BurningFalls added the confirm need confirmation! label Dec 18, 2024
@linirini
Copy link
Contributor

닉네임에 띄어쓰기를 허용할지 미허용할지에 대해서 논의해보고 싶습니다.

띄어쓰기 허용/미허용 여부는 저는 다수결 따르겠습니다 ㅎㅎ 다만, 띄어쓰기 사용은 허용하더라도 닉네임의 시작과 끝에 있는 공백 제거 정책은 유지하면 좋을 것 같습니다.

아래 사진의 인스타그램 UI 예시처럼, 닉네임에는 ~~만 사용할 수 있습니다.와 같이 메시지 구체화가 필요해보입니다.

동의합니다.! 서버 측에서 메세지를 수정하면 좋을 것 같네요:)

아예 닉네임 입력 칸에 띄어쓰기 입력 자체를 불가능하게 만드는 방법도 생각해볼 수 있을 것 같습니다.

띄어쓰기를 불가능하게 하는 것보단 띄어쓰기를 포함했을 때 예외 메시지(닉네임에는 ~~만 사용할 수 있습니다.)에서 확인하는 것이 사용자 입장에서 가시적일 것 같습니다.

@BurningFalls
Copy link
Contributor Author

닉네임의 시작과 끝에 있는 공백 제거 정책은 유지하면 좋을 것 같습니다.

동의합니다! 시작과 끝에 있는 n개의 공백은 제거하는 것이 좋아보여요.

띄어쓰기를 불가능하게 하는 것보단 띄어쓰기를 포함했을 때 예외 메시지(닉네임에는 ~~만 사용할 수 있습니다.)에서 확인하는 것이 사용자 입장에서 가시적일 것 같습니다.

저도 리니가 말해준 방식(인스타그램과 똑같은 방식)이 좀더 나은 것 같습니다.
띄어쓰기만 입력 못하게 막고, 다른 예외는 안막으면서 예외 메시지로 표시해주면, 통일성도 깨지는 것 같아요.

@Junyoung-WON
Copy link
Contributor

Junyoung-WON commented Dec 18, 2024

닉네임에 띄어쓰기 허용 / 미허용

저는 띄어쓰기를 허용해도 좋을 것 같아요!
인스타그램처럼 아이디의 형식이 아니므로 작명에 대한 자유도가 있다고 생각합니다. 그래서 띄어쓰기를 사용할 수 없도록 제약을 둘 이유는 없어보여요.

메시지 구체화

인스타그램 UI 예시처럼, "닉네임에는 ~~만 사용할 수 있습니다." 와 같이 메시지 구체화가 필요해보입니다.

저도 동의합니다!
서버 측에서 예외 메시지를 구체화하는 것도 좋을 것 같고,
또는 클라이언트 측에서 검증 로직을 추가하여 처리를 해도 좋을 것 같아요.
위의 경우라면 닉네임을 검증하기 위해 굳이 서버에게 요청을 보내지 않아도 되며, 사용자에게 즉각적으로 피드백을 줄 수 있다는 장점이 있습니다.
이 부분은 의견을 제시해서 어느 구현이 좋을지 이야기를 나누어보고 싶어요!

서버에서 예외 메시지를 구체화하던, 클라이언트에서 검증을 수행하던,
인스타그램의 UI 예시처럼 텍스트 필드의 테두리 색상을 빨갛게 변경하고, 에러 메시지를 텍스트 필드 아래에 나타내는 것이 가시성이 좋아보여요!
사용자 입력(텍스트 필드)에 대해서는 입력한 위치에 에러를 나타내는 것이 사용자에게 더 눈에 띄고, 확실한 피드백을 줄 것 같아요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android We are android>< backend We are backend>< confirm need confirmation! refactor 리팩토링 (변수 및 메서드 네이밍 변경)
Projects
Status: No status
Development

No branches or pull requests

3 participants