[선재] WEEK09 Solution #515
Merged
[선재] WEEK09 Solution #515
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wogha95
approved these changes
Oct 12, 2024
Comment on lines
+24
to
+29
| if (answer < nums[nums.length - 1]) return answer; | ||
|
|
||
| for (let i = nums.length - 1; i >= 0; i--) { | ||
| if (answer < nums[i]) return answer; | ||
| answer = nums[i]; | ||
| } |
There was a problem hiding this comment.
answer < nums[nums.length - 1] 조건문이 for문 안에 있는 if문에 포함되어 제거 가능해보입니다!
There was a problem hiding this comment.
오호?! 그러네요 중복되는 로직이었군요..!
감사합니다.
| * | ||
| * n = length of height | ||
| * m = length of height[i] | ||
| * time complexity: O(n*m * 4^n*m) |
There was a problem hiding this comment.
상하좌우에 대한 4방향 접근으로 인해 4^n*m으로 계산하신걸까요?
visited 를 사용해서 memoization 하니 각 방향이n*m만큼 탐색일까 하는 의문이 듭니다!
혹시 가능하시다면 시간 복잡도 계산 과정 설명 부탁드립니다!
There was a problem hiding this comment.
안녕하세요. @wogha95 님! 리뷰 코멘트가 늦어진점 죄송합니다🥲
코멘트에 대한 리뷰는 다음과 같아요!
pacific이며 atlantic인 경우에만 memization이 되도록 구현했어요.
그래서 각 인덱스마다 접근시 memization이 안되는 경우가 존재하며 최악의 상황의 시간복잡도는 O(nm * 4^nm) 으로 계산했어요.
추가로 visited는 전역에서 사용하는 변수이지만 해당 bfs 스코프에서 다룰수 있게 값을 설정하여 매번 4방향을 탐색하도록 동작하도록 구현했어요.
그래서 제가 계산한 시간복잡도는 O(nm * 4^nm) 인데 이해가 됐을까요?
@Sunjae95 님, Week 설정 좀 까먹지 말고 좀 부탁드립니다 🥲
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
답안 제출 문제
체크 리스트
In Review로 설정해주세요.