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

[bky373] Solutions for week 2 problems #50

Merged
merged 5 commits into from
May 11, 2024
Merged

Conversation

bky373
Copy link
Contributor

@bky373 bky373 commented May 7, 2024

  • 20. Valid Parentheses
  • 206. Reverse Linked List -> 공간 복잡도를 O(1) 로 할 수도 있었네요.. (주중에 새로 풀이해보고 개선해보겠습니다~)
  • 226. Invert Binary Tree
  • 21. Merge Two Sorted Lists -> 흠.. 더 좋은 방법이 있을 것 같은데 이것도 개선할 수 있으면 해보겠습니다.
  • 141. Linked List Cycle

@bky373 bky373 changed the title [bky373] Solutions for week 2 problems [WIP][bky373] Solutions for week 2 problems May 7, 2024
@DaleSeo DaleSeo added the week2 label May 8, 2024
@dev-jonghoonpark dev-jonghoonpark added this to the week2 milestone May 8, 2024
}
return a;
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막 라인에 line break가 안걸렸네요

이거 찾아보시면 에디터나 IDE에 자동으로 삽입해주는 옵션 있으실꺼에요...!

Copy link
Contributor Author

@bky373 bky373 May 8, 2024

Choose a reason for hiding this comment

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

아 네 감사합니다~ 지금 설정 추가해서 푸시하겠습니다 (완료!)

@SamTheKorean
Copy link
Contributor

푼 문제도 보면서 지속적으로 복기하는 자세를 본받겠습니다!

Copy link
Contributor

@SamTheKorean SamTheKorean left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@DaleSeo
Copy link
Contributor

DaleSeo commented May 10, 2024

@bky373 PR이 아직 draft 상태인데, 다 푸셨다면 "Read for review" 버튼을 클릭해주시면 감사하겠습니다.

@SamTheKorean SamTheKorean marked this pull request as ready for review May 10, 2024 18:05
@SamTheKorean
Copy link
Contributor

SamTheKorean commented May 10, 2024

비활성화 되어있는 줄 알았는데 실수로 눌렀네요.. 문제 보니까 다 푸신 것 같아 승인은 해놓았습니다!

Comment on lines +11 to +19
int min = -10001;

while (head.next != null) {
if (head.next.val == min) {
return true;
}
head.val = min;
head = head.next;
}
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.

네네 저걸로 방문 기록을 대신 했습니다 ㅋㅋ 다른 분들 풀이 보니 어떻게 저렇게 생각하셨을까.. 감탄했습니다

}

Object[] arr = nums.toArray();
Arrays.sort(arr);
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)이 될까요? 🤔

Copy link
Contributor Author

@bky373 bky373 May 11, 2024

Choose a reason for hiding this comment

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

아 제 풀이에서는 처음에 모든 node 를 돌아야 하기 때문에 O(N) 으로 작성하였습니다
찾아보니 Arrays.sort() 는 평균 시간 복잡도가 O(nlog(n)) 이고, 최악의 경우 O(n^2) 까지도 나오는군요..
현재 풀이가 만족스러운 풀이는 아니어서 한 번 더 풀게 되면 좀더 개선해보려 합니다!

@bky373
Copy link
Contributor Author

bky373 commented May 11, 2024

@bky373 PR이 아직 draft 상태인데, 다 푸셨다면 "Read for review" 버튼을 클릭해주시면 감사하겠습니다.

문제 풀이를 개선한 후에 상태를 변경하려 했는데, 개인적인 일로 예상보다 변경이 늦어졌네요 😭
다음에는 미리 오픈해서 리뷰를 일찍 받아보는 것도 좋을 것 같습니다 감사합니다!

@bky373
Copy link
Contributor Author

bky373 commented May 11, 2024

비활성화 되어있는 줄 알았는데 실수로 눌렀네요.. 문제 보니까 다 푸신 것 같아 승인은 해놓았습니다!

네네 다행히 문제는 다 풀어둔 상황이라 괜찮습니다~ 승인해주셔서 감사합니다!

@SamTheKorean SamTheKorean merged commit 4e6204f into DaleStudy:main May 11, 2024
@bky373 bky373 changed the title [WIP][bky373] Solutions for week 2 problems [bky373] Solutions for week 2 problems May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants