[thispath98] Week 1 #660
Merged
[thispath98] Week 1 #660
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
DaleSeo
approved these changes
Dec 10, 2024
| class Solution: | ||
| def topKFrequent(self, nums: List[int], k: int) -> List[int]: | ||
| count_dict = Counter(nums) | ||
| top_k_list = count_dict.most_common(k) |
There was a problem hiding this comment.
실제 코딩 인터뷰에서 다른 프로그래밍 언어에서 흔히 볼 수 없는 표준 라이브러리를 사용할 경우 해당 언어를 잘 모르는 면접관을 만날 경우 양날의 검으로 작용할 수 있으니 주의바라겠습니다.
There was a problem hiding this comment.
넵 감사합니다! 사실 sort를 이용해서 구현했었는데, Counter 메소드를 사용하는 게 성능이 더 좋아서 첨부했습니다. 다음부터는 여러 시행착오도 업로드해보겠습니다 ㅎㅎ
| num_dict = {} | ||
| for num in nums: | ||
| if num not in num_dict: | ||
| num_dict[num] = True |
There was a problem hiding this comment.
사전에 저장하는 값의 종류가 한 가지인데, 이런 경우에는 세트를 쓰는 게 더 적합하지 않을까 생각이 들었습니다.
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로 설정해주세요.