Skip to content

Conversation

SamTheKorean
Copy link
Contributor

No description provided.

@SamTheKorean SamTheKorean changed the title [Sam} Week 2 solutions [Sam] Week 2 solutions May 5, 2024
Copy link
Contributor

@dev-jonghoonpark dev-jonghoonpark left a comment

Choose a reason for hiding this comment

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

LGTM!

@dev-jonghoonpark dev-jonghoonpark added this to the week2 milestone May 5, 2024
@SamTheKorean
Copy link
Contributor Author

LGTM 이 Looks good to me 였군요 ㅎㅎ 덕분에 하나 배웁니다!

@dev-jonghoonpark
Copy link
Contributor

@DaleSeo 라벨과 마일스톤까지는 제가 권한이 부여되어서 설정 가능한데
프로젝트는 선택이 불가능 한 걸로 보이네요 ; )

혹시나 싶어서 기록 남깁니다...!

@DaleSeo
Copy link
Member

DaleSeo commented May 8, 2024

@DaleSeo 라벨과 마일스톤까지는 제가 권한이 부여되어서 설정 가능한데 프로젝트는 선택이 불가능 한 걸로 보이네요 ; )

혹시나 싶어서 기록 남깁니다...!

저장소 쓰기 권한을 드렸으니 이제 되실 거에요 :)

Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

잘 푸셨네요! 👏 Reverse Linked List 풀이가 흥미로워서 피드백 드렸는데 의견 부탁드립니다.

if not node:
continue

node.left, node.right = node.right, node.left
Copy link
Member

Choose a reason for hiding this comment

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

파이썬은 이게 한 줄로 되서 넘 편해요 ㅋㅋ

# Space complexity : O(n)
class Solution:
def reverseList(self, head: Optional[ListNode]) -> Optional[ListNode]:
stack = []
Copy link
Member

Choose a reason for hiding this comment

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

Sam 님은 재귀보다는 스택쓰시는 것을 선호하시는 듯?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

당시에 재귀 보다는 스택이 더 이해가 편했는데 이제 조금씩 재귀적 사고에 익숙해지는 것 같습니다

Copy link
Member

Choose a reason for hiding this comment

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

참고로 저는 개인 선호를 존중해요 ㅋㅋ 물론 둘 다 자유자재로 쓸 수 있다면 더 좋겠지만요 😸

SamTheKorean and others added 2 commits May 9, 2024 19:51
Co-authored-by: Dale Seo <5466341+DaleSeo@users.noreply.github.com>
Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@DaleSeo DaleSeo merged commit a3fc68a into DaleStudy:main May 10, 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.

3 participants