code clean up

Hi there,

I made a vba code with double click function

i wonder if there is a cleanup necessary for make it go faster?

Please Login or Register  to view this content.

Hi Ya,

Not sure what does the macro do? But I dont see any unwanted codes in there.


Thanks,

Bonny Tycoon



**If I was able to help please click the small star icon at the bottom left of my post **

Ok thanks

is there no way to speed up the code? because its verry slow.

bye the way its for a sport event. for the scoring paper.

Hi,

I dont see a challenge as to why it would be slow.. Is the source data too big??

Thanks,

Bonny Tycoon



**If I was able to help please click the small star icon at the bottom left of my post **

Originally Posted by AlvaroSiza

One way:

Change this line

Please Login or Register  to view this content.

to

Please Login or Register  to view this content.

Then at the end just before you exit the sub, turn everything back on via:

Please Login or Register  to view this content.

when i use this code its quicker but the calculation isn't working and the next cel i double click the function isn't working anymore.

Thank you all

i tried everything you suggested and found the fastest way.

AlvaroSiza, I hope you don't mind me sharing this, but others may benefit from the question/answer.

Originally Posted by AlvaroSiza

Whiz,

Thanks for the heads up on capturing the current calculation state prior to switching. As I am always learning, what is the benefit of passing the state to string and using the string to 'reset to default state' versus just xlManual / xlAutomatic?

Assume a user's default state is manual and I utilize that default state string (e.g. 'strCalcMode') during my close-out procedures. My code would then lack the re-calculation inherent (and which I am counting on) in the xlAutomatic switch. Under this scenario, I have effectively told excel during initialization of the sub to go to manual (which is my default state and would therefore be redundant and unneccesary) and then at code wrap-up, reset to the default state once again, having never achieved a sheet recalculation.

Am I off? Happy for the pointers and thanks again,

-as-

By changing the calculation state via VBA, you change the expected behavior. A user that has it set as manual will expect it to stay in manual after the code is run. And of course the opposite is true. But then, as you point out, it may be that calculation does not occur because you are changing from manual to manual. The fix for this is to force a calculation when necessary.

Please Login or Register  to view this content.

http://msdn.microsoft.com/en-us/libr...ffice.11).aspx

The nice thing about the Calculate method is that you can calculate the entire workbook, a specific sheet, or even a specific range.