Error while creating Package body

Question

Hi I tried the following package is created but body is created with errors
CREATE OR REPLACE PACKAGE pack_ttt IS
TYPE rec_ttt_colinfo IS RECORD (
  columnName VARCHAR2(30));
TYPE nt_ttt_colInformation IS TABLE OF rec_ttt_colinfo;
FUNCTION getColumns(schemaNm VARCHAR2 , TableNm VARCHAR2)
  RETURN nt_ttt_colInformation;
END;
CREATE OR REPLACE PACKAGE BODY pack_ttt IS
function getColumns(
  schemaNm VARCHAR2 ,
  TableNm VARCHAR2)
RETURN nt_ttt_colInformation IS
colvarfunc1 nt_ttt_colInformation;
EXECUTE IMMEDIATE 'SELECT COLUMN_NAME, DATA_TYPE FROM ALL_TAB_COLUMNS WHERE OWNER = ''' || schemaNm || ''' AND TABLE_NAME = ''' || TableNm || ''''
      BULK COLLECT INTO columnsList;
      RETURN colvarfunc1;
END;
END;And the error is
LINE/COL ERROR
7/20     PLS-00103: Encountered the symbol "SELECT COLUMN_NAME, DATA_TYPE
         FROM ALL_TAB_COLUMNS WHERE OWNER =" when expecting one of the
         following:
         := . ( @ % ; not null range default character
         The symbol ":=" was substituted for "SELECT COLUMN_NAME,
         DATA_TYPE FROM ALL_TAB_COLUMNS WHERE OWNER =" to continue.
8/7      PLS-00103: Encountered the symbol "BULK" when expecting one of
         the following:
         * & = - + ; < / > at in is mod remainder not rem
         <an exponent (**)> <> or != or ~= >= <= <> and or like LIKE2_
LINE/COL ERROR
         LIKE4_ LIKEC_ between || member SUBMULTISET_

Answer 1

Why are you using dynamic SQL, there is absolutely no need for that. And the way you are using it is plain wrong, since it is subject to SQL injection and hard parsing.
CREATE OR REPLACE PACKAGE BODY pack_ttt IS
  FUNCTION getcolumns (
    schemanm    VARCHAR2,
    tablenm     VARCHAR2
    RETURN nt_ttt_colinformation IS
    colvarfunc1   nt_ttt_colinformation;
  BEGIN
    SELECT column_name,
           data_type
    BULK   COLLECT INTO columnslist
    FROM   all_tab_columns
    WHERE  owner = schemanm
           AND table_name = tablenm;
    RETURN colvarfunc1;
  END;
END;
/Don't get into the habbit of doing dynamic sql, it just demonstrates that you have no clue whatsoever as to what you are dealing with.
Regards
Peter