-
-
Notifications
You must be signed in to change notification settings - Fork 303
[Sbeo-Joe] WEEK 01 solutions #1975
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
|
@DaleStudy 리뷰해줘 |
|
전체적으로 깔끔하게 문제별 해결책이 잘 정리되어 있습니다. 특히 두 번째 풀이부터는 효율적인 알고리즘을 사용하여 성능 향상도 기대됩니다. 몇 가지 피드백을 드리자면:
전반적으로 깔끔하고 문제별 핵심 해결책이 잘 드러납니다. 각 솔루션에 복잡도 표기를 추가하고, 더 최적화할 수 있는 부분을 고려하면 완성도가 더욱 높아질 것입니다. 계속해서 좋은 코드 기대할게요! |
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주차 문제 풀이도 파이팅입니다!
| @@ -0,0 +1,26 @@ | |||
| class Solution { | |||
| public: | |||
| int map[101]; | |||
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.
minor) memo라는 이름을 사용하면 변수의 목적을 더 쉽게 확인할 수 있을 것 같아요!
| public: | ||
| int longestConsecutive(vector<int>& nums) { | ||
| if(nums.size() == 0) return 0; | ||
| set<int> us{nums.begin(), nums.end()}; |
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.
us라는 변수명이 좀 더 의미를 가지면 좋을 것 같습니다!
답안 제출 문제
작성자 체크 리스트
In Review로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!