-
Notifications
You must be signed in to change notification settings - Fork 17
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
[정하윤] sprint3 #32
The head ref may contain hidden characters: "basic-\uC815\uD558\uC724-sprint3"
[정하윤] sprint3 #32
Conversation
}; | ||
|
||
// 이메일 유효성 검사 | ||
if (emailInput) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요소가 있는지 확인하고 실행시키는 방어 코드 좋아요 👍
대신 이렇게 작성하면 요소가 없는 경우 아무 동작도 일어나지 않기 때문에 디버깅이 어려울 수 있어요. 이처럼 반드시 요소가 존재해야 한다면, 요소가 없을 때 에러를 발생시켜 보세요. 추후에 실수로 HTML을 변경함으로써 발생하는 문제에 대한 디버깅이 훨씬 쉬워질 수 있어요.
ex)
const emailInput = document.querySelector('.email-input')
if (!emailInput) {
throw Error('페이지에 이메일 입력 요소가 없습니다.')
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그리고 조건문 블록으로 불필요한 들여쓰기 한번 더 하지 않아도 되죠!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 감사합니다 ㅎㅎ
요구사항
기본
심화
랜딩 페이지
로그인, 회원가입 페이지
주요 변경사항
멘토에게
미션에서는 focus out이 명시되어있어 첫번째 이벤트리스너에는 'focusout'을 썼지만, focus in 상태의 input동작 중에도 이벤트가 발동될 수 있도록 나머지에는 'input'으로 하였습니다. 어떤 방식이 더 UX에 적합한 방법인가요?
�로그인기능과 회원가입기능 파일을 분리해서 만들다가 기능이 겹치는 것 같아 한 파일에 합쳐 작성해보았는데 만들고 나니 오히려 더 가독성이 안 좋을 수도 있다고 생각이 되었습니다. 어떤 방법이 더 나은 방법인지 궁금합니다.
배포 URL