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

[강희찬] WEEK 10 Solution #532

Merged
merged 7 commits into from
Oct 18, 2024
Merged

[강희찬] WEEK 10 Solution #532

merged 7 commits into from
Oct 18, 2024

Conversation

HC-kang
Copy link
Contributor

@HC-kang HC-kang commented Oct 13, 2024

답안 제출 문제

체크 리스트

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

@github-actions github-actions bot added the ts label Oct 13, 2024
@HC-kang HC-kang requested a review from wogha95 October 13, 2024 22:35
@HC-kang HC-kang changed the title Feat: 226. Invert Binary Tree [강희찬] WEEK 10 Solution Oct 13, 2024
let right = nums.length - 1;

while (left <= right) {
let mid = left + ((right - left) >> 1);
Copy link
Contributor

Choose a reason for hiding this comment

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

JS/TS 에서는 정수 나눗셈 연산을 이렇게 구현할 수 있군요 :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 이전에는 (left + right) / 2 로 하다가, 최근에 오버플로우도 막고 좀더 있어보이는(?) 방법이라고 해서 따라해봤습니다.

그런데 확인해보니 JS에서는 그냥 Math.floor((left + right) / 2) 쓰는게 낫다고 하네요 ㅋㅋ

Copy link
Contributor

Choose a reason for hiding this comment

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

어떤 이유에서 Math.floor가 더 나은가요?

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. JS 엔진 내부적으로 이미 성능 최적화가 되어서 속도 차이도 별로 나지 않고, 오히려 가독성을 해친다
  2. 부동 소수점을 사용해서, 정밀도가 손실 될지언정 오버플로는 발생하지 않음

요 두개로 기억합니다!
정밀도도 10^15 가까이 큰 수를 다루는게 아니라면 큰 문제가 없었구요

/**
* https://leetcode.com/problems/invert-binary-tree
* T.C. O(n)
* S.C. 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.

재귀호출 스택의 깊이는 트리의 높이에 비례하여 증가하니까 공간 복잡도를 O(N)으로 표기하는 것보다는 O(H)로 표현하는 것이 좀 더 적절할 것 같다는 생각입니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@obzva 안녕하세요 플린님 요즘 잘 지내시죠?
말씀주신 내용도 맞다고 생각합니다!
하지만 최악의 경우, 그러니까 n개의 노드가 극단적으로 치우친 경우, 결국 h=n이 되므로 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.

네 ㅎㅎ 희찬님도 잘 지내시죠?

@HC-kang HC-kang marked this pull request as ready for review October 16, 2024 23:18
@HC-kang HC-kang requested a review from a team as a code owner October 16, 2024 23:18
Comment on lines 12 to 13
* T.C. O(n * k^2) n: average length of list, k: number of lists
* S.C. O(k)
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.

방금 리뷰 드리면서 확인했는데, 저는 n을 각 리스트의 평균 길이로 두고 계산했고, 플린님께서는 전체 노드 수를 N으로 두고 계산하셨더라구요!

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.

엇 공간복잡도였군요
이제보니 제가 잘못 기입한것같아요. 반복호출한다는 생각에 스택을 O(k)로 착각한것같네요
O(1)으로 수정하겠습니다~!

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.

제가 담당 리뷰어는 아니지만, 한 번 리뷰를 마쳤기에 어프루브 하나 놓고 갑니다 :)

@HC-kang HC-kang merged commit 3c980d6 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.

2 participants