这是暂时改变当前线程文化的好方法吗？

Question

我正在开发一个相当大的 ASP .NET Web Forms 应用程序，该应用程序目前主要在美国使用。我们正在将其推广到世界其他地区，这当然意味着我们目前正在努力本地化该应用程序的所有领域。一般来说，我们的方法是在每个请求开始时设置当前线程的 CurrentCulture 和 CurrentUICulture 属性，以支持基于当前用户区域设置的正确格式和资源提取。

然而，在某些情况下，我们需要使用当前用户的文化之外的文化来运行某些代码。例如，“用户 A”居住在德国，但在一家与法国其他公司有业务往来的公司工作。当“用户 A”想要为其中一家法国公司创建发票 (PDF) 时，我们希望发票生成代码以“fr-FR”文化而不是“de-DE”文化运行。

我考虑了几种轻松做到这一点的方法，并且想知道我的做法是否正确。我主要关心的是性能和线程安全。

一种方法涉及静态方法，旨在使用提供的区域性运行给定任务。像这样的事情：

 public static void RunWithCulture(CultureInfo culture, Action task)
    {
        if (culture == null)
            throw new ArgumentNullException("culture");

        var originalCulture = new
                                  {
                                      Culture = Thread.CurrentThread.CurrentCulture,
                                      UICulture = Thread.CurrentThread.CurrentUICulture
                                  };

        try
        {
            Thread.CurrentThread.CurrentCulture = culture;
            Thread.CurrentThread.CurrentUICulture = culture;
            task();
        }
        finally
        {
            Thread.CurrentThread.CurrentCulture = originalCulture.Culture;
            Thread.CurrentThread.CurrentUICulture = originalCulture.UICulture;
        }
    }

然后可以像这样调用此方法：

var customerCulture = new CultureInfo(currentCustomer.Locale);
CultureRunner.RunWithCulture(customerCulture, () => invoiceService.CreateInvoice(currentCustomer.CustomerId));

我还考虑创建一个实现 IDisposable 的类，该类负责在其构造函数中设置线程区域性，然后在 Dispose 方法中返回原始区域性，因此您可以这样称呼它：

var customerCulture = new CultureInfo(currentCustomer.Locale);
using(new CultureRunner(currentCustomer.Locale))
{
  invoiceService.CreateInvoice(currentCustomer.CustomerId);
}

我的做法是错的吗？如果有的话，哪种方法更可取？

Answer 1

我喜欢 using 方法。我还会创建一个扩展方法来让内容读起来更好：

var customerCulture = new CultureInfo(currentCustomer.Locale);  
using (customerCulture.AsCurrent()) {
  invoiceService.CreateInvoice(currentCustomer.CustomerId);
}

像这样：

public static class CultureInfoExtensions {
  public static IDisposable AsCurrent(this CultureInfo culture) {
    return new CultureRunner(culture);
  }
}

CultureRunner 示例：

public class CultureRunner : IDisposable
{
    readonly CultureInfo originalCulture;
    readonly CultureInfo originalUICulture;

    public CultureRunner(CultureInfo culture)
    {
        if (culture == null)
            throw new ArgumentNullException(nameof(culture));

        originalCulture = Thread.CurrentThread.CurrentCulture;
        originalUICulture = Thread.CurrentThread.CurrentUICulture;

        Thread.CurrentThread.CurrentCulture = culture;
        Thread.CurrentThread.CurrentUICulture = culture;
    }

    public void Dispose()
    {
        Thread.CurrentThread.CurrentCulture = originalCulture;
        Thread.CurrentThread.CurrentUICulture = originalUICulture;
    }
}

或者，如果始终是您的客户对象设置文化，则另一个扩展方法将进一步提升抽象：

using (currentCustomer.CultureContext()) {
  invoiceService.CreateInvoice(currentCustomer.CustomerId);
}

Answer 2

既然你问暂时改变当前线程的文化是否是个好主意，我只能回答：否。当且仅当没有其他方法使事情正常运转时才可以使用它。那只是因为这种切换很容易出错。好吧，你不会忘记用 Jordão（尊重）给你的代码把事情改回来，但是......
目前，您有想要创建法国发票的客户。我假设您想使用法语日期、数字和货币格式。没关系。但是...如果将来某个未来需要以其他格式打印出来怎么办，例如这个源自德语的格式？你打算创造某种丑陋的解决方法吗？

我知道这可能超出您的控制范围（例如报告软件可能是第三方独立解决方案，您无法控制它如何处理 ToString()），但是如果它在您的控制范围内，我建议首先以正确的格式提供数据。例如，您可以创建一些数据转换层 (DTO) 并正确格式化数据（通过 ToString(IFormatProvider)）。我知道这是一项相当大的努力，但既然你问的是正确的做事方法......

如果我们在同一个组织中并且我会进行 I18n 代码审查，你可以肯定我会指出临时改变文化作为缺点。通常有一种方法可以避免这种情况。