"On Error" activates here and I don't know why

I've written the below code which goes through a worksheet (after I've imported some text files), and deletes the header rows. The code works except for one thing. If there are no duplicates, it activates the "On Error" code. Can anyone tell me why this might happen? If there are duplicates, it removes them and everything works fine.

I was thinking I could put a counter in there and if there are no duplicates found, then I could "exit Sub" but I feel like I'm missing out on understanding something here.

Thanks if you can help

Please Login or Register  to view this content.

Originally Posted by Norie

If the code doesn't error then after End With execution moves straight to the message box.

yes. I don't understand why. Doesn't it have to be caused by something tripping the "On Error" part of the code at the top?

Originally Posted by Norie

Putting an Exit Sub after the End With should stop the message box being shown if there are no errors.

Yes, that would be simpler. But I still don't understand why I would need to?

Originally Posted by Norie

What error are you trying to catch anyway?

Any errors that might happen.

Originally Posted by Norie

Code execution moves from top to bottom, so it's eventually going to reach the message box as nothing's stopping it.

Interesting. Thank you. Where should I be putting that ErrorCatch code then?

Originally Posted by HaHoBe

Hi, njs27,

if dealing the deletion you better start at the bottom and work up in order to avoid manipulation of the counter.

Please Login or Register  to view this content.

Ciao,
Holger

Thanks, Holger. That's actually why I put in this part:

Please Login or Register  to view this content.

but I like your way better! Thanks!

Originally Posted by Norie

Do you need it?

I may not need it here, but I will probably need it in some code eventually, so I'd just like to know where it's supposed to go. Is it dependent on the type of error?

D'oh! Ok I missed a fundamental piece of Error Handling code - that I NEED the Exit Sub before the end block. Sorry!