G-1050
🆓Warning
Avoid using literals in your code.
Reason
Literals are often used more than once in your code. Having them defined as a constant reduces typos in your code and improves the maintainability.
All constants should be collated in just one package used as a library. If these constants should be used in SQL too it is good practice to write a deterministic package function for every constant.
To avoid an extreme plethora of constants or false positives, a literal should not occur more than once within a file.
Example
Non-Compliant Example
begin some_api.setup(in_department_id => 10); some_api.process(in_department_id => 10); some_api.teardown(in_department_id => 10); end; /
Issues
| Line | Column | Message |
|---|---|---|
| 2 | 39 | Literal 10 used more than 1 time. |
| 3 | 41 | Literal 10 used more than 1 time. |
| 4 | 42 | Literal 10 used more than 1 time. |
Compliant Solution - ★★★★★
create or replace package constants_up is co_dept_admin constant departments.department_id%type := 10; end constants_up; / begin some_api.setup(in_department_id => constants_up.co_dept_admin); some_api.process(in_department_id => constants_up.co_dept_admin); some_api.teardown(in_department_id => constants_up.co_dept_admin); end; /
Parameters
Use parameters to customize the rule to your needs.
| Parameter | Description | Default Value |
|---|---|---|
| AllowedMagicValues | Comma separated List of allowed literals in code. | -1, 0, 1 |
| MaxNumberOfLiterals | Maximum number of same literal allowed in a file. An issue is reported when the value is exceeded. | 1 |
References
- same as Trivadis G-1050