Existe-t-il d'autres méthodes pour dire "next" dans une boucle pl / sql for?
Question
J'ai donc une boucle for qui traite une liste d'identifiants et qui a des tâches assez complexes à faire. Sans entrer dans tous les détails laids, voici essentiellement ceci:
DECLARE l_selected APEX_APPLICATION_GLOBAL.VC_ARR2; ...snip... BEGIN -- get the list ids l_selected := APEX_UTIL.STRING_TO_TABLE(:P4_SELECT_LIST); -- process each in a nice loop FOR i IN 1..l_selected.count LOOP -- do some data checking stuff... -- here we will look for duplicate entries, so we can noop if duplicate is found BEGIN SELECT county_id INTO v_dup_check FROM org_county_accountable WHERE organization_id = :P4_ID AND county_id = v_county_id; -- NEXT;! NOOP;! but there is no next! EXCEPTION WHEN NO_DATA_FOUND THEN dbms_output.put_line('no dups found, proceeding'); END; -- here we have code we only want to execute if there are no dupes already IF v_dup_check IS NULL THEN -- if not a duplicate record, proceed... ELSE -- reset duplicate check variable v_dup_check := NULL; END; END LOOP; END;
La façon dont je le fais normalement consiste à sélectionner une valeur, puis à envelopper le code suivant dans une instruction IF pour vérifier que la variable de contrôle en double est NULL. Mais c'est agaçant. Je veux juste pouvoir dire NEXT; ou NOOP; ou quelque chose. Surtout que je dois déjà attraper l'exception NO_DATA_FOUND. Je suppose que je pourrais écrire une lettre à Oracle, mais je suis curieux de voir comment les autres gèrent cela.
Je pourrais aussi envelopper cela dans une fonction aussi, mais je cherchais quelque chose d'un peu plus propre / plus simple.
La solution
Compter le nombre de lignes est également possible (voir Pourquoi Litytestdata) mais vous pouvez également faire ce que vous voulez faire dans le bloc when_no_data_found exception
.
declare
l_selected apex_application_global.vc_arr2;
l_county_id org_county_accountable.count_id%type;
begin
l_selected := apex_util.string_to_table(:p4_select_lst);
for i in l_selected.first..l_selected.last loop
begin
select count_id
into l_county_id
from org_county_accountable
where organization_id = :p4_id
and county_id = v_county_id;
exception
when no_data_found then
-- here we have code we only want to execute if there are no dupes already
-- if not a duplicate record, proceed...
end;
end loop;
end;
Autres conseils
Oracle 11g ajoute un style C " continue " la construction de boucle en PL / SQL, ce qui ressemble syntaxiquement à ce que vous recherchez.
Pour vos besoins, pourquoi ne pas simplement éliminer les doublons avant d’entrer dans la boucle? Pour ce faire, vous pouvez interroger l_selected à l'aide d'une fonction de table, puis filtrer les enregistrements non souhaités au lieu d'itérer chaque valeur . Quelque chose comme ...
declare
l_selected APEX_APPLICATION_GLOBAL.VC_ARR2;
cursor no_dups_cur (p_selected APEX_APPLICATION_GLOBAL.VC_ARR2) is
select * from (
select selected.*,
count(*) over (partition by county_id) cnt -- analytic to find counts grouped by county_id
from table(p_selected) selected -- use table function to treat VC_ARR2 like a table
) where cnt = 1 -- remove records that have duplicate county_ids
;
begin
l_selected := APEX_UTIL.STRING_TO_TABLE(:P4_SELECT_LIST);
for i in no_dups_cur(l_selected) loop
null; -- do whatever to non-duplicates
end loop;
end;
Remplacez simplement la logique de détermination d'un " duplicate " avec le vôtre (vous n'avez pas eu assez d'informations de votre exemple pour vraiment répondre à cette partie)
Au lieu d'attraper NO_DATA_FOUND
, pourquoi ne pas SÉLECTIONNER le nombre d'entrées correspondantes dans une variable, par exemple, l_count
, et procéder si ce nombre est égal à zéro? Quelque chose comme ce qui suit:
DECLARE l_selected APEX_APPLICATION_GLOBAL.VC_ARR2; l_count INTEGER; ...snip... BEGIN -- get the list ids l_selected := APEX_UTIL.STRING_TO_TABLE(:P4_SELECT_LIST); -- process each in a nice loop FOR i IN 1..l_selected.count LOOP -- do some data checking stuff... -- here we will count duplicate entries, so we can noop if duplicate is found SELECT COUNT(*) INTO l_count FROM org_county_accountable WHERE organization_id = :P4_ID AND county_id = v_county_id; IF l_count = 0 THEN -- here we have code we only want to execute if there are no dupes already -- if not a duplicate record, proceed... END IF; END LOOP; END;
<xmp>
<<next_loop>>
loop
...
...
if ....
then
goto next_loop;
</xmp>
C’est le cas où une instruction GOTO pourrait être utile. Consultez la Documentation Oracle dans la documentation structures de contrôle pour voir comment faire cela. En outre, vous voudrez peut-être effectuer une recherche ici pour savoir comment rechercher l'existence d'un enregistrement. Exécuter une requête et attendre une exception n'est pas optimal.
Autre moyen - transformez le contrôle en une fonction locale:
DECLARE
l_selected APEX_APPLICATION_GLOBAL.VC_ARR2;
...snip...
FUNCTION dup_exists
( p_org_id org_county_accountable.organization_id%TYPE
, p_county_id org_county_accountable.county_id%TYPE
) RETURN BOOLEAN
IS
v_dup_check org_county_accountable.county_id%TYPE;
BEGIN
SELECT county_id INTO v_dup_check FROM org_county_accountable
WHERE organization_id = p_org_id AND county_id = p_county_id;
RETURN TRUE;
EXCEPTION WHEN NO_DATA_FOUND THEN
RETURN FALSE;
END;
BEGIN
-- get the list ids
l_selected := APEX_UTIL.STRING_TO_TABLE(:P4_SELECT_LIST);
-- process each in a nice loop
FOR i IN 1..l_selected.count
LOOP
-- do some data checking stuff...
-- here we have code we only want to execute if there are no dupes already
IF NOT dup_exists (:P4_ID, v_county_id) THEN
-- if not a duplicate record, proceed...
END;
END LOOP;
END;
Bien sûr, la fonction locale peut être réécrite pour utiliser la méthode de comptage si vous préférez:
FUNCTION dup_exists
( p_org_id org_county_accountable.organization_id%TYPE
, p_county_id org_county_accountable.county_id%TYPE
) RETURN BOOLEAN
IS
l_count INTEGER;
BEGIN
SELECT COUNT(*) INTO l_count
FROM org_county_accountable
WHERE organization_id = p_org_id AND county_id = p_county_id;
RETURN (l_count > 0);
END;
Une autre méthode consiste à déclencher et à gérer une exception définie par l'utilisateur:
DECLARE
l_selected APEX_APPLICATION_GLOBAL.VC_ARR2;
duplicate_org_county EXCEPTION;
...snip...
BEGIN
-- get the list ids
l_selected := APEX_UTIL.STRING_TO_TABLE(:P4_SELECT_LIST);
-- process each in a nice loop
FOR i IN 1..l_selected.count
LOOP
BEGIN
-- do some data checking stuff...
-- here we will look for duplicate entries, so we can noop if duplicate is found
BEGIN
SELECT county_id INTO v_dup_check FROM org_county_accountable
WHERE organization_id = :P4_ID AND county_id = v_county_id;
RAISE duplicate_org_county;
EXCEPTION WHEN NO_DATA_FOUND THEN
dbms_output.put_line('no dups found, proceeding');
END;
-- here we have code we only want to execute if there are no dupes already
EXCEPTION
WHEN duplicate_org_county THEN NULL;
END;
END LOOP;
END;
Je ne ferais normalement pas cela, mais s'il y avait une demi-douzaine de raisons de passer au prochain enregistrement, cela serait peut-être préférable à plusieurs FI imbriquées.
Je sais que c’est un vieil homme, mais j’ai remarqué qu’aucune des réponses ci-dessus ne tient compte du curseur attributs :
Il existe quatre attributs associés aux curseurs: ISOPEN, FOUND, NOTFOUND et ROWCOUNT. Vous pouvez accéder à ces attributs avec le délimiteur% pour obtenir des informations sur l’état du curseur.
La syntaxe d'un attribut de curseur est la suivante:
cursor_name%attribute
où nom_curseur est le nom du curseur explicite.
Ainsi, dans ce cas, vous pouvez utiliser ROWCOUNT (qui indique le nombre de lignes récupérées jusqu'à présent) pour vos besoins, comme ceci:
declare
aux number(10) := 0;
CURSOR cursor_name is select * from table where something;
begin
select count(*) into aux from table where something;
FOR row IN cursor_name LOOP
IF(aux > cursor_name%ROWCOUNT) THEN 'do something is not over';
ELSE 'do something else';
END IF;
END LOOP;
end;