Javaで非静的メソッド変数（オブジェクト）を同期する方法

Question

ファイルに書き込むメソッドが1つあります。私は、ファイルオブジェクト

class MessageFile{ 

public static final String fileName="Main.html" 
@AutoWired 
AppConifg appconfig; 

public boolean writeToFile(String fileContent) throws Exception{ 

     String path = appConfig.getNewsPath() + File.separator + fileName; // getNewsPath is non-static method 
     final File alertFile= new File(path); 
     FileOutputStream out = null; 
     synchronized (alertFile) { 
      if (!alertFile.exists()) { 
       alertFile.createNewFile(); 
      } 
      try { 
       out = new FileOutputStream(alertFile, false); 
       out.write(fileContent.getBytes()); 
       out.flush(); 
      } finally { 
       if (out != null) { 
        out.close(); 
       } 
      } 
     } 
     return true; 
    } 


    } 

しかし、上記のコードは、このクラスの別のインスタンスなどのファイルオブジェクトの排他的ロックは、このクラスのロックを持っているし、ファイルに書き込むことができますロック取るwon`t同期させる必要があります。 だから私はこのケースをどのように処理したいですか？ タイムスタンプを追加する一時ファイル名を作成する回避策が見つかったので（一時ファイル名は常に一意になります）、内容を書き込んだ後、最初に元のファイルを削除し、一時ファイルを元のファイル名に変更します。

Answer 1

MessageFile.classがファイルにアクセスする唯一のオブジェクトであれば、同期を試みることができます。

Answer 2

メソッドが実行されるたびに新しいFileオブジェクト（alertFile）が作成されるため、ロックはメソッドが実行されるたびに異なるので何も実行しません。すべてのメソッド呼び出しでスタティックFileインスタンスを共有する必要があります。

pathは、メソッドが実行されるたびに異なることができれば、あなたはstatic Map<String, File>インスタンスを作成し、このようにそれを使用することができます。

1. は、ファイルのパスを取得します。
2. このパスに関連付けられているFileがない場合は、作成します。
3. それ以外の場合は、マップから既存のFileインスタンスを回復してください。
4. ロックとしてこのFileを使用して操作します。修正回答に基づいて

例：

class MessageFile{ 

    public static final String fileName="Main.html" 
    @AutoWired 
    AppConifg appconfig; 

    private static final Map<String, File> filesMap = new HashMap<>(); 

    public boolean writeToFile(String fileContent) throws Exception{ 

     String path = appConfig.getNewsPath() + File.separator + fileName; // getNewsPath is non-static method 

     final File alertFile; 
     synchronized(filesMap) { 
      if (filesMap.containsKey(path)) { 
       alertFile = filesMap.get(path); 
      } 
      else { 
       alertFile = new File(path); 
       filesMap.put(path, alertFile); 
      } 
     } 

     FileOutputStream out = null; 
     synchronized (alertFile) { 
      if (!alertFile.exists()) { 
       alertFile.createNewFile(); 
      } 
      try { 
       out = new FileOutputStream(alertFile, false); 
       out.write(fileContent.getBytes()); 
       out.flush(); 
      } finally { 
       if (out != null) { 
        out.close(); 
       } 
      } 
     } 
     return true; 
    } 


} 

Answer 3

あなたはクラスMessageFile（それぞれのインスタンス間で共有されていないローカル変数alertFileに​​を使用しているので、あなたのプログラムは、ファイルの排他ロックを取得できません。オブジェクトには独自のalertFileがあります）。

1静的オブジェクトをいくつか作成して同期します（既にあるのでfileNameを使用することができます）。

2同じオブジェクト（コンストラクタで渡された）を指すすべてのオブジェクト内の参照を持ち、それを同期させます。

Answer 4

クラスレベルのオブジェクト、つまりMessageFile.classを同期させるか、静的同期メソッドwrtietofile（）を使用します。一度に1つのスレッドだけがファイルに書き込むようにします。また、スレッドによってファイル全体にデータが書き込まれると、ロックが解除されることが保証されます。