This repository was archived by the owner on Mar 18, 2024. It is now read-only.
[2023-09-25] sumin #267#279
Merged
Merged
Conversation
Closed
zsmalla
approved these changes
Sep 25, 2023
Contributor
zsmalla
left a comment
There was a problem hiding this comment.
여기서는 조건 만족 여부를 따로 빼진 않으셨군요 😂 사실 만족 여부가 간단해서 이 구조도 보기에 깔끔해서 전혀 이해가 어렵지 않았습니다! 고생하셨습니다 수민님!
Contributor
There was a problem hiding this comment.
피드백 드릴게 이런 거 밖에 없지만.. 파이썬으로 코테 볼때는 pythonic하게 짜는 것도 중요하다고 한 것 같아서 이런 부분은 tot += mid if city > mid else city 이런 식으로 짜려고 노력하는 편입니다!
Contributor
Author
There was a problem hiding this comment.
조건대로 짜고 최적화를 안해서 그런가봐요😂 담부턴 좀 더 파이써닉하게 최적화하겠습니당 :)
Woo-Yeol
approved these changes
Sep 25, 2023
Member
Woo-Yeol
left a comment
There was a problem hiding this comment.
대체적인 풀이 내용이 유사하여 큰 피드백은 없습니다. 다만 마지막 right를 반환하는 부분이 지수님의 풀이와 유사하여서 지수님께 질문을 남겼었는데 다시 한 번 천천히 생각해보니 최대 값을 찾기 위해서 right를 반환하는 방법이 효율적일 것 같다는 것을 깨달았네요! 감사합니다 수민님!!
limstonestone
approved these changes
Sep 25, 2023
Contributor
limstonestone
left a comment
There was a problem hiding this comment.
고생하셨습니다람쥐!!
문제 설명도 잘해주셔서 감사합니당 ㅎ
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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
PR Summary
풀이시간 : 20분
<input>
<solution>
<시간복잡도>
O(nlogn)