To improve Efficiency of code, code running too long

**andywsw** · 06-18-2012, 09:53 PM

Hi guys,

i have this code to be run on excel. What i have done previously was spliting the email message body each text to each cells before running this code.

However every alternate cells has a blank cell for this particular sender( i have many sender with different formart, only this affecting) so by running this code will take out all the blank cells. i also notice that the blank cell contains Chr(10) .

This code is working but taking way too long to process for just one email only. So is there anyway to improve on this ? Timing for this was 30 sec ~ 50 Sec to process ( using timer ). About me i am new to VBA , those codes is i happen to learn through the net.

Thanks in an advance !

Please Login or Register  to view this content.

**rylo** · 06-18-2012, 11:37 PM

Hi

If there is always something that will be to the far right on each of the rows to be actioned (ie past the last of the chr(10) items that are to be deleted) then try something like

Please Login or Register  to view this content.

HTH

rylo

**andywsw** · 06-19-2012, 12:25 AM

Hi rlyo

thanks for the response !did try your codes.

It works! but it takes jsut as long as the original i been using

But i am going to use ur code seems more stable ( just the feeling ) !

i think there is not much way to make it faster i assume

Cheers

**rylo** · 06-19-2012, 01:20 AM

Hi

Don't give up quite so easily.

What would happen if you did a replace on the chr(10) items and made them blank? Sure the blank cells are still there, but can you just not process them?

Maybe give us an example of the file, and explain what it is you want to get from it.

rylo

**andywsw** · 06-19-2012, 04:45 AM

try.xlsx

this a sample of a file

**rylo** · 06-19-2012, 05:54 PM

and you want the result to be what? Or the processing to be?

Are you merely trying to make sure that all the data is in the first 3 columns?

What are you wanting to do?

rylo