-
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 #39
The head ref may contain hidden characters: "basic-\uC724\uBBFC\uD638-mission03"
[윤민호] sprint3 #39
Conversation
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.
열심히 해주신 것 같습니다! 수고하셨어요.
전반적으로 오타가 많고, naming convention 이 잘 지켜지지 않아 코드 가독성이 떨어집니다! 또한 함수나 변수 명명도 조금 아쉬워요. 짧아서 무슨 변수인지 모르는 것 보다는 차라리 긴 변수명이 낫습니다.
코드를 직접 수정해보시면서 JS 레벨업 해보시길 바랍니다.
const EmPattern = /^[A-Za-z0-9_\.\-]+@[A-Za-z0-9\-]+\.[A-za-z0-9\-]+/; | ||
const PwdPattern = /^(?=.*[A-Za-z])(?=.*\d)[A-Za-z\d]{8,}$/; | ||
|
||
const valigadeLength = (length) => length >= 8; |
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.
validate 오타가 난 거 같아요
const signupbox = document.querySelector("#signup"); | ||
const signup = () => signupbox.style.background = '#3692FF'; | ||
|
||
const EmPattern = /^[A-Za-z0-9_\.\-]+@[A-Za-z0-9\-]+\.[A-za-z0-9\-]+/; |
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.
emailPattern 의 camelCase 로 사용해주시는 게 좋겠습니다!
순수 JS 변수명은 camelCase, (처음은 소문자, 이후 단어가 바뀔 때 마다 대문자)
리액트 등에서 컴포넌트 명 같은 경우 PascalCase (첫 글자도 대문자)
로 사용하는 컨벤션이 있습니다.
이에 맞게 수정해보세요.
const EmailPatterncheck = (email) => EmPattern.test(email); | ||
const PwdPatterncheck = (pwd) => PwdPattern.test(pwd); | ||
const PwdSame = () => pwd.value === pwdcheckout.value; | ||
const TureLine = (e) => e.target.style.outline = '1px solid #3692FF'; |
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.
true, false 오타
pwd : false, | ||
} | ||
|
||
function executeIfAllTrue(obj, callback) { |
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.
checkIfAllTrue 정도가 좋을 거 같고, checkIfAllValid 가 좀 더 좋겠습니다.
email.addEventListener("focusout", function (e) { | ||
if(EmptyValue(e)){ | ||
FalesLine(e); | ||
EmailTextEmptyToggle(0); |
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.
파라미터에 0, 1 이런식으로 들어가니까 0, 1 이 뭘 의미하는 지 정확히 알기가 어렵습니다.
다시 짜보세요!
배포링크
https://pada-save3.netlify.app/
기본
심화
랜딩 페이지
로그인, 회원가입 페이지
스크린샷
멘토에게