klenwell information services : CodeSmells

Revision history for CodeSmells


Revision [3159]

Last edited on 2018-09-05 02:31:33 by KlenwellAdmin
Additions:
[[https://en.wikipedia.org/wiki/Magic_number_(programming)#Unnamed_numerical_constants https://en.wikipedia.org/wiki/Magic_number_(programming)]]
Deletions:
[https://en.wikipedia.org/wiki/Magic_number_(programming)#Unnamed_numerical_constants https://en.wikipedia.org/wiki/Magic_number_(programming)]


Revision [3158]

Edited on 2018-09-05 02:31:21 by KlenwellAdmin
Additions:
[https://en.wikipedia.org/wiki/Magic_number_(programming)#Unnamed_numerical_constants https://en.wikipedia.org/wiki/Magic_number_(programming)]
Deletions:
https://en.wikipedia.org/wiki/Magic_number_(programming)#Unnamed_numerical_constants


Revision [3157]

Edited on 2018-09-05 02:30:59 by KlenwellAdmin
Additions:
A more authoritative list: https://blog.codinghorror.com/code-smells/


Revision [3156]

Edited on 2018-09-05 02:28:49 by KlenwellAdmin
Additions:
Please do not reinvent the wheel or a natural language date parser: https://en.wikipedia.org/wiki/Not_invented_here#In_computing
Interpolation not concatenation: https://github.com/rubocop-hq/ruby-style-guide#strings
https://en.wikipedia.org/wiki/Magic_number_(programming)#Unnamed_numerical_constants
Use a constant or config setting.
%%
# bad
admin_styles.scss
scripts-admin.js
# good
admin.scss
admin.js
# bad
public function handle_archiving_of_courses( ) {}
# good
public function archive_courses( ) {}
%%


Revision [3155]

Edited on 2018-09-05 02:17:37 by KlenwellAdmin
Additions:
Use a guard clause: https://refactoring.com/catalog/replaceNestedConditionalWithGuardClauses.html
Deletions:
Use a guard clause.


Revision [3154]

Edited on 2018-09-05 02:05:35 by KlenwellAdmin
Additions:
==== Not Invented Here (NIH) Syndrome / Uncited Sources ====
Deletions:
==== Not Invented Here ====


Revision [3153]

The oldest known version of this page was created on 2018-09-05 02:04:55 by KlenwellAdmin