3 ms·
Again this is over-engineering. Put functions in modules to namespace them and be done with it. If you really want types, use classes. It removes the need for
by BiteCode_dev 7y ago
Again this is over-engineering. Put functions in modules to namespace them and be done with it.
If you really want types, use classes. It removes the need for single dispatch, creates a self documenting namespace and saves you an import when you want to do the calculation.
import math
from dataclasses import dataclass
@dataclass
class Rectangle:
length: int
breadth: int
def area(self) -> int:
return self.length * self.breadth
@dataclass
class Circle:
radius: int
def area(self) -> float:
return math.pi * self.radius ** 2
@dataclass is of course not mandatory, and just make the code shorter.
We are talking about very simple things here. No need to add complexity to them.
- anentropic 7y agoYes I agree, if I was writing this for my own project I'm pretty sure I would do what you have done and use classes with an `area` method. (FWIW I think the dataclass approach is nicer for this example than namespacing separate `area` functions using modules) You could define it as a typing.Protocol too. My point was just that, even if for some reason you prefer function overloading, the restrictions needed to fit within the constraints of functool.singledispatch arguably lead to cleaner code than the complicated mess in the OP article. This would look messy if you implemented as just a single function with variable no of kwargs that you sniffed in the body, and I don't think applying function overloading alone as OP has done solves the underlying problem.
- BiteCode_dev 7y ago> My point was just that, even if for some reason you prefer function overloading, the restrictions needed to fit within the constraints of functool.singledispatch arguably lead to cleaner code than the complicated mess in the OP article. Indeed. Less is more sometimes.
- jakearmitage 7y agoDidn't know about @dataclass until now. Thank you.
- BiteCode_dev 7y agoPython 3.7+ only.
- oefrha 7y agoSee also attrs http://www.attrs.org/ http://www.attrs.org/ which is one of the main inspirations of py37 dataclass and is more feature-rich.