Long Method Violates Command- Query Separation Commented Out Code Requires horizontal scrolling Comments Explaining Code Non- searchable variable name Error codes instead of exception Code Duplication Too Many Responsibilities Catching exceptions and logging only Too many arguments (>3) Multiple Return Points vertical openness between concepts Unpronounceable naming Switch Statement Abuse missing legal comments Message Chains – a.b().c().d() trainwreck variable declared far from its usage Use of pun Confusing Naming Deep Nesting large functions with nested structures Missing summary comment on a public method Dependent functions are far apart Long Method Violates Command- Query Separation Commented Out Code Requires horizontal scrolling Comments Explaining Code Non- searchable variable name Error codes instead of exception Code Duplication Too Many Responsibilities Catching exceptions and logging only Too many arguments (>3) Multiple Return Points vertical openness between concepts Unpronounceable naming Switch Statement Abuse missing legal comments Message Chains – a.b().c().d() trainwreck variable declared far from its usage Use of pun Confusing Naming Deep Nesting large functions with nested structures Missing summary comment on a public method Dependent functions are far apart
(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.
Long Method
Violates Command-Query Separation
Commented Out Code
Requires horizontal scrolling
Comments Explaining Code
Non-searchable variable name
Error codes instead of exception
Code Duplication
Too Many Responsibilities
Catching exceptions and logging only
Too many arguments (>3)
Multiple Return Points
vertical openness between concepts
Unpronounceable naming
Switch Statement Abuse
missing legal comments
Message Chains – a.b().c().d() trainwreck
variable declared far from its usage
Use of pun
Confusing Naming
Deep Nesting
large functions with nested structures
Missing summary comment on a public method
Dependent functions are far apart