Skip to content

Conversation

@mustafapc
Copy link

my solution works on more complex Expressions and Operations than the old one

…plex Expressions and Operations than the old one
@google-cla
Copy link

google-cla bot commented Oct 18, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@gribozavr
Copy link
Collaborator

Sorry, the change looks like a stylistic refactoring to me. What's the motivation?

@gribozavr gribozavr added the waiting for author Waiting on the issue author to reply label Oct 18, 2025
@mustafapc
Copy link
Author

mustafapc commented Oct 18, 2025

@gribozavr the motivation is that the solution used on the tutorial only works on trees that's two or three operation deep and mine can work with complex trees, try a operation for example ((((4 + 7) * (8 - 2)) / ((9 + 14) * (16 - 4))) + 5) or something even deeper than this my solution would work and the one in tutorial won't

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for author Waiting on the issue author to reply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants