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 Object-Oriented Python Dice Roller Compare and Contrast

George Clement
George Clement
2,816 Points
Posted by George Clement
George Clement
2,816 Points

not sure what I'm missing

I think I have it providing the length back as an int, and then I've put the comparisons in the class .. not sure what I'm missing

songs.py 
class Song:
    def __init__(self, artist, title, length):
        self.artist = artist
        self.title = title
        self.length = length

    def __int__(self):
        return self.length

    def __eq__(self, other):
        return int(self) eq int(other):

    def __ne__(self, other):
        return int(self) != int(other):

    def __gt__(self, other):
        return int(self) > int(other):

    def __lt__(self, other):
        return int(self) < int(other):

    def __ge__(self, other):
        return int(self) > int(other) or int(self) eq int(other) :

    def __le__(self, other):
        return int(self) < int(other) or int(self) eq int(other) :

2 Answers

Jeff Muday
MOD
Jeff Muday
Treehouse Moderator 28,717 Points
Jeff Muday
Jeff Muday
Treehouse Moderator 28,717 Points

Nice work! You have the right idea, there were some minor syntax errors in your code -- extraneous colons and replacing pseudocode operators with Python primitive operators. I cleaned it up the code and it passed for me.

Good luck with Python, it's a fun and useful language.

class Song:
    def __init__(self, artist, title, length):
        self.artist = artist
        self.title = title
        self.length = length

    def __int__(self):
        return int(self.length)

    def __eq__(self, other):
        return int(self) == int(other)

    def __ne__(self, other):
        return int(self) != int(other)

    def __gt__(self, other):
        return int(self) > int(other)

    def __lt__(self, other):
        return int(self) < int(other)

    def __ge__(self, other):
        return int(self) > int(other) or int(self) == int(other)

    def __le__(self, other):
        return int(self) < int(other) or int(self) == int(other)
Mark Chesney
Mark Chesney
11,747 Points
Mark Chesney
Mark Chesney
11,747 Points

Hi George,

Yes it's quite close. I'd say the main focus area is your int method, which needs the int() function:

    def __int__(self):
        return int(self.length)

Also, your use of eq returns a syntax error in my Python console. I'd use ==

Great job. I'm learning this stuff too, and it's not easy to remember, so I refer to the teacher's notes frequently.