Quiz #2 - To be clearer than Ayende

[arialdomartini]

The quiz

Here is a snipped of code I copied from @ayende's RavenDB repository

Find the original code here

The if to be remove is pretty trivial

public void EnsureHasField(string ensuredFieldName)
{
    if (ensuredFieldNames == null)
        ensuredFieldNames = new HashSet<string>();
    ensuredFieldNames.Add(ensuredFieldName);
}

I added a .net solution, with a simple build script in PowerShell, which will download the dependencies with NuGet and will run the tests with nUnit (@naighes, I'm looking forward to see a pull request with a switch to xUnit).

Come on! Can you remove this if?

[arialdomartini]

Proposed solutions

• initialize members into ctor and clone ensuredFieldNames on get Fields by @matteobaglini
• My attempt to remove the two if by @emadb