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

BST 구현 #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

passwd10
Copy link
Member

@passwd10 passwd10 commented Jan 1, 2021

No description provided.

Copy link
Collaborator

@gyim1345 gyim1345 left a comment

Choose a reason for hiding this comment

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

코드 엄청 깔끔하게 짜셨네요! 수고하셨습니다!

}

searchNode(root, value) {
if (root.data < value) {
Copy link
Collaborator

@gyim1345 gyim1345 Jan 1, 2021

Choose a reason for hiding this comment

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

data 랑 value가 같은 type?것?을 가르키는 것 같은데 혹시 연관이 잘 안되는(저만 그렇게 느낄수도??ㅋㅋ) 두 변수 이름을 쓰시는 이유가 있을까요?

Copy link
Collaborator

@daadaadaah daadaadaah left a comment

Choose a reason for hiding this comment

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

이번 코드 너무 읽기 편했습니다 👍👍👍👍👍

깔끔한 코드 굿굿!!!

@Woomin-Jeon Woomin-Jeon removed their request for review September 6, 2023 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants