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

12 tree #12

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

12 tree #12

wants to merge 17 commits into from

Conversation

jaeeunHwang
Copy link
Collaborator

@jaeeunHwang jaeeunHwang commented Nov 22, 2023

인적사항

학번 : 2176427
이름 : 황재은

과제 제출

기존 제출 : 15681, 5639
추가 제출 : 3190

Copy link

@sawoll sawoll left a comment

Choose a reason for hiding this comment

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

15681번 코드리뷰 완료했습니다. 로직도 좋고 주석도 적절히 잘 달아주셔서 따로 코멘트 드릴 부분은 없었습니다. 수고 많으셨습니다.

Comment on lines +52 to +55
// 전위 순회 결과로 이진 트리 생성
while (cin >> value) {
root = insertNode(root, value);
}
Copy link

Choose a reason for hiding this comment

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

이진 트리를 먼저 만든 후에 후위 순회를 하는 방법으로 구현해 주셨습니다.👍 이 문제는 전위순회순서로부터 후위 순회 순서를 구하기만 하면 되므로, 이에 집중하여 전위 순회 순서를 입력받은 후 함수에서 이진 트리의 특성을 이용해서 후위 순회 순서를 바로 구하도록 할 수도 있답니다! 이 점도 고려해주시면 좋을 것 같아요!

Copy link

@sawoll sawoll left a comment

Choose a reason for hiding this comment

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

[12주차 트리 코드리뷰 완료]
5639(P3)
안녕하세요 재은님! 트리 필수 2문제 코드리뷰 완료했고 3190번 추가제출까지 확인 완료했습니다. 수고 많으셨습니다.😍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants