PowerBuilder

Two branches in a conditional structure should not have exactly the same implementation

Description

    This rule states that when a conditional structure is used, the two branches should not have the same implementation. This means that the code executed in each branch should be different. This helps to ensure that the code is more organized and readable, and that the code runs as expected.

Key Benefits

  • Reduced Complexity : By avoiding having two branches with the same implementation, the overall complexity of the conditional structure is reduced.
  • Easier Maintenance : With only one implementation for each branch, maintenance and debugging of the conditional structure is made simpler.
  • Improved Performance : By avoiding having the same implementation for both branches, SQL Server can optimize the conditional structure for improved performance.

 

Non-compliant Code Example

function string CallFunctionTest (string cnt)

string str;

if cnt = 1 then
	cnt = 'B';
	str = 'A';
elseif cnt = 2 then
	cnt = 'B'; //Non compliant code (IF clause and ELSEIF clause is having exactly the same implementation )
	str = 'A';
else 
	cnt = 'C';
	str = 'A';
end if

return cnt

end function

Compliant Code Example

function string CallFunctionTest (string cnt)

string str;

if cnt = 1 OR cnt = 2 then //Compliant code
	cnt = 'B';
	str = 'A';
else 
	cnt = 'C';
	str = 'A';
end if

return cnt

end function
Visual Expert 2024
 VEPBRULE46