I don't use Oracle, but I think you should be doing a LOT of error checking while you're developing. See the examples here for some ideas. And don't use "select *" - use the column names you want - using column names is more efficient and safer down the road.