There may be a reason why it's rebuilt every time. You'll need to look into that.

static private  Dictionary<string, object> PreferencesDictionary =   BuildPreferencesDictionary();



 static public Dictionary<string, object> GetPreferenceDictionary()
        {
            return PreferencesDictionary;
        }

>Hi everybody,
>
>I see the following code being called:
>
>

> static public Dictionary<string, object> GetPreferenceDictionary()
>        {
>            return BuildPreferencesDictionary();
>        }
>
>

>
>and then in the same class
>
>

> static private Dictionary<string, object> BuildPreferencesDictionary()
>        {
>            Dictionary<string, object> preferenceDictionary = new Dictionary<string, object>();
>           //loading code here
>
>            return preferenceDictionary;
>        }
>
>

>
>this method is called many times.
>
>Why cannot we just return the dictionary if it was already created? What do I need to change here to make this simpler and to build only once?
>
>Thanks in advance.