Python: Это хороший способ переопределить __eq__ и __hash__?

Я новичок в Python, и я хотел убедиться, что я правильно переопределл __eq__ и __hash__, чтобы позднее не вызывать болезненных ошибок:

(Я использую Google App Engine.)

class Course(db.Model):
    dept_code = db.StringProperty()
    number = db.IntegerProperty()
    title = db.StringProperty()
    raw_pre_reqs = db.StringProperty(multiline=True)
    original_description = db.StringProperty()

    def getPreReqs(self):
        return pickle.loads(str(self.raw_pre_reqs))

    def __repr__(self):
        title_msg = self.title if self.title else "Untitled"
        return "%s %s: %s" % (self.dept_code, self.number, title_msg)

    def __attrs(self):
        return (self.dept_code, self.number, self.title, self.raw_pre_reqs, self.original_description)

    def __eq__(self, other):
        return isinstance(other, Course) and self.__attrs() == other.__attrs()

    def __hash__(self):
        return hash(self.__attrs())

Несколько более сложный тип:

class DependencyArcTail(db.Model):
    ''' A list of courses that is a pre-req for something else '''
    courses = db.ListProperty(db.Key)

    ''' a list of heads that reference this one '''
    forwardLinks = db.ListProperty(db.Key)

    def __repr__(self):
        return "DepArcTail %d: courses='%s' forwardLinks='%s'" % (id(self), getReprOfKeys(self.courses), getIdOfKeys(self.forwardLinks))

    def __eq__(self, other):
        if not isinstance(other, DependencyArcTail):
            return False

        for this_course in self.courses:
            if not (this_course in other.courses):
                return False

        for other_course in other.courses:
            if not (other_course in self.courses):
                return False

        return True

    def __hash__(self):
        return hash((tuple(self.courses), tuple(self.forwardLinks)))

Все выглядит хорошо?

Обновлено, чтобы отразить комментарии @Alex

class DependencyArcTail(db.Model):
    ''' A list of courses that is a pre-req for something else '''
    courses = db.ListProperty(db.Key)

    ''' a list of heads that reference this one '''
    forwardLinks = db.ListProperty(db.Key)

    def __repr__(self):
        return "DepArcTail %d: courses='%s' forwardLinks='%s'" % (id(self), getReprOfKeys(self.courses), getIdOfKeys(self.forwardLinks))

    def __eq__(self, other):
        return isinstance(other, DependencyArcTail) and set(self.courses) == set(other.courses) and set(self.forwardLinks) == set(other.forwardLinks)

    def __hash__(self):
        return hash((tuple(self.courses), tuple(self.forwardLinks)))

Ответ 1

Первый - это хорошо. Вторая проблема проблематична по двум причинам:

могут быть дубликаты в .courses
два объекта с идентичным .courses, но разные .forwardLinks будут сравнивать одинаковые, но имеют разные хэши

Я бы исправил второй, сделав равенство зависимым от обоих курсов и прямых ссылок, но оба изменения в наборах (следовательно, не дубликаты), и то же самое для хэширования. То есть:.

def __eq__(self, other):
    if not isinstance(other, DependencyArcTail):
        return False

    return (set(self.courses) == set(other.courses) and
            set(self.forwardLinks) == set(other.forwardLinks))

def __hash__(self):
    return hash((frozenset(self.courses), frozenset(self.forwardLinks)))

Это, конечно, предполагает, что прямые ссылки имеют решающее значение для объекта "реальное значение", в противном случае они должны быть опущены как из __eq__, так и __hash__.

Изменить: удалено из __hash__ вызовов tuple, которые были в лучшем случае избыточными (и, возможно, повреждающими, как это было предложено комментарием @Mark [[tx!!!]]); изменил set на frozenset в хэшировании, как это было предложено комментарием @Phillips [[tx!!!]].