Having a bunch of standalone functions or procedures reduces maintainability because it becomes harder to find them and to see how they are
related. Instead, they should be logically grouped into meaningful packages.
Noncompliant code example
CREATE PROCEDURE show_name(name VARCHAR2) AS -- Non-Compliant
BEGIN
DBMS_OUTPUT.PUT_LINE('Name: ' || name);
END;
/
DROP PROCEDURE show_name;
Compliant solution
CREATE PACKAGE employee AS
PROCEDURE show_name;
END;
/
CREATE PACKAGE BODY employee AS
name VARCHAR2(42);
PROCEDURE show_name AS -- Compliant
BEGIN
DBMS_OUTPUT.PUT_LINE('Name: ' || name);
END;
END;
/
DROP PACKAGE employee;
