Skip to content

Conversation

hyer0705
Copy link
Contributor

@hyer0705 hyer0705 commented Oct 10, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

  • 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
  • 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
  • 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

@hyer0705 hyer0705 moved this from Solving to In Review in 리트코드 스터디 5기 Oct 10, 2025
Copy link
Contributor

@delight010 delight010 left a comment

Choose a reason for hiding this comment

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

안녕하세요. 주말동안 감기몸살로 앓아서 늦게 리뷰 다는 점 죄송합니다.
가능하면 시간 복잡도,공간복잡도도 같이 적어주시면 좋을 것 같습니다.
(달래스터디에도 써있는 내용입니다)

Copy link
Contributor

@delight010 delight010 left a comment

Choose a reason for hiding this comment

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

5문제 모두 푸느라 고생 많으셨습니다

Comment on lines +3 to +5
* @param n: the number of vertices
* @param edges: the edges of undirected graph
* @return: the number of connected components
Copy link
Contributor

Choose a reason for hiding this comment

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

세심하게 변수에 대한 설명을 적어놓으신 게 인상 깊습니다.

Comment on lines +16 to +17
if (!p && !q) return true;
if (!p || !q) return false;
Copy link
Contributor

Choose a reason for hiding this comment

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

타입스크립트 문법은 신기하네요. 첫번째는 둘 다 null인지 확인하는 부분, 두번째는 한쪽만 null인지 확인하는 if조건문 맞죠?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 맞습니다~

@hyer0705 hyer0705 merged commit a2f003c into DaleStudy:main Oct 13, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Review to Completed in 리트코드 스터디 5기 Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

2 participants