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

[TONY] Week 08 Solutions #501

Merged
merged 5 commits into from
Oct 5, 2024
Merged

[TONY] Week 08 Solutions #501

merged 5 commits into from
Oct 5, 2024

Conversation

TonyKim9401
Copy link
Contributor

@TonyKim9401 TonyKim9401 commented Sep 29, 2024

답안 제출 문제

체크 리스트

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

@TonyKim9401 TonyKim9401 changed the title Merge Two Sorted Lists [TONY] Week 07 Solutions Sep 29, 2024
@TonyKim9401 TonyKim9401 changed the title [TONY] Week 07 Solutions [TONY] Week 08 Solutions Sep 29, 2024
@TonyKim9401 TonyKim9401 requested review from HC-kang and wogha95 and removed request for HC-kang September 30, 2024 19:34
@TonyKim9401 TonyKim9401 requested a review from DaleSeo October 3, 2024 23:33
@TonyKim9401 TonyKim9401 marked this pull request as ready for review October 3, 2024 23:33
@TonyKim9401 TonyKim9401 requested a review from a team as a code owner October 3, 2024 23:33
@TonyKim9401 TonyKim9401 requested review from wogha95 and removed request for wogha95 October 3, 2024 23:53
Copy link
Contributor

@wogha95 wogha95 left a comment

Choose a reason for hiding this comment

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

리뷰가 늦어 죄송합니다!

문제 푸는 데 있어 생각을 깊게 하신 부분도 많이 느껴졌고 새로운 관점이나 더 개선할 수 있다는 것을 깨닫게 되었습니다. 특히 8주차 문제들은 저에게 어렵게 느껴졌는데 Tony님 코드를 통해 많이 배울 수 있었습니다.🙇‍♂️

궁금한 점, 느낀 점 위주로 코멘트 남겨드렸고 8주차 문제 푸시느라 고생하셨습니다!!

Comment on lines +1 to +4
// TC: O(n)
// -> visit all elements once for each to clone them
// SC: O(n)
// -> all elements are stored in HashMap and values are limited maximum 2
Copy link
Contributor

Choose a reason for hiding this comment

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

n은 노드의 수를 뜻하는 걸까요? n에 대한 추가 설명이 있으면 좋을 듯 싶어요!

Comment on lines +14 to +16
for (Node neighbor : node.neighbors) {
clone.neighbors.add(cloneGraph(neighbor));
}
Copy link
Contributor

Choose a reason for hiding this comment

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

저는 노드에 연결된 간선의 수만큼 순회를 하게 될 듯 싶어서 TC: O(총 간선수)라고 생각했습니다. 혹시 어떠한 방식으로 시간복잡도를 구하셨는지 추가 설명 가능하실까요??

Comment on lines +3 to +4
// SC: O(n)
// both size of text1 and text2 can be the size of dp
Copy link
Contributor

Choose a reason for hiding this comment

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

일차원배열의 dp를 가지고도 풀 수 있군요!!👍

코드로 이해를 완벽하게 못해서 모임에서 설명을 들어보고 싶어요!

public int characterReplacement(String s, int k) {
int ans = 0;
int n = s.length();
for (char c = 'A'; c <= 'Z'; c++) {
Copy link
Contributor

Choose a reason for hiding this comment

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

오, 각 알파벳의 the longest substring를 구하는 방법도 있군요!
주어진 문자열에서만 해결하려던 제 고정관념을 깨는 느낌이 들었어요😮
좋은 아이디어 감사합니다!!

Comment on lines +3 to +4
// SC: O(n)
// n = node 0 ~ length sum of list1 and list2
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 공간복잡도 구하신 부분 설명 부탁드려도 될까요?

저는 매개변수로 주어진 데이터를 제외하고 추가로 사용된 데이터공간에 대해서 복잡도를 구해서 O(1)로 계산했습니다. @TonyKim9401 님께서도 동일한 로직이라 동일한 공간복잡도가 나올줄 알았으나 O(n)으로 계산하셔서 궁금했습니다!

Comment on lines +21 to +22
if (list1 == null) node.next = list2;
if (list2 == null) node.next = list1;
Copy link
Contributor

@wogha95 wogha95 Oct 5, 2024

Choose a reason for hiding this comment

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

아, 마지막 남은 linked list 연결할때 저는 휴리스틱하게 while문으로 작성했습니다.. 그런데 if문으로 한번만 연결하면 되었군요...

다시 제 코드를 돌아보게 되었습니다 😂
좋은 코드 감사합니다!!

@TonyKim9401 TonyKim9401 merged commit e11c1b2 into DaleStudy:main Oct 5, 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.

2 participants