Code running slow

**lou031205** · 11-06-2007, 06:40 AM

I have had lots of help with this code, and now have a userform that does everything I want it to, but it seems suddenly very slow. Could anyone take a look and see if I need to do anything to speed things up?

Also, what would I do to make sure there were no spaces around the Surname and Initial entries in the text boxes?

Please Login or Register  to view this content.

Thank you in advance!

**mudraker** · 11-06-2007, 07:02 AM

I do not understand the logic of having the For Each loop inside a Do loop

**lou031205** · 11-06-2007, 07:22 AM

I don't know, I was just told to put that code in when I asked for help.

I am very new to VBA, and am learning slowly.

Any suggestions?

**lou031205** · 11-07-2007, 04:42 AM

bump for any suggestions in streamlining code better, or reassurance that the code is ok as it is

**royUK** · 11-07-2007, 12:09 PM

This Function checks if a worksheet exists. Place it in a Standard Module

Please Login or Register  to view this content.

Use it in your code to avoid the loops

Please Login or Register  to view this content.

**lou031205** · 11-07-2007, 03:51 PM

Thank you, Roy - have just logged on and seen your reply. Will adjust code accordingly.

**royUK** · 11-08-2007, 02:48 AM

I haven't tested it so let us know if it speeds up.

**lou031205** · 11-08-2007, 11:13 AM

Have to say it doesn't seem to improve things.

I think the noticable difference came when I added the code to format everything into Proper case. I also added extra lines of code at that time to put the textbox entries in three places instead of two. The trouble is, I do need it to do that. Is there something there that is a long-winded way of doing things?

I am just concerned that if it is slow on my brand new laptop using Excel 2007, then I dread to think what it will be like on the old computer my husband uses at work using Excel 2000.

**Norie** · 11-08-2007, 11:16 AM

Why not use UCase instead of StrConv?

Also if you want to lose spaces take a look at the Trim function.

**lou031205** · 11-08-2007, 11:50 AM

Is UCase not upper case? I want to just capitalise the Surname and initial's first letters.

How would I use the Trim function?

Sorry, quite new to all this.