make the comparison methods compare individually to match eq

Author: DavidCEllis

docs/code_examples/outputs/docs_ex08_converters.output

@@ -1,6 +1,8 @@
 ConverterEx(unconverted='42', converted=42)
 Source:
     def __eq__(self, other):
+        if self is other:
+            return True
         return (
             self.unconverted == other.unconverted
             and self.converted == other.converted

docs/code_examples/outputs/docs_ex09_annotated.output

@@ -15,6 +15,8 @@ Slots: {'x': None, 'a': None, 'b': None, 'c': None, 'd': None, 'e': None, 'f': N
 
 Source:
 def __eq__(self, other):
+    if self is other:
+        return True
     return (
         self.x == other.x
         and self.a == other.a

src/ducktools/classbuilder/__init__.py

@@ -486,6 +486,8 @@ def eq_generator(cls, funcname="__eq__"):
     # fmt: off
     code = (
         f"def {funcname}(self, other):\n"
+        f"    if self is other:\n"
+        f"        return True\n"
         f"    return (\n"
         f"        {instance_comparison}\n"
         f"    ) if {class_comparison} else NotImplemented\n"
@@ -497,20 +499,36 @@ def eq_generator(cls, funcname="__eq__"):
 
 
 def get_order_generator(cls, funcname, *, operator):
+    class_comparison = "self.__class__ is other.__class__"
     field_names = [
         name
         for name, attrib in get_fields(cls).items()
         if attrib.compare
     ]
 
-    self_tuple = ", ".join(f"self.{name}" for name in field_names)
-    other_tuple = self_tuple.replace("self.", "other.")
+    # Equal objects should be False for gt/lt comparisons
+    eq_return = "True" if "=" in operator else "False"
+
+    instance_comparisons = [
+        (
+            f"        if self.{name} != other.{name}:\n"
+            f"            return self.{name} {operator} other.{name}\n"
+        )
+        for name in field_names
+    ]
+    instance_comparisons.append(
+        f"        return {eq_return}"
+    )
+
+    instance_comparison = "".join(instance_comparisons)
 
     # fmt: off
     code = (
         f"def {funcname}(self, other):\n"
-        f"    if self.__class__ is other.__class__:\n"
-        f"        return ({self_tuple}) {operator} ({other_tuple})\n"
+        f"    if self is other:\n"
+        f"        return {eq_return}\n"
+        f"    if {class_comparison}:\n"
+        f"{instance_comparison}\n"
         f"    return NotImplemented\n"
     )
     # fmt: on