You are getting "a lot of uninitialized value messages" because you are accessing a lot of data without checking if it exists or not.

Also, you are using the, by now, very old interface of Spreadsheet::ParseExcel. Try rewrite your example based on the example at the start of the current docs and employ the same type of error checking. That should ensure that you don't access uninitialised data.

If you still have a problem after that post your example again. As it is at the moment there are just too many issues to even start fixing the code.

--
John.