Pergunta

Então eu tenho um loop que processa uma lista de IDs e tem algumas coisas bastante complexas para fazer. Sem entrar em todos os detalhes feios, basicamente isto:

    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;

Como eu normalmente lidar com isso é selecionando em um valor e, em seguida, enrole o seguinte código em uma instrução IF verificando se essa variável cheque duplicado é NULL. Mas é irritante. Eu só quero ser capaz de dizer a seguir; ou NOOP; ou alguma coisa. Especialmente desde que eu já tenho para capturar a exceção NO_DATA_FOUND. Acho que eu poderia escrever uma carta para a Oracle, mas estou curioso como os outros lidar com isso.

Eu também poderia envolver isso em uma função, também, mas eu estava procurando algo um pouco mais limpo / mais simples.

Foi útil?

Solução

Para contar o número de linhas também é possível (ver Pourquoi Litytestdata), mas você também pode fazer o que quer fazer no bloco 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;

Outras dicas

Oracle 11g adiciona um estilo C "continuar" construção de loop para PL / SQL, o que soa sintaticamente como o que você está procurando.

Para seus propósitos, porque não basta eliminar as duplicatas antes de entrar no loop? Isso poderia ser feito por meio de consulta l_selected usando uma função de tabela, e depois filtrar registros que você não quer, em vez de iteração sobre todas valor. Algo como ...

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;

Basta substituir a lógica para determinar um "duplicado" com seu próprio (não têm informação suficiente do seu exemplo para realmente responder a essa parte)

Em vez de pegar NO_DATA_FOUND, como sobre selecionando o número de entradas correspondentes em uma variável, digamos l_count e processo se esta contagem trabalha para fora para ser zero? Algo como o seguinte:

    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>

Este é um caso em que uma declaração GOTO pode ser útil. Veja a a Oracle Documentação na estruturas de controle para ver como fazer isso. Além disso, você pode querer procurar por aqui para descobrir como a consulta para a existência de um registro. Executar uma consulta e à espera de uma exceção não é o ideal.

Outra forma - transformar o cheque em uma função local:

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;

É claro que a função local pode ser re-escrita para utilizar o método de contagem, se você preferir:

  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;

Outro método é para levantar e segurar um excepção definido pelo utilizador:

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;

Eu normalmente não faria isso, mas se houvesse meia dúzia de razões para saltar para o próximo registro, isso pode ser preferível a vários IFs aninhados.

Eu sei que este é um oldie mas não consegui aviso de ajuda que nenhuma das respostas acima tomada em conta o cursor atributos :

Há quatro atributos associados com cursores: ISOPEN, encontrado, NOTFOUND e ROWCOUNT. Esses atributos podem ser acessados ??com o delimitador% para obter informações sobre o estado do cursor.

A sintaxe para um atributo de cursor é:

cursor_name%attribute

onde cursor_name é o nome do cursor explícito.

Portanto, neste caso você poderia usar ROWCOUNT (que indica o número de linhas buscadas até agora) para os seus fins, como este:

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;
Licenciado em: CC-BY-SA com atribuição
Não afiliado a StackOverflow
scroll top