From: Victor Stinner Date: Mon, 19 Dec 2016 12:09:28 +0000 (+0100) Subject: doc: Suggest to hash(tuple of attr) rather than XOR X-Git-Tag: v3.5.3rc1~45 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=509476b37085abda9c8cc0fba1451cb035e0b442;p=python doc: Suggest to hash(tuple of attr) rather than XOR Issue #28383: __hash__ documentation recommends naive XOR to combine but this is suboptimal. Update the doc to suggest to reuse the hash() method using a tuple, with an example. --- diff --git a/Doc/reference/datamodel.rst b/Doc/reference/datamodel.rst index b763f10eeb..73eec1c557 100644 --- a/Doc/reference/datamodel.rst +++ b/Doc/reference/datamodel.rst @@ -1305,11 +1305,14 @@ Basic customization Called by built-in function :func:`hash` and for operations on members of hashed collections including :class:`set`, :class:`frozenset`, and - :class:`dict`. :meth:`__hash__` should return an integer. The only - required property is that objects which compare equal have the same hash - value; it is advised to somehow mix together (e.g. using exclusive or) the - hash values for the components of the object that also play a part in - comparison of objects. + :class:`dict`. :meth:`__hash__` should return an integer. The only required + property is that objects which compare equal have the same hash value; it is + advised to mix together the hash values of the components of the object that + also play a part in comparison of objects by packing them into a tuple and + hashing the tuple. Example:: + + def __hash__(self): + return hash((self.name, self.nick, self.color)) .. note::