Skip to content

Conversation

TonyKim9401
Copy link
Contributor

@TonyKim9401 TonyKim9401 commented Nov 5, 2024

답안 제출 문제

체크 리스트

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

Copy link
Contributor

Choose a reason for hiding this comment

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

안녕하세요 토니님 :)
혹시 이 문제 푸신 걸까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@obzva 헐 다른 문제 풀었네요 세상엨ㅋㅋㅋㅋ
다시 풀어서 올리겠습니다 ㅠㅠㅠ..

@TonyKim9401 TonyKim9401 marked this pull request as ready for review November 8, 2024 15:27
@TonyKim9401 TonyKim9401 requested a review from a team as a code owner November 8, 2024 15:27
@TonyKim9401 TonyKim9401 requested a review from obzva November 9, 2024 03:10
Copy link
Contributor

@bky373 bky373 left a comment

Choose a reason for hiding this comment

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

한주간 고생 많으셨습니다!

@@ -0,0 +1,13 @@
// TC: O(h)
// h = the high of binary search tree
// SC: O(1)
Copy link
Contributor

Choose a reason for hiding this comment

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

스택 메모리를 고려해야 하지 않을까요~?

// only constant space necessary
class Solution {
public int eraseOverlapIntervals(int[][] intervals) {
Arrays.sort(intervals, (o1, o2) -> Integer.compare(o1[1], o2[1]));
Copy link
Contributor

Choose a reason for hiding this comment

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

Comparator 를 다양하게 구현해주셔서 보는 입장에서도 한번 더 살펴볼 수 있어 좋은 것 같습니다!

Copy link
Contributor

@obzva obzva left a comment

Choose a reason for hiding this comment

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

한 주 고생하셨습니다 ㅎㅎ
리더보드 프로젝트 병행하시는 분들은 많이 바쁘시겠어요

@TonyKim9401 TonyKim9401 merged commit 5ca8ce3 into DaleStudy:main Nov 10, 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.

3 participants