G-4110
🆓Warning
Always use %NOTFOUND instead of NOT %FOUND to check whether a cursor returned data.
Reason
The readability of your code will be higher when you avoid negative sentences.
Example
Non-Compliant Example
declare
cursor c_employees is
select *
from employees
order by employee_id;
type t_employees_type is table of c_employees%rowtype;
t_employees t_employees_type;
co_bulk_size constant simple_integer := 10;
begin
open c_employees;
<<process_employees>>
loop
fetch c_employees bulk collect into t_employees limit co_bulk_size;
<<display_employees>>
for i in 1..t_employees.count()
loop
sys.dbms_output.put_line(t_employees(i).last_name);
end loop display_employees;
exit process_employees when not c_employees%found;
end loop process_employees;
close c_employees;
end;
/Issues
| Line | Column | Message |
|---|---|---|
| 23 | 35 | Use %NOTFOUND instead of NOT %FOUND. |
Compliant Solution - ★★★★★
declare
cursor c_employees is
select *
from employees
order by employee_id;
type t_employees_type is table of c_employees%rowtype;
t_employees t_employees_type;
co_bulk_size constant simple_integer := 10;
begin
open c_employees;
<<process_employees>>
loop
fetch c_employees bulk collect into t_employees limit co_bulk_size;
<<display_employees>>
for i in 1..t_employees.count()
loop
sys.dbms_output.put_line(t_employees(i).last_name);
end loop display_employees;
exit process_employees when c_employees%notfound;
end loop process_employees;
close c_employees;
end;
/References
- same as Trivadis G-4110
- same as plsql:Notfound
- same as plsqlopen:NotFound