Set dim with if statements

[silkpantsman]

Hi,

I am attempting to set a declared dim inside an if statement and obviously not doing it correctly, declaring "inv1" multiple times is not the solution but the code below should show what i am trying to achieve. Any advice would be great.

Thanks,
SPM

Please Login or Register  to view this content.

[Kyle123]

Just put the Dim at the top of the sub

[BadlySpelledBuoy]

Please Login or Register  to view this content.

BSB

[silkpantsman]

Yeah that is what i did first time around and thought i was losing my nut when it was not working. So i am assuming the problem lies in the rest of my code. I get an application defined or object defined error.

Please Login or Register  to view this content.

[Norie]

Do you have variables named AT, DE and BE etc or are you checking for the strings 'AT', 'DE' and 'BE'?

Please Login or Register  to view this content.

[silkpantsman]

Your correct i should have been using "AT" etc as i am checking for a string, ty. I am still getting same error.

[Norie]

Where do you get the error?

[silkpantsman]

When i run code from VBA menu run time 1004 application defined or object defined error.

[Norie]

When I run the code in the workbook I do get an error but not the one you describe.

However if I replace the code in the workbook with the code I posted in post #5 the code runs without error.

[Kyle123]

Your post does not comply with Rule 8 of our Forum RULES. Do not crosspost your question on multiple forums without including links here to the other threads on other forums.

Cross-posting is when you post the same question in other forums on the web. The last thing you want to do is waste people's time working on an issue you have already resolved elsewhere. We prefer that you not cross-post at all, but if you do (and it's unlikely to go unnoticed), you MUST provide a link (copy the url from the address bar in your browser) to the cross-post.

Expect cross-posted questions without a link to be closed and a message will be posted by the moderator explaining why. We are here to help so help us to help you!

Read this to understand why we ask you to do this, and then please edit your first post to include links to any and all cross-posts in any other forums (not just this site).

[silkpantsman]

Your post does not comply with Rule 8 of our Forum RULES. Do not crosspost your question on multiple forums without including links here to the other threads on other forums.

Cross-posting is when you post the same question in other forums on the web. The last thing you want to do is waste people's time working on an issue you have already resolved elsewhere. We prefer that you not cross-post at all, but if you do (and it's unlikely to go unnoticed), you MUST provide a link (copy the url from the address bar in your browser) to the cross-post.

Expect cross-posted questions without a link to be closed and a message will be posted by the moderator explaining why. We are here to help so help us to help you!

Read this to understand why we ask you to do this, and then please edit your first post to include links to any and all cross-posts in any other forums (not just this site).

Apologies, wont happen again...thanks for the help.

[Norie]

Nice.