このコードはスレッドセーフですか？ C＃

Question

多くのコードが削除されましたが、実際にこの部分だけを表示する必要があります。主な考え方：メソッドローカライズ（）はすばやく動作する必要があるため、他のスレッドでロックしてはいけません。フラグisLoadedを使用して、使用するローカライゼーションのタイプ（OldまたはCurrent）を検出します。

static class Localization 
{ 
    static readonly object locker = new object(); 

    static string LocalizationDir; 

    static bool isLoaded; 
    static Tuple<string, string> OldLanguage; 
    static Dictionary<string, string> OldLocalization; 
    static Dictionary<string, string> CurrentLocalization; 

    public static Tuple<string, string> CurrentLanguage { 
     get; 
     private set; 
    } 

    static Localization() { 
     lock (locker) { 
      OldLanguage = null; 
      CurrentLanguage = new Tuple<string, string>("en", "English"); 
      isLoaded = true; 

      OldLocalization = null; 
      CurrentLocalization = null; 
     } 
    } 

    public static bool SetLanguage(string languageShortName) { 
     lock (locker) { 
      string languagePath = Path.Combine(LocalizationDir, languageShortName + ".loc"); 

      // save localization, be ready to return it back 
      OldLocalization = CurrentLocalization; 
      OldLanguage = CurrentLanguage; 
      isLoaded = false; 

      try { 
       using (TextReader i = new StreamReader(languagePath)) { 
        /* 
         Parse file, 
         Modify CurrentLocalization, CurrentLocalization 
        */ 
       } 
      } 
      catch (Exception e) { 
       // Just return back our good localization data 

       CurrentLocalization = OldLocalization; 
       CurrentLanguage = OldLanguage; 
       isLoaded = true; 

       OldLocalization = null; 
       OldLanguage = null; 

       return false; 
      } 

      // everything is good 
      { 
       OldLocalization = null; 
       OldLanguage = null; 
       isLoaded = true; 

       UpdateControls(); 
      } 

      return true; 
     } 
    } 


    // <summary> 
    // We think that there are no bugs in this method 
    // No locking 
    // </summary> 
    public static string Localize(this string Text) { 
     if (CurrentLanguage.Item2 == "English") 
      return Text; 

     Dictionary<string, string> ChoosedLocalization = null; 
     if (!isLoaded && OldLocalization != null) 
      ChoosedLocalization = OldLocalization; 
     else if (isLoaded && CurrentLocalization != null) 
      ChoosedLocalization = CurrentLocalization; 

     if (ChoosedLocalization != null) { 
      string Translate; 

      if (!ChoosedLocalization.TryGetValue(Text, out Translate)) 
       return Text; 
      else 
       return Translate; 
     } 
     else 
      return Text; 
    } 
} 

Answer 1

私はそうは思わない。私はまたあなたが物事を思っていると思う。 SetLanguageでは、新しい辞書をロードして古い辞書に入れ替えるだけです。その割り当てはアトミックです。つまり、TranslateコードがCurrentLanguageを2回連続して使用せず、前回の呼び出しが完了するまで（または他の競合状態を防ぐために）UpdateControlsコールが待機すると仮定した場合、シンプルで清潔で信頼性が高くなければなりません。