Optimization- blocking for- loop over a collection stored in a field Gigantic switch‑case w/ strings Start(), Start2(), Start3(), StartReal(), StartReal2() Virtual methods that should have been interfaces 24 directions are strings Banana and Voucher workaround is still there Status effects are still traits Undisclosed exceptions Switch statement on direction string realName, realName2, fakeActive Inefficient vector math X and Y instead of Vector2 Loop- switch sequence isAgent, isItem, isFire, isBullet, isObjectReal fields Locked doors are still one- directional Unrelated code A method with 10+ parameters Vehicle is a subclass of Agent ——— OR ——— Agents ARE vehicles Enormous if-chain Data clumps Random- Selection is still ridiculous Dozens of incredibly specific fields Expensive string concatenation in a loop Relationships still use both strings and enums sprite, spriteH, spriteWB, spriteWBH tk2d Binary serialization Improper list population w/ excessive copying Custom localization problems Optimization- blocking for- loop over a collection stored in a field Gigantic switch‑case w/ strings Start(), Start2(), Start3(), StartReal(), StartReal2() Virtual methods that should have been interfaces 24 directions are strings Banana and Voucher workaround is still there Status effects are still traits Undisclosed exceptions Switch statement on direction string realName, realName2, fakeActive Inefficient vector math X and Y instead of Vector2 Loop- switch sequence isAgent, isItem, isFire, isBullet, isObjectReal fields Locked doors are still one- directional Unrelated code A method with 10+ parameters Vehicle is a subclass of Agent ——— OR ——— Agents ARE vehicles Enormous if-chain Data clumps Random- Selection is still ridiculous Dozens of incredibly specific fields Expensive string concatenation in a loop Relationships still use both strings and enums sprite, spriteH, spriteWB, spriteWBH tk2d Binary serialization Improper list population w/ excessive copying Custom localization problems
(Print)
Optimization-blocking for-loop over a collection stored in a field
Gigantic switch‑case w/ strings
Start(), Start2(), Start3(), StartReal(), StartReal2()
Virtual methods that should have been interfaces
24 directions are strings
Banana and Voucher workaround is still there
Status effects are still traits
Undisclosed exceptions
Switch statement on direction string
realName, realName2, fakeActive
Inefficient vector math
X and Y instead of Vector2
Loop-switch sequence
isAgent, isItem, isFire, isBullet, isObjectReal fields
Locked doors are still one-directional
Unrelated code
A method with 10+ parameters
Vehicle is a subclass of Agent ——— OR ——— Agents ARE vehicles
Enormous if-chain
Data clumps
Random-Selection is still ridiculous
Dozens of incredibly specific fields
Expensive string concatenation in a loop
Relationships still use both strings and enums
sprite, spriteH, spriteWB, spriteWBH
tk2d
Binary serialization
Improper list population w/ excessive copying
Custom localization problems
