Skip to content

Trie traversal#7

Open
chamberi wants to merge 112 commits into
masterfrom
trie-traversal
Open

Trie traversal#7
chamberi wants to merge 112 commits into
masterfrom
trie-traversal

Conversation

@chamberi
Copy link
Copy Markdown
Owner

No description provided.

CCallahanIV and others added 30 commits January 16, 2017 12:44
…o bst

sdf building a binary search tree.
…ctures into traversal-bst

g
g Lines starting with '#' will be ignocquire post_order_trav. red, and an empty message aborts
Comment thread src/trie.py
@@ -0,0 +1,93 @@
"""Module for Trie tree."""

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I would not make multi-line comments like this, use the triple """ (that's what they are there for)

Comment thread src/trie.py
class Trie(object):
"""Trie class, which is the Trie tree."""

"""
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

not necessary. Your comments should look like this:
"""Trie class, which is the Trie tree.

insert blah blah
contains blah blah
"""

Comment thread src/trie_trav.py
@@ -0,0 +1,157 @@
"""Module for Trie tree with traversal."""

# TRIE TREE TRAVERSAL (TTT)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

same with these comments

Comment thread src/trie_trav.py
class Trie(object):
"""Trie class, which is the Trie tree."""

"""
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

and here

Comment thread src/trie_trav.py
comparing = False
found = False
stack = []
stack.append(self.root)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

compare, index, comparing, found, stack = '', 0, False, False, [self.root]

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.

3 participants