Opgeslagen procedure-SQL-ontwikkelaar

Waarom zou je de hele reeks parameters doorgeven als je alleen een camper-ID nodig hebt (en er is niets dat je met de rest zou doen)? Dat is gewoon niet logisch ...

Hier is een optie die laat zien hoe het moet.

Voorbeeldtabel:

SQL> select * from charges;

  CAMPERID CATEGORY   CHARGEDATE                 AMT
---------- ---------- ------------------- ----------
         1 Category 1 12.10.2021 00:00:00        500

SQL>

Werkwijze:

SQL> create or replace procedure p_test
  2    (par_camperid   in charges.camperid%type,
  3     par_message   out varchar2
  4    )
  5  as
  6    l_camperid   charges.camperid%type;
  7  begin
  8    select c.camperid
  9      into l_camperid
 10      from charges c
 11      where c.camperid = par_camperid;
 12
 13    par_message := 'Camper ' || par_camperid || ' exists';
 14  exception
 15    when no_data_found then
 16      par_message := 'Error: no such camper ID (' || par_camperid || ')';
 17  end;
 18  /

Procedure created.

Testen:

SQL> set serveroutput on;
SQL> declare
  2    l_msg varchar2(100);
  3  begin
  4    p_test(1, l_msg);
  5    dbms_output.put_line(l_msg);
  6  end;
  7  /
Camper 1 exists

PL/SQL procedure successfully completed.

SQL> declare
  2    l_msg varchar2(100);
  3  begin
  4    p_test(2, l_msg);
  5    dbms_output.put_line(l_msg);
  6  end;
  7  /
Error: no such camper ID (2)

PL/SQL procedure successfully completed.

SQL>

Code die je schreef:

typefouten, typefouten, typefouten ... is het p_error of p_erorr ?
if moet end if . hebben
if mag geen select bevatten statement (je moet de waarde apart selecteren, en dan gebruiken in if )
where toestand is verkeerd. Geef parameters nooit dezelfde naam als kolomnamen. Dit:where camperid = camperid is gelijk aan where 1 = 1 en je zou alle rijen van de tafel krijgen, niet alleen degene die je zoekt