Skip to content

Conversation

wogha95
Copy link
Contributor

@wogha95 wogha95 commented Nov 19, 2024

답안 제출 문제

체크 리스트

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

@wogha95 wogha95 added the js label Nov 19, 2024
@wogha95 wogha95 self-assigned this Nov 19, 2024
@wogha95 wogha95 marked this pull request as ready for review November 20, 2024 07:10
@wogha95 wogha95 requested a review from a team as a code owner November 20, 2024 07:10
Copy link
Contributor

@HC-kang HC-kang left a comment

Choose a reason for hiding this comment

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

@wogha95 님 마지막까지 고생 많으셨습니다!!

}

for (let row = 0; row < N; row++) {
matrix[row].reverse();
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

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

이미 충분히 잘 하시지만 앞으로도 더 많은 문제들을 풀어 나가시면서 언제 올지 모르는 좋은 기회가 주어지셨을 때 코딩 테스트가 재호님의 큰 무기가 되셨으면 좋겠습니다.
15주 동안 꾸준히 문제풀이 하시느라 고생 많으셨습니다!

var isSubtree = function (root, subRoot) {
const queue = [root];

while (queue.length > 0) {
Copy link
Contributor

Choose a reason for hiding this comment

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

queue로 푸는건 생각도 못했는데 배워갑니다;

Comment on lines +22 to +23
Number.MIN_SAFE_INTEGER,
Number.MAX_SAFE_INTEGER
Copy link
Contributor

Choose a reason for hiding this comment

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

희찬님의 경우 이 부분을 null로 해서 풀이를 하셨는데, 주어진 범위의 MIN, MAX를 사용하니 코드가 더 깔끔해 지는것 같습니다. null로 했을때 약간 null을 왜 쓸까 라는 한번 더 생각하는 부분도 MIN, MAX의 범위를 넘어가지 않도록 한다는것에 의미가 더 전달되는것 같아 파악하기도 좋은것 같구요!

@EgonD3V
Copy link
Contributor

EgonD3V commented Nov 22, 2024

이전에 리뷰 자세히 적어주셔서 감사했습니다. 역시 마지막 최후의 6인 중 한 분이실 것 같았습니다. 15주 동안 수고 많으셨습니다.

@wogha95 wogha95 merged commit b4b5a70 into DaleStudy:main Nov 24, 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.

4 participants