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

Erica #3

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

Erica #3

wants to merge 1 commit into from

Conversation

norrise120
Copy link

I need some help!! Please see comment in code!

Copy link

@CheezItMan CheezItMan left a comment

Choose a reason for hiding this comment

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

Yeah, it's a good start, but like you noted not working yet see my comment below.

# is that I should be moving through the dislikes as they appear instead of just
# linearly. However, for whatever when I tried using a stack and popping off the
# values, it didnt alter my stack--meaning the stack grew in length and I was
# trapped in a loop.

Choose a reason for hiding this comment

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

Yeah, you are going to need a stack or queue to traverse the graph so that you can keep consistent that you going toggling between the two "colors" or groups while on a path.

Your code here isn't traversing a graph, but instead iterating through the list of nodes. Take a look at either my breadth-first-search or depth-first-search solutions.

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