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.

Était-ce utile?

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;
Licencié sous: CC-BY-SA avec attribution
Non affilié à StackOverflow
scroll top