Macro goes into an infinite loop

**excelforum123** · 10-14-2010, 08:54 AM

There are two sheets in the workbook. First sheet has certain values in a number of rows in a column. The second sheet also has values in each row in repeating patterns in a column. Like this:

Please Login or Register  to view this content.

The macro is supposed to go through each row in Sheet2 per each row in the first sheet; assign a certain value to a variable when any given row in Sheet2 includes some other certain values; compare this variable's value to the row value in Sheet1 and quit once this process has been repeated for 4 rows in Sheet1:

Please Login or Register  to view this content.

This is supposed to stop once the match for the 3rd row in Sheet1 is found in Sheet2 but it only happens for the 1st row in Sheet1 and then goes into infinite loop.

I am very confused. Supposed to be that once a match for the first row of Sheet1 is made, dRow is incremented by 1 and thus looking for a match in for the second row of Sheet1 while sRow is reset to 0 so it starts from the first row of Sheet2 so as not to miss any rows. What is wrong here?

**TMS** · 10-14-2010, 12:29 PM

It didn't go into an infinite loop for me. I displayed "AAAA", "BBBB", "CCCC", "DDDD" and finished.

I stepped through with F8 and then just let it run (F5).

Not sure what it's trying to achieve but it doesn't loop (more than it should)

Regards

**TMS** · 10-14-2010, 12:37 PM

What is interesting is, if you change the first search string from 1111 to 1112 so that it doesn't match, it never goes through the part of the code which increments dRow.

The implication is that it isn't finding what it is looking for.

You could try setting the fourth parameter in INSTR

Regards

**snb** · 10-14-2010, 01:12 PM

Please Login or Register  to view this content.

**excelforum123** · 10-21-2010, 10:08 AM

Thanks. I don't know why it was infinitely looping for me. The same script started working properly without any changes. Weird.

Also, thank you, snb. Novel suggestion, one I should remember but not for this particular example, as the actual has about 200 such entries going through the same procedure.