Question

Is there a good reason (advantage) for programming this style

XmlDocument doc = null;
doc = xmlDocuments[3];

vs

XmlDocument doc = xmlDocuments[3];

I have seen it many times but to me it just seem overly verbose

Answer 1

No – it’s generally considered best practice to declare a variable as late as you can, preferably setting it at the point of declaration. The only time when I don’t do that is when I have to set a variable conditionally, or it’s set in a more restrictive scope:

String name;
using (TextReader reader = ...)
{
    // I can't declare name here, because otherwise it isn't
    // accessible afterwards
    name = reader.ReadToEnd();
}

Reasons for declaring at the point of first use where possible:

• It keeps the variable’s type close to its use – no need to scroll back up in the method to find it.
• It keeps the scope as narrow as possible, which makes the use of the variable more obvious.