-
Notifications
You must be signed in to change notification settings - Fork 98
문제 풀이 추가 #65
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
문제 풀이 추가 #65
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.
좋은 코드 잘 보았습니다! 함수형도 잘 활용하시고 문법적으로 뛰어나시네요!
주석, 파일명이나 리뷰 남긴 부분 확인 부탁드립니다. 리뷰가 늦어서 죄송합니다 :)
level-2/k-진수에서-소수-개수-구하기.js
Outdated
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.
멋진 풀이네요! 해설 상단에 다른 파일들과 같이 주석 부탁드립니다 :)
level-2/방금-그곡.js
Outdated
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.
풀이 잘 보았습니다! 파일명을 '[3차]-방금그곡' 으로 변경하고, 해설 상단에도 주석을 추가해주세요!
level-2/방금-그곡.js
Outdated
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.
+finish 와 같은 구문은 문자를 숫자로 변환하기 위한 것인가요? 명시적이지 않은 타입 변환으로 보여서 혼란이 있을 수 있을 것 같다고 생각합니다 :)
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.
해당 식에 괄호가 많아져서 Number 함수를 사용하지 않았었습니다. 말씀 듣고 코드를 확인해보니 산술 연산자들 때문에 +와 같은 암묵적 타입 변환이 코드를 읽는데 번잡스러울 것 같습니다. 명시적으로 타입 변환하게끔 수정해두었습니다.
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.
해당 해설에도 주석 부탁드립니다!
@minjongbaek 코드에 대한 리뷰는 아니지만, 복잡한 PR 등 없이 바로 코드만 제보하실 수 있는 사이트를 제작중입니다. 사용가능해지면 바로 멘션드리겠습니다 감사합니다 :)
주석 추가 + 대신 Number() 함수로 형변환하도록 수정
리뷰 감사합니다. 커밋 하기전 봤던 몇몇 파일이 하필 주석이 없는 파일이라 주석을 누락했습니다. 🥲
주석 추가하여 다시 커밋하였으니 확인 부탁드리겠습니다.
완벽하네요 👍 머지하고 contributor 에도 추가해드리겠습니다!
Lv. 2