Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Python

Code challenge morse.py

Maybe its my syntax, any help is greatly appreciated.

morse.py
class Letter:
    def __init__(self, pattern=None):
        self.pattern = pattern

    def __str__(self):
        Morse = []
        for i in patterns:
            if i == ".":
                Morse.append("dot")
            elif i == "_":
                Morse.append("dash")
        return "-".join(Morse)

class S(Letter):
    def __init__(self):
        pattern = ['.', '.', '.']
        super().__init__(pattern)

1 Answer

for i in patterns is the problem. patterns doesn't exist. You named your variable on top in singular: pattern not patterns. Also you forgot to access it with self. Finally two suggestions, lowercase the variable Morse as titlecase is usually reserved for classes and the second suggestion being for letter in pattern reads better than for i in pattern as usually single letters such as i are used as a "counter" variable and you can be more descriptive than that for better readability.

Thank you Pedro