1 has been discussed before
2 convert the code above into ordinary sql removing the begin end;
pair and you will be ok
3 It is generally considered bad programming to create tables on the
fly in pl/sql

--
Sybrand Bakker, Senior Oracle DBA

Why are you creating a table?  Also for insert it is considered better to do
insert into tablex(col1,col2,..) values (...);

Jim

Thanks all for your useful comments

Have received an answer from another group fyi:

Oracle evaluates the whole block before attempting to execute, but the
string used in execute immediate is not validated until it runs.  So,
it looks at exeucte immediate 'anything' and considers that valid, then

looks at the insert statement and cannot validate it, because the table

to insert into does not exist yet.  If you are going to create the
table dynamically, then you must also insert into it dynamically, as
demonstrated below.

scott@ORA92> begin
 2    execute immediate 'create table c (col1 number,col2
varchar2(30))';
 3    execute immediate 'insert into c values (123,''data'')';
 4  end;
 5  /

PL/SQL procedure successfully completed.

scott@ORA92> desc c
Name                                      Null?    Type
------------------------------­----------- --------
----------------------------
COL1                                               NUMBER
COL2                                               VARCHAR2(30)

scott@ORA92> select * from c
 2  /

     COL1 COL2
---------- ------------------------------
      123 data

scott@ORA92>