i have an if then statement that works, how to make it work as a with or case statement

i have this bit of code that works.

Please Login or Register  to view this content.

what i want is it to work with the code below, so i tried a multiple IF statement that failed, basically i put an "if xyz... then if sheets..." but the compiler balked at that. so now that i know it wont work that way, can i wrap the code using a WITH statement or a CASE statement to make it work? just a note that i have several of these IF xyz statements so the solution needs to work with that.

Please Login or Register  to view this content.

this failed below

Please Login or Register  to view this content.

Try this:

Please Login or Register  to view this content.

I posted before I saw 6StringJazzer's response.

His answer is good too. Both his code and mine will do the exact same thing.

many thanks to both of you, gold stars all around!

And I should also add that GoTo bothers me. I don't want to pry, but I would be willing to review your entire application if you want it more thoroughly vetted. Can you show all your code?

i cant post it here, its too big.

i tried to upload it through attachments but the interface changed and i don't know how to attach it. if you can help me with that, i will post the code.

thanks 6stringjazzer

any thoughts on how to do this?

To attach a file, click Go Advanced under the edit window, then on the next screen scroll down and click on Manage Attachments, which will show a pop-up window to Select and then Upload a file. Then close the window and the file will be shown as an attachment in your post.

I have a few high-level suggestions but can't offer any rewritten code without having your file to see how everything works. Here are my thoughts.

You should use Option Explicit and declare all your variables.

You start out with 12 If statements. Each one asks for user input, and the first time they say Yes then you GoTo another line. I would do this with a UserForm that has 12 option buttons and an OK button instead. That would be so much faster than making the user say No 11 times and finally say Yes.

Not sure why you are making calls like this

Please Login or Register  to view this content.

You can just do this

Please Login or Register  to view this content.

You generally do not need to Select objects to work on them, unless you need the user to see what is selected. This is typical of what the macro recorder does.

Please Login or Register  to view this content.

can be done like this

Please Login or Register  to view this content.

You then have two long Case Select statements on xyz. They can be combined into one.

Please Login or Register  to view this content.

can be condensed to this (the first case is weird because you have the OCV case commented out)

Please Login or Register  to view this content.

That's all I have for now.

thanks so much 6stringjazzer, i will make changes to my code to reflect what you have done. gold stars all around!