Skip to content

Conversation

forest000014
Copy link
Contributor

@forest000014 forest000014 commented Feb 10, 2025

답안 제출 문제

Week 02

Week 09

Week 10

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@forest000014 forest000014 requested a review from mike2ox February 10, 2025 14:35
@forest000014 forest000014 self-assigned this Feb 10, 2025
@forest000014 forest000014 requested a review from a team as a code owner February 10, 2025 14:35
@github-actions github-actions bot added the java label Feb 10, 2025
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.

안녕하세요 forest님 몸은 좀 괜찮으신가요?
다시 살펴볼 문제들 백로그를 만들어두셨네요 ㅎㅎ 멋집니다
아마 이번 한 주 바쁘셨던 것 같은데, 한 문제라도 푸신 모습 칭찬 드리고 싶습니다
미리 승인 남기고 가겠습니다 고생하셨습니다

Copy link
Contributor

@mike2ox mike2ox left a comment

Choose a reason for hiding this comment

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

1문제라도 풀으신 부분, 리스펙합니다! (저는 1월 2주정도 펑크...ㅎㅎ)
다음주도 화이팅입니다 :)

Comment on lines +33 to +38
invertTree(root.left);
invertTree(root.right);

TreeNode temp = root.left;
root.left = root.right;
root.right = temp;
Copy link
Contributor

Choose a reason for hiding this comment

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

요 부분은 저도 피드백 받았는데 TreeNode의 생성자가 오버로딩 되있기에 이를 활용하면 내부 로직을 엄청 간소화할 수 있더라구요(메모리도 아낄 수 있고). 한번 검토해보시는걸 추천합니다!

@SamTheKorean SamTheKorean merged commit 486a6ef into DaleStudy:main Feb 16, 2025
3 checks 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.

4 participants