Archive
Code Insights
-
Use of ‘return’ statement is not recommended
[vc_row][vc_column][vc_btn title=”SOFTWARE RESILIENCY” style=”flat” color=”black” i_icon_fontawesome=”fa fa-shield” add_icon=”true” link=”url:http%3A%2F%2Fdoc.casthighlight.com%2Fsoftware-resiliency%2F|||”][vc_btn title=”BEST PRACTICES” style=”outline” color=”black” i_icon_fontawesome=”fa fa-code” add_icon=”true” link=”url:http%3A%2F%2Fdoc.casthighlight.com%2Fcategory%2Fproduct%2Findicators-methodology%2Fcode-insights%2Fsoftware-resiliency%2Fbest-practices%2F|||”][vc_empty_space height=”12px”][vc_row_inner][vc_column_inner][vc_column_text] Why you should care [/vc_column_text][vc_column_text]Specifically in…
-
Do not compare to null
[vc_row][vc_column][vc_btn title=”SOFTWARE RESILIENCY” style=”flat” color=”black” i_icon_fontawesome=”fa fa-shield” add_icon=”true” link=”url:http%3A%2F%2Fdoc.casthighlight.com%2Fsoftware-resiliency%2F|||”][vc_btn title=”BEST PRACTICES” style=”outline” color=”black” i_icon_fontawesome=”fa fa-code” add_icon=”true” link=”url:http%3A%2F%2Fdoc.casthighlight.com%2Fcategory%2Fproduct%2Findicators-methodology%2Fcode-insights%2Fsoftware-resiliency%2Fbest-practices%2F|||”][vc_empty_space height=”12px”][vc_row_inner][vc_column_inner][vc_column_text] Why you should care [/vc_column_text][vc_column_text]Whenever null…
-
Avoid long ‘if/else if’ chains that could be replaced by a ‘switch’ statement
For code readability purpose, prefer using switch if there are three or more options in a if.
-
phpinfo() should not be used in production
For security purpose, developers should not leave phpinfo() in production code, as it displays information which can be used to compromise the server that…
-
Avoid useless overriding method
Overring methods is a very powerfull way of factoring the code and produce complete fonctionnalities. However useless overriding methods can be a problem for…
-
The code contains too many uppercase control structure keywords
THE CODE CONTAINS TOO MANY UPPERCASE CONTROL STRUCTURE KEYWORDS
-
The code contains too many PHP4 deprecated constructor naming.
Developers should avoid using deprecated constructors. Since PHP5, constructor should be named __construct.
-
The code contains too many functions not complying with naming conventions
Complying with naming conventions make the source code easier to ready and so to maintain. Function names should start with an upper case and…