Avoid having too many conditions using variables without operators

[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=”CODE RELIABILITY” 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%2Fcode-reliability%2F|||”][vc_empty_space height=”12px”][vc_row_inner][vc_column_inner width=”1/2″][vc_column_text]

Why you should care

[/vc_column_text][vc_column_text]When a variable without any operators is found within a ‘IF’ condition, in most of cases, it means that the developer wanted to perform some controls on this variable. However, in order to avoid any unexpected behavior during the execution, and depending on the verification type the developer intends to perform (existing/set or empty or true/false or null variable?), it is recommended to use the appropriate method, so that the risk of misinterpretation of the verification is limited.[/vc_column_text][/vc_column_inner][vc_column_inner width=”1/2″][vc_column_text]

Business Impacts

It is recommended to use an explicit method to verify conditions of your variables, in order to avoid misinterpretation and possible bugs.[/vc_column_text][vc_empty_space][/vc_column_inner][/vc_row_inner][vc_empty_space height=”12px”][vc_empty_space][vc_single_image image=”5362″ img_size=”full”][vc_row_inner][vc_column_inner width=”1/2″][vc_column_text]

CAST Recommendations

CAST recommends checking your variables with the appropriate method, depending on your specific case (e.g. isset() to check the variable exists, empty(), is_null, etc.) in order to avoid any misinterpretation of the verification (boolean, empty, exist).
[/vc_column_text][/vc_column_inner][vc_column_inner width=”1/2″][vc_column_text color=”#dd3333″]

References

https://stackoverflow.com/questions/6693876/how-exactly-does-ifvariable-work
https://stackoverflow.com/questions/30191521/php-check-if-variable-is-undefined[/vc_column_text][/vc_column_inner][/vc_row_inner][vc_separator][vc_column_text]

How we detect

[/vc_column_text][vc_column_text]This Code Insight counts one occurrence each time a variable is found in a codition, without any operator.

Example:

if($_GET[‘myvariable’]) { … }
if($foo) { … }[/vc_column_text][vc_empty_space][vc_row_inner][vc_column_inner][vc_single_image image=”5362″ img_size=”full”][vc_column_text]

About CAST and Highlight’s Code Insights

Over the last 25 years, CAST has leveraged unique knowledge on software quality measurement by analyzing thousands of applications and billions of lines of code. Based on this experience and community standards on programming best practices, Highlight implements hundreds of code insights across 15+ technologies to calculate health factors of a software.

[/vc_column_text][vc_empty_space height=”16px”][vc_btn title=”SEE FEATURES” style=”flat” color=”black” i_icon_fontawesome=”fa fa-desktop” add_icon=”true” link=”url:https%3A%2F%2Fwww.castsoftware.com%2Fproducts%2Fhighlight%2Foutputs-analytics||target:%20_blank|”][vc_btn title=”HOW IT WORKS” style=”outline” color=”black” i_icon_fontawesome=”fa fa-cog” add_icon=”true” link=”url:https%3A%2F%2Fwww.castsoftware.com%2Fproducts%2Fhighlight%2Fhow-it-works||target:%20_blank|”][/vc_column_inner][/vc_row_inner][/vc_column][/vc_row]

?>