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 15 Solutions #607

Merged
merged 1 commit into from
Nov 24, 2024
Merged

[선재] WEEK 15 Solutions #607

merged 1 commit into from
Nov 24, 2024

Conversation

Sunjae95
Copy link
Contributor

@Sunjae95 Sunjae95 commented Nov 20, 2024

답안 제출 문제

체크 리스트

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

@Sunjae95 Sunjae95 requested a review from TonyKim9401 November 20, 2024 06:04
@Sunjae95 Sunjae95 self-assigned this Nov 20, 2024
@github-actions github-actions bot added the js label Nov 20, 2024
Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

마지막주차 까지 문제 풀이 하시느라 고생 많으셨습니다.
앞으로도 꾸준히 릿코드 하시며 이직시 코딩 테스트를 모두 통과하는 선재님이 되시길 응원하겠습니다!

Comment on lines +14 to +17
if (!findTree(tree.left, target.left)) return false;
if (!findTree(tree.right, target.right)) return false;

return true;
Copy link
Contributor

Choose a reason for hiding this comment

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

return findTree(tree.left, target.left) || findTree(tree.right, target,right);
으로 바꾸면 둘 중 하나면 false라도 false가 반환 될테니 짧게 줄일 수 있을것 같아요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

or연산자를 사용하면 피연산자가 true가 되면 평가하지 않는점을 이용한 풀이 좋은데요?!

Comment on lines +26 to +29
if (preOrder(tree.left)) return true;
if (preOrder(tree.right)) return true;

return false;
Copy link
Contributor

Choose a reason for hiding this comment

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

return preOrder(tree.left) && preOrder(tree.right)
으로 바꾸면 subTree는 무조건 양쪽 모두 값이 같아야 하니 코드를 줄일 수 있을것 같습니다.

Comment on lines +6 to +7
* time complexity: O(n^2)
* space complexity: O(n^2)
Copy link
Contributor

Choose a reason for hiding this comment

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

저는 시간 복잡도는 root의 노드 수인 n, subTree의 노드 수인 m으로 뒀을 때
최악의 경우 n과 m의 노드를 모두 비교해야 해서 O(n * m) 으로,
공간 복잡도의 경우 각 트리의 높이를 h1, h2로 두고
O(h1 +h2)가 소요될것 같은데 의견 부탁드립니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아이고 제가 코멘트를 볼 시간이 없어서 바보같은 질문을 했네요.
이 부분은 스터디에서 소통했기에 스킵하겠습니다!

@EgonD3V
Copy link
Contributor

EgonD3V commented Nov 22, 2024

돌이켜보면 선재님 PR은 항상 주중에 일찍 올라왔던 것 같습니다. 저는 미루다 몰아서 풀어버리니 경험치를 다 못 얻는 것 같아 아쉬웠는데, 15주 동안 꾸준하셨던 것 같아 성실하시다는 생각을 했었습니다. 15주 동안 고생많으셨습니다. 좋은 풀이 감사합니다.

@Sunjae95
Copy link
Contributor Author

@TonyKim9401 @lymchgmk 두분 마지막까지 리뷰와 응원감사합니다.
앞으로도 두분다 좋은 일만 가득하길 바랍니다 :)

@Sunjae95 Sunjae95 marked this pull request as ready for review November 24, 2024 11:07
@Sunjae95 Sunjae95 requested a review from a team as a code owner November 24, 2024 11:07
@Sunjae95 Sunjae95 merged commit c19b630 into DaleStudy:main Nov 24, 2024
3 checks 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