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

[Tony] WEEK 10 Solutions #534

Merged
merged 5 commits into from
Oct 18, 2024
Merged

[Tony] WEEK 10 Solutions #534

merged 5 commits into from
Oct 18, 2024

Conversation

TonyKim9401
Copy link
Contributor

@TonyKim9401 TonyKim9401 commented Oct 14, 2024

답안 제출 문제

체크 리스트

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

@github-actions github-actions bot added the java label Oct 14, 2024
@TonyKim9401 TonyKim9401 changed the title Invert Binary Tree [Tony] WEEK 10 Solution Oct 14, 2024
@TonyKim9401 TonyKim9401 changed the title [Tony] WEEK 10 Solution [Tony] WEEK 10 Solutions Oct 14, 2024
@TonyKim9401 TonyKim9401 requested a review from obzva October 14, 2024 17:37
Comment on lines +3 to +4
// SC: O(n)
// -> create all nodes again to exchange
Copy link
Contributor

Choose a reason for hiding this comment

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

공간복잡도 분석에서 저와 생각이 다르신 것 같습니다
invertTree 함수 내에서 left, right를 선언하여 root.left = right; root.right = left 해주는 일련의 과정을 creating all nodes라고 볼 순 없을 것 같습니다
이 부분은 root.left와 root.right의 레퍼런스만 바꿔주는 것이기 때문에 공간복잡도 고려에 포함되지 않아도 될 것 같다는 생각입니다

이 풀이는 재귀 함수 호출을 이용하고 있으므로 위에서 언급한 부분 대신 재귀 호출 스택을 공간복잡도 고려에 포함시키는게 좋을 것 같습니다.

Copy link
Contributor Author

@TonyKim9401 TonyKim9401 Oct 18, 2024

Choose a reason for hiding this comment

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

@obzva 의견 감사합니다! 제가 설명을 오해의 소지를 불러 일으키도록 적어뒀네요!
좋은 분석 너무 감사합니다.
제가 분석한 공간 복잡도의 케이스는 크게 2가지 입니다.
1. 균형 잡힌 트리, 2. 편향된 트리
1. 균형 잡힌 트리 의 경우 재귀호출로 O(log n)만큼 공간을 차지한다고 생각되구요.
2. 편향된 트리 의 경우 모든 트리를 다 조회하고 만들어야 하기 때문에 O(n)의 공간복잡도가 소요될거라 생각합니다.
따라서, 최악의 경우 평향된 트리를 생각했을때 O(n) 의 공간복잡도가 나올 거라 생각됩니다.

@TonyKim9401 TonyKim9401 requested review from obzva and HC-kang October 18, 2024 04:39
Copy link
Contributor

@HC-kang HC-kang left a comment

Choose a reason for hiding this comment

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

@TonyKim9401 님 이번 한 주도 고생 많으셨습니다!

current = current.next;

if (minNode.next != null) {
pq.offer(minNode.next);
Copy link
Contributor

Choose a reason for hiding this comment

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

오.. 우선순위 큐가 정말 좋네요. 신기하고 간단한 풀이네요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

사실 리스트가 1 ~ n 까지 있으면 1, 2 머지 -> (1,2머지), 3머지 이런식으로 풀었는데 시간이 너무 오래 걸리더라구요..
다른 풀이를 살짝 참고하여 풀었습니다. 덕분에 시간 복잡도가 확 줄었네요!

@TonyKim9401 TonyKim9401 marked this pull request as ready for review October 18, 2024 23:38
@TonyKim9401 TonyKim9401 requested a review from a team as a code owner October 18, 2024 23:38
@TonyKim9401 TonyKim9401 merged commit cd10345 into DaleStudy:main Oct 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

3 participants