If string contains "word" then...

[dougmcc1]

I'm using InStr to look for words within a string but its acting buggy.

Here's the code I'm using:

Please Login or Register  to view this content.

Heres a value where it contains "lpd" and also "course" so the code should return "/lpd/ with Courses" but instead returns "/lpd/ Other":
http://xyz.com/school/lpd/science-wo...ml?type=course

Any idea why? Why is InStr not finding "lpd" and "course" in the above value? Is there a better to do what I'm trying to do?

Thanks for any help.

[stnkynts]

Your link didn't work for me. Hard to tell without seeing what tempString is but I think you may be having an issue with the parameters of using the Instr() function. Instr returns an integer and you have not specified that as part of your function. I could be wrong though. Here is an example.

Please Login or Register  to view this content.

[AlphaFrog]

Perhaps there is there a capitalisation case difference of the text you're looking for and the data? If yes, this will ignore case.

Please Login or Register  to view this content.

On another note; if you start the code block by testing if the string contains "lpd", you wouldn't have to test for that again in the subsequent If-ElseIf statements.

Please Login or Register  to view this content.

[dougmcc1]

Hard to tell without seeing what tempString is

tempString = "http://xyz.com/school/lpd/science-worksheets.html?type=course"

Perhaps there is there a capitalisation case difference

The url I'm looking for strings in is lowercase so there shouldnt be any capitalization issues

Any other ideas? This is killing me. It seems like such a simple thing so its really frustrating that I cant figure out why it's not working...

[AlphaFrog]

This simple test works.

Please Login or Register  to view this content.

So I would have to suspect that in your case tempstring isn't what you think it is, or is it possible r.Offset(, -1).Value isn't the cell you think it is, or this block of code isn't run for some reason.

[dougmcc1]

That approach works for me too - thanks! I guess there's something funky that happens when using "If InStr... And InStr..." instead of nested if statements (as in your example above). If anyone has an answer, I would love to know the technical reason behind this as, to me, both pieces of code should produce the same result, no?

[AlphaFrog]

You're welcome.

This test uses your original code with "If InStr... And InStr..." and it worked just as well. So i don't know what to tell you.

Please Login or Register  to view this content.