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 09 Solutions #516

Merged
merged 5 commits into from
Oct 13, 2024
Merged

[재호] WEEK 09 Solutions #516

merged 5 commits into from
Oct 13, 2024

Conversation

wogha95
Copy link
Contributor

@wogha95 wogha95 commented Oct 6, 2024

답안 제출 문제

체크 리스트

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

@wogha95 wogha95 added the js label Oct 6, 2024
@wogha95 wogha95 self-assigned this Oct 6, 2024
@wogha95 wogha95 marked this pull request as ready for review October 12, 2024 01:53
@wogha95 wogha95 requested a review from a team as a code owner October 12, 2024 01:53
@wogha95 wogha95 requested a review from Sunjae95 October 12, 2024 01:54
Copy link
Contributor

@Sunjae95 Sunjae95 left a comment

Choose a reason for hiding this comment

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

안녕하세요. 재호님 리뷰완료했습니다.
처음 리뷰드린거같은데 전체적으로 코드흐름이 가독성이 좋아서 리뷰어로써 보기좋았습니다.
특히 pacific-atlantic-water-flow에서 각 대양을 세팅하는 방법을 어떻게 풀지 궁금했었는데 좋았습니다.
이번주 고생하셨습니다!

@@ -0,0 +1,37 @@
/**
* TC: O(log 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)으로 풀지않고 다른방법이 있을거라 생각했는데 이진탐색을 하면 시간복잡도를 줄일수 있군요!

@@ -0,0 +1,36 @@
/**
* Floyd tortoise and hare 알고리즘을 바탕으로
Copy link
Contributor

Choose a reason for hiding this comment

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

오호..! 처음보는 알고리즘인데 리뷰하면서 알게됐네요 감사합니다👍

@wogha95 wogha95 merged commit 3ac0528 into DaleStudy:main Oct 13, 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