huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make the ResourceManagerStringLocalizerFactory cache localizer instances 

- #156
This commit is contained in:
damianedwards 2015-12-07 18:15:20 -08:00 committed by DamianEdwards
parent 36bc26fb55
commit 96462b9528
2 changed files with 20 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/Microsoft.Extensions.Localization/ResourceManagerStringLocalizer.cs
View File

@ -3,7 +3,6 @@
using System;
using System.Collections;
using System.Collections.Concurrent;
using System.Collections.Generic;
using System.Globalization;
using System.Reflection;
@ -18,9 +17,7 @@ namespace Microsoft.Extensions.Localization
/// </summary>
public class ResourceManagerStringLocalizer : IStringLocalizer
{
private readonly ConcurrentDictionary<string, object> _missingManifestCache =
new ConcurrentDictionary<string, object>();
private readonly Dictionary<string, object> _missingManifestCache = new Dictionary<string, object>();
private readonly IResourceNamesCache _resourceNamesCache;
private readonly ResourceManager _resourceManager;
private readonly AssemblyWrapper _resourceAssemblyWrapper;
@ -188,7 +185,7 @@ namespace Microsoft.Extensions.Localization
}
catch (MissingManifestResourceException)
{
_missingManifestCache.TryAdd(cacheKey, null);
_missingManifestCache.Add(cacheKey, null);
return null;
}
}

30
src/Microsoft.Extensions.Localization/ResourceManagerStringLocalizerFactory.cs
View File

@ -2,11 +2,12 @@
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
using System;
using System.Collections.Concurrent;
using System.IO;
using System.Reflection;
using System.Resources;
using Microsoft.Extensions.PlatformAbstractions;
using Microsoft.Extensions.OptionsModel;
using Microsoft.Extensions.PlatformAbstractions;
namespace Microsoft.Extensions.Localization
{
@ -16,7 +17,8 @@ namespace Microsoft.Extensions.Localization
public class ResourceManagerStringLocalizerFactory : IStringLocalizerFactory
{
private readonly IResourceNamesCache _resourceNamesCache = new ResourceNamesCache();
private readonly ConcurrentDictionary<string, ResourceManagerStringLocalizer> _localizerCache =
new ConcurrentDictionary<string, ResourceManagerStringLocalizer>();
private readonly IApplicationEnvironment _applicationEnvironment;
private readonly string _resourcesRelativePath;
@ -67,11 +69,13 @@ namespace Microsoft.Extensions.Localization
var baseName = _applicationEnvironment.ApplicationName + "." + _resourcesRelativePath + typeInfo.FullName;
return new ResourceManagerStringLocalizer(
new ResourceManager(baseName, assembly),
assembly,
baseName,
_resourceNamesCache);
return _localizerCache.GetOrAdd(baseName, _ =>
new ResourceManagerStringLocalizer(
new ResourceManager(baseName, assembly),
assembly,
baseName,
_resourceNamesCache)
);
}
/// <summary>
@ -91,11 +95,13 @@ namespace Microsoft.Extensions.Localization
var assembly = Assembly.Load(new AssemblyName(rootPath));
baseName = rootPath + "." + _resourcesRelativePath + baseName;
return new ResourceManagerStringLocalizer(
new ResourceManager(baseName, assembly),
assembly,
baseName,
_resourceNamesCache);
return _localizerCache.GetOrAdd(baseName, _ =>
new ResourceManagerStringLocalizer(
new ResourceManager(baseName, assembly),
assembly,
baseName,
_resourceNamesCache)
);
}
}
}