The following is a procedure in my package. It works, but I know they are not good codes and I want to improve them. I want to use a temporary table, but realize it is not a good way in Oracle. I want to use FOR LOOP and cursor, but should I put these mini procedures into a physical table first? Your advice would be greatly appreciated.
procedure mapData
is
begin
...
if g_err_found is null then
doSPBPERS;
end if;

if g_err_found is null then
doSPRADDR_TELE;
end if;

if g_err_found is null then
doGOREMAL(g_scRec.EMAIL);
end if;

if g_err_found is null then
doSRBRECR('COMA');
end if;

if g_err_found is null then
doSARADAP('COMA');
end if;

if g_err_found is null then
doSORHSCH_SORPCOL('COMA');
end if;

if g_err_found is null then
doSORLCUR_SORLFOS('COMA');
end if;

if g_err_found is null then
doSARRSRC('COMA');
end if;

if g_err_found is null then
doSRRRSRC('COMA');
end if;
--
if g_err_found is null then
doSORTEST;
end if;

if g_err_found is null then
popSARCHKLrec('COMA');
end if;

if g_err_found is null then
popSORCONTrec('COMA');
end if;

if g_err_found is null then
doSARACMT('COMA');
end if;
--
if g_err_found is null then
popGORPRACrec;
end if;
--
if g_err_found is null then
popGOBINTLrec;
end if;
--
if g_err_found is null then
popGORVISArec;
end if;
--
if g_err_found is null then
doSARQUAN;
end if;-- ADDED 11/3/2010

-- end if;
--
exception
when others then
dbms_output.put_line('error in mapData: ' || SQLERRM);
doError(' mapData ',SQLERRM);
end mapData;

procedure mapData
is
begin
     doSPBPERS;
     doSPRADDR_TELE;
     doGOREMAL(g_scRec.EMAIL);
     doSRBRECR('COMA');
     ...
     popGORVISArec;
     doSARQUAN;
exception
when others 
then
    doError(' mapData ',SQLERRM);
    RAISE;
end mapData;