Dependent functions are far apart Long Method variable declared far from its usage Code Duplication Too many arguments (>3) Comments Explaining Code Unpronounceable naming large functions with nested structures Use of pun Error codes instead of exception Violates Command- Query Separation Non- searchable variable name Catching exceptions and logging only Requires horizontal scrolling vertical openness between concepts Deep Nesting Switch Statement Abuse Too Many Responsibilities Multiple Return Points Missing summary comment on a public method Commented Out Code missing legal comments Message Chains – a.b().c().d() trainwreck Confusing Naming Dependent functions are far apart Long Method variable declared far from its usage Code Duplication Too many arguments (>3) Comments Explaining Code Unpronounceable naming large functions with nested structures Use of pun Error codes instead of exception Violates Command- Query Separation Non- searchable variable name Catching exceptions and logging only Requires horizontal scrolling vertical openness between concepts Deep Nesting Switch Statement Abuse Too Many Responsibilities Multiple Return Points Missing summary comment on a public method Commented Out Code missing legal comments Message Chains – a.b().c().d() trainwreck Confusing Naming
(Print) Use this randomly generated list as your call list when playing the game. There is no need to say the BINGO column name. Place some kind of mark (like an X, a checkmark, a dot, tally mark, etc) on each cell as you announce it, to keep track. You can also cut out each item, place them in a bag and pull words from the bag.
Dependent functions are far apart
Long Method
variable declared far from its usage
Code Duplication
Too many arguments (>3)
Comments Explaining Code
Unpronounceable naming
large functions with nested structures
Use of pun
Error codes instead of exception
Violates Command-Query Separation
Non-searchable variable name
Catching exceptions and logging only
Requires horizontal scrolling
vertical openness between concepts
Deep Nesting
Switch Statement Abuse
Too Many Responsibilities
Multiple Return Points
Missing summary comment on a public method
Commented Out Code
missing legal comments
Message Chains – a.b().c().d() trainwreck
Confusing Naming