Hello, this is my first time posting.
I've just started a new role taking over from someone who has just built a tool in VBA for a project. I'm reviewing the code to get up to speed with how the tool works. I haven't got all the way through it yet, but I know the code takes a couple of hours to run (which seems excessive to me), and also cashes often.
I've noticed that the person has used what I would consider a large number of Public variables - 54, amongst these there are several dictionaries.
I'm only meant to be spending time on the code if I see anything that would solve the speed or stability issues.
So I don't really want to spend time restructing the code only to find it doesn't make much of a difference.
My understanding is that best practice is to keep Public variables to a minimum due to memory uses.
I've just started a new role taking over from someone who has just built a tool in VBA for a project. I'm reviewing the code to get up to speed with how the tool works. I haven't got all the way through it yet, but I know the code takes a couple of hours to run (which seems excessive to me), and also cashes often.
I've noticed that the person has used what I would consider a large number of Public variables - 54, amongst these there are several dictionaries.
I'm only meant to be spending time on the code if I see anything that would solve the speed or stability issues.
So I don't really want to spend time restructing the code only to find it doesn't make much of a difference.
My understanding is that best practice is to keep Public variables to a minimum due to memory uses.