-
-
Notifications
You must be signed in to change notification settings - Fork 212
[shinsj4653] Week 15 Solutions #1667
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정 코드를 작성하면서 오류 포인트는 이미 충분히 파악하셨겠지만, 혹시라도 다른 분들이 참고하실 수 있도록 함께 정리해두었습니다.
마지막 주차까지 정말 고생 많으셨습니다! 😊
|
||
else: | ||
print('continue compare') | ||
return compare(r.left, sr.left) or compare(r.right, sr.right) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
두 트리가 완전히 동일한지 비교하는 함수에서 or 사용은 부적절
|
||
def compare(r, sr): | ||
print('start compare') | ||
if r is not None and sr.val is not None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sr.val 접근 전에 sr is not None 체크가 먼저 필요
print('not same, exit compare') | ||
return False | ||
|
||
elif r.val == sr.val == None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r.val != sr.val을 이미 처리했기 때문에 작성하지 않아도됨
답안 제출 문제
작성자 체크 리스트
In Review
로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!