Umbraco Domain Caching

internal static List<Domain> GetDomains()
{
    return Cache.GetCacheItem<List<Domain>>("UmbracoDomainList", getDomainsSyncLock, TimeSpan.FromMinutes(30),
        delegate
        {
            List<Domain> result = new List<Domain>();
            using(IRecordsReader dr = SqlHelper.ExecuteReader(
                                                    "select id, domainName from umbracoDomains"))
            {
                while(dr.Read())
                {
                    string domainName = dr.GetString("domainName");
                    int domainId = dr.GetInt("id");

                    if (result.Find(delegate(Domain d) { return d.Name == domainName; }) == null)
                        result.Add(new Domain(domainId));
                    else
                        Log.Add(LogTypes.Error, User.GetUser(0), -1,
                            string.Format("Domain already exists in list ({0})", domainName));
                }
            }
            return result;
        });
}