Skip to content
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

[똘치] Week1 #667

Merged
merged 9 commits into from
Dec 15, 2024
Merged

[똘치] Week1 #667

merged 9 commits into from
Dec 15, 2024

Conversation

easyone-jwlee
Copy link
Contributor

@easyone-jwlee easyone-jwlee commented Dec 9, 2024

답안 제출 문제

체크 리스트

  • PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@easyone-jwlee easyone-jwlee requested a review from a team as a code owner December 9, 2024 22:38
@github-actions github-actions bot added the go label Dec 9, 2024
@HC-kang
Copy link
Contributor

HC-kang commented Dec 9, 2024

안녕하세요 @easyone-jwlee 님!
프로젝트와 주차 세팅까지 모두 확실하게 해주셔서 좋네요!

CI 실패 케이스는 어제 저녁 linelint 깃허브 워크플로우가 수정되어서 최신화가 필요한 상태인데요, 최신 main 브랜치로 rebase 해 주시면 해결될것으로 보입니다!

@easyone-jwlee
Copy link
Contributor Author

안녕하세요 @easyone-jwlee 님! 프로젝트와 주차 세팅까지 모두 확실하게 해주셔서 좋네요!

CI 실패 케이스는 어제 저녁 linelint 깃허브 워크플로우가 수정되어서 최신화가 필요한 상태인데요, 최신 main 브랜치로 rebase 해 주시면 해결될것으로 보입니다!

넵. 감사합니다.

@easyone-jwlee easyone-jwlee requested review from sungjinwi and removed request for sungjinwi December 10, 2024 23:06
@SamTheKorean
Copy link
Contributor

안녕하세요! https://github.com/DaleStudy/leetcode-study/pull/676/files pr처럼 시간 복잡도 및 공간 복잡도 분석도 주석으로 남겨주시면 코딩테스트 연습에 도움이 많이 되실 것 같습니다 ㅎㅎ

@easyone-jwlee
Copy link
Contributor Author

안녕하세요! https://github.com/DaleStudy/leetcode-study/pull/676/files pr처럼 시간 복잡도 및 공간 복잡도 분석도 주석으로 남겨주시면 코딩테스트 연습에 도움이 많이 되실 것 같습니다 ㅎㅎ

넵! 조언해주셔서 감사합니다. 추가하면서 풀이해보겠습니다.

Comment on lines +7 to +10
// 순회하는 map안에서 for문을 또 호출하긴 하지만,
// 모든 값이 연속되는 값이라고 했을 때
// 연속이 시작되는 값 외에는 한 번씩 바로 지나가게 되고(n*1), 시작되는 값부터 연속이 끝나는 시점까지 n번이라(1*n)
// O(n+n) 이기 때문에 TC는 O(n)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이중 for문은 n^2라는 생각만 가지고 있었는데 조건문으로 제한해서 시간복잡도를 개선한 점이 신기했어요!

Copy link
Contributor

@sungjinwi sungjinwi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요! 알고리즘 공부는 처음인데다가 C언어로 막 공부 시작하는 중이라 배우는 느낌으로 리뷰했습니다
복잡도 계산 / 개선사항이나 대안도 적어주셔서 풀이하면서 다양한 고민을 하신 걸 느꼈어요
풀이를 적어주셨지만 제가 go lang은 처음 접해서코드 중에 궁금한 부분이 있어서 질문 드렸어요!
수고하셨습니다

top-k-frequent-elements/easyone-jwlee.go Show resolved Hide resolved
@SamTheKorean
Copy link
Contributor

SamTheKorean commented Dec 14, 2024

리뷰어님이 좋은 리뷰 남겨주셨는데 병합을 위해 승인은 해놓겠습니다!ㅎㅎ 추후에라도 리뷰 확인해보시면 도움이 될 것 같습니다! ㅎㅎ 이번 주 고생많으셨습니다!
@easyone-jwlee

@SamTheKorean SamTheKorean merged commit 2399863 into DaleStudy:main Dec 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants