43 lines
673 B
Plaintext
43 lines
673 B
Plaintext
|
A <code>GOTO</code> statement is an unstructured change in the control flow. They should be avoided and replaced by structured constructs.
|
||
|
|
|
||
|
|
== Noncompliant Code Example
|
||
|
|
|
||
|
|
----
|
||
|
|
SET SERVEROUTPUT ON
|
||
|
|
|
||
|
|
DECLARE
|
||
|
|
i PLS_INTEGER := 42;
|
||
|
|
BEGIN
|
||
|
|
IF i < 0 THEN
|
||
|
|
GOTO negative; -- Noncompliant
|
||
|
|
END IF;
|
||
|
|
|
||
|
|
DBMS_OUTPUT.PUT_LINE('positive');
|
||
|
|
goto cleanup; -- Noncompliant
|
||
|
|
|
||
|
|
<<negative>>
|
||
|
|
DBMS_OUTPUT.PUT_LINE('negative!');
|
||
|
|
|
||
|
|
<<cleanup>>
|
||
|
|
NULL;
|
||
|
|
END;
|
||
|
|
/
|
||
|
|
----
|
||
|
|
|
||
|
|
== Compliant Solution
|
||
|
|
|
||
|
|
----
|
||
|
|
SET SERVEROUTPUT ON
|
||
|
|
|
||
|
|
DECLARE
|
||
|
|
i PLS_INTEGER := 42;
|
||
|
|
BEGIN
|
||
|
|
IF i < 0 THEN
|
||
|
|
DBMS_OUTPUT.PUT_LINE('negative!'); -- Compliant
|
||
|
|
ELSE
|
||
|
|
DBMS_OUTPUT.PUT_LINE('positive');
|
||
|
|
END IF;
|
||
|
|
END;
|
||
|
|
/
|
||
|
|
----
|