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

[choidabom] Week 4 #837

Merged
merged 5 commits into from
Jan 4, 2025
Merged

[choidabom] Week 4 #837

merged 5 commits into from
Jan 4, 2025

Conversation

choidabom
Copy link
Contributor

@choidabom choidabom commented Jan 3, 2025

답안 제출 문제

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@choidabom choidabom added the ts label Jan 3, 2025
@choidabom choidabom self-assigned this Jan 3, 2025
@choidabom choidabom requested a review from a team as a code owner January 3, 2025 13:07
@choidabom choidabom requested a review from sungjinwi January 3, 2025 13:07
Copy link
Contributor

@obzva obzva left a comment

Choose a reason for hiding this comment

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

한 주 수고하셨습니다 :)

@@ -0,0 +1,45 @@
/**
* Runtime: 0ms, Memory: 52.30MB
*
Copy link
Contributor

Choose a reason for hiding this comment

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

안녕하세요 상단에 Big O 분석 추가해주시면 더 좋을 것 같습니다 :)

@obzva
Copy link
Contributor

obzva commented Jan 4, 2025

@sungjinwi 담당 PR 리뷰도 진행해주시길 바랍니다
참여자 역할 참고

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.

접근 방법을 써놓으셔서 코드 진행을 비교하면서 읽으니 어떻게 진행되는지 훨씬 이해가 쉬운것 같아요
추가적인 풀이 같이 정리해놓는 것도 나중에 스스로 리뷰할 때 여러가지 시각으로 볼 수 있는 것 같아 좋습니다

* 접근
* 직관적으로 생각했을 때, 0부터 n까지의 숫자 중에서 없는 숫자를 찾아야 한다.
* 완전 탐색으로 정렬한 배열에서 순서대로 비교하면서 없는 숫자를 찾을 수 있다.
* Time Complexity: 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.

ts의 sort()가 어떤 정렬을 사용하는지는 모르지만 보통 nlogn 시간복잡도를 가지고 있다고 알고 있습니다
한번 확인해보시면 좋을것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

V8엔진을 사용하는 경우에 sort() 메서드는 Timsort 알고리즘을 사용한다고 합니다. timsort는 합병 정렬이랑 삽입 정렬을 결합한 알고리즘이네요. 배열을 분할하는 과정에서 O(logn)의 시간 복잡도가 나오고 병합, 비교에서 O(n)의 시간 복잡도라 말씀하신 O(nlogn) 시간 복잡도가 나오는게 맞습니다.
놓친 부분 잘 짚고 넘어갑니다. 리뷰 감사합니다 ~

@choidabom choidabom merged commit b4f2b30 into DaleStudy:main Jan 4, 2025
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.

3 participants