Details
-
Improvement
-
Status: Open
-
Minor
-
Resolution: Unresolved
-
None
-
None
-
None
Description
A frequent anti-pattern in the Phoenix code base is the use of huge anonymous inner classes, often in the midst of functions that are already very large, or even the parameter list of constructor calls. (The latter means that we have single statements that are technically hundreds of lines long!)
This makes the logic difficult to follow and prevents abstraction. Modern IDEs make changing these to named inner classes quite simple.
This is intended as an umbrella item with subtasks refactoring particular classes, which can be added to over time.