-
-
Notifications
You must be signed in to change notification settings - Fork 304
[hongseoupyun] WEEK 01 Solutions #1971
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
Conversation
|
@hongseoupyun 님, 안녕하세요? 아직 기수가 시작되지 않았는데 벌써 답안을 제출해주셨네용? |
|
아 달레님 제가 시작일을 착각해서,, 저도 방금 pr 넣고 알았습니다.. ㅎㅎ
죄송합니다..😅
2025년 11월 5일 (수) 오후 9:42, Dale Seo ***@***.***>님이 작성:
… *DaleSeo* left a comment (DaleStudy/leetcode-study#1971)
<#1971 (comment)>
@hongseoupyun <https://github.com/hongseoupyun> 님, 안녕하세요? 아직 기수가 시작되지
않았는데 벌써 답안을 제출해주셨네용?
—
Reply to this email directly, view it on GitHub
<#1971 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVKJFHOOK3HVE7ZSWYSILNL33KYSFAVCNFSM6AAAAACLI7CHACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIOJUGYYDENZWGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
…y in twoSum implementations
|
@DaleStudy 리뷰해줘 |
|
이 코드는 Two Sum 문제에 대한 효율적인 해시맵 기반 솔루션으로, 시간 복잡도 O(n)과 공간 복잡도 O(n)을 잘 명시하고 있습니다. 알고리즘 설명도 명확해서 이해하기 쉽고, 변수명도 직관적입니다. 가독성을 위해 몇 가지 제안을 드리자면, 예를 들어 추가로, 코드에 시간/공간 복잡도를 명시하는 주석을 넣어주셨는데, 이 내용이 적절하고 정확합니다. 더 발전시키려면, 혹시라도 다른 접근법(예: 정렬 후 이진 검색)을 고려할 수 있지만, 해시맵 솔루션이 가장 최적에 가깝습니다. 전체적으로 아주 깔끔하고 이해하기 쉬운 코드입니다. 계속해서 이런 명료성을 유지하며, 변수명을 조금 더 직관적으로 다듬는 것도 추천드립니다. 훌륭한 작업입니다! |
TonyKim9401
left a comment
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.
1주차 문제 풀이 고생하셨습니다!
온보딩 및 스터디 적응 하시는데 앞으로 조금 더 힘내주시고 앞으로의 15주간 여정 응원합니다.
2주차 문제 풀이도 파이팅입니다!
답안 제출 문제
작성자 체크 리스트
In Review로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!