Run time Error 91 - End Loop not working correctly

I have no knowledge whatever of programming languages so I am hoping that someone can be of help. I need to take a report exported from a Filemaker Pro database as a .pdf document and import the report into Excel 2007. The report can only be exported as a .pdf file. I have saved the .pdf file as a text document and imported the text document into Excel. I have then created a macro by using the record macro function to delete irrelevant data from the column. The difficulty is that when I try to loop the macro to perform the operation through the entire column of data I receive a Error 91 message. I assume that the macro is not ending the loop properly. Below is the code. If anyone can help with how to close the loop properly that would be wonderful.

Please Login or Register  to view this content.

Try skipping the loop and using an AUTOFILTER instead. Data > Filter > AutoFilter allows you to selectively filter the data by a value one time, then you can highlight and delete all the visible rows in one go.

Please Login or Register  to view this content.

Hi

You'll get a runtime error with your original because once all the data has been deleted the Find() method won't return a cell reference and so you can't activate the found cell (thus you get the error). You could control for this like:

Please Login or Register  to view this content.

Richard

Originally Posted by rowing190

Thank you for the suggestion. I will try to implement it.

The code that I originally posted actually works. But a Run Time Error 91 occurs because the loop is not being closed properly. As a consequence I cannot merge that macro with another similar macro to reduce the number of macros to actually be run.

rowing190

The main reason I suggested the alternate approach is that LOOPS have to evaluate each row/cell individually, and you are deleting rows individually. That's a lot of time spent.

The Auto Filter does the same thing in just a few steps, deleting ALL the matching rows in a single command. It's a faster approach.

Good call Richard, I missed that completely on the LOOP method.

=========
If that takes care of your need, be sure to EDIT your original post, click Go Advanced and mark the PREFIX box [SOLVED].


(Also, use the blue "scales" icon in our posts to leave Reputation Feedback, it is appreciated)