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

Counting Full Nodes in a Binary Tree #415

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aarzoopundir
Copy link

@aarzoopundir aarzoopundir commented Oct 13, 2021

Issue Id you have worked upon -

#351

Briefly explain your program logic -

Full Nodes are the ones whose left and right child are not null. The idea is to traverse the tree in postorder. If the current node is full, we increment result by 1 and add returned values of left and right subtrees.

Screenshots(Attach 2 screenshots of your own input and output) -

Screenshot (122)
Screenshot (121)


Checklist:

Eg - If your code follow the below guidelines. Kindly change [] to [x]

All the conditions should be fulfilled for considering your code for merging -

  • I have mentioned the question as comment in my solution file.

  • My code follows the guidelines of this project.

  • I have performed a self-review of my own code.

  • I have commented my code.

  • My code gives the correct output.

  • I confirm that I have not copied the code from anywhere. In case its found that I have copied even after successful merge then I can be banned from the repository and hacktoberfest.

  • I affirm that I strictly follow contributing guidelines and code of conduct.

@SarthakKeshari
Copy link
Owner

@aarzoopundir,
Kindly create an issue first and then PR. And follow the PR template and contribution guidelines.
I am not closing this PR since this was your first time. You may refer CONTRIBUTING.md(present in the repository) and create an issue(iff program has not been done in repositories probable folders).

If above given lines follow, then kindly create an issue now and claim it. I will assign it to you.

@SarthakKeshari
Copy link
Owner

@aarzoopundir,
Kindly follow the guidelines. This is the last warning.

@aarzoopundir
Copy link
Author

aarzoopundir commented Oct 13, 2021 via email

@SarthakKeshari
Copy link
Owner

@aarzoopundir,
PR template is not filled properly. Kindly go through other PRs to get a clear idea

@aarzoopundir
Copy link
Author

aarzoopundir commented Oct 13, 2021 via email

@aarzoopundir
Copy link
Author

aarzoopundir commented Oct 13, 2021 via email

@SarthakKeshari
Copy link
Owner

@aarzoopundir,
You haven't filled the Issue ID, Screenshots section and checklist. Kindly go through other PRs to see how other contributors are doing it.

@aarzoopundir
Copy link
Author

aarzoopundir commented Oct 13, 2021 via email

@SarthakKeshari
Copy link
Owner

@aarzoopundir,
Nothing has been updated

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