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

[환미니니] Week1 문제풀이 #325

Merged
merged 12 commits into from
Aug 18, 2024
Merged

Conversation

JEONGHWANMIN
Copy link
Contributor

@JEONGHWANMIN JEONGHWANMIN commented Aug 15, 2024

@JEONGHWANMIN JEONGHWANMIN self-assigned this Aug 15, 2024
@JEONGHWANMIN JEONGHWANMIN marked this pull request as draft August 15, 2024 11:23
@JEONGHWANMIN JEONGHWANMIN marked this pull request as ready for review August 17, 2024 08:56
dfs(root)

return results[k-1]
};
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막에 line break 없는 파일이 메인 브랜치에 유입되지 않도록 각별히 주의바랍니다. (참고: #334)

Copy link
Contributor

@naringst naringst Aug 18, 2024

Choose a reason for hiding this comment

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

해당 문제 이렇게 작성했을 때 올바르게 동작하나요?! root.left가 배열로 나오는 경우들이 있는데, 그런 경우들에는 동작하지 않을 것 같아 질문드립니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

기존에 console.log 삭제하면서 sort가 삭제되어서 없어서 추가했고
leetcode 제출 시 문제는 없었던 거 같아요..!

@gitsunmin gitsunmin self-requested a review August 18, 2024 02:45
@JEONGHWANMIN JEONGHWANMIN merged commit e55594c into DaleStudy:main Aug 18, 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