方法は、私のJavaコードは次のようになり、更新フィールド

に同期します私はsync'edする必要があり、他のスレッドは、この期間中に何かを送信し、何かを送信する必要があります。方法は、私のJavaコードは次のようになり、更新フィールド

FindBugsのは私と呼ばれる警告を与える：メソッドが更新フィールド

このメソッドは、変更可能なフィールドから参照されるオブジェクトを同期化に同期します。異なるスレッドが異なるオブジェクト上で同期している可能性があるため、これは有用なセマンティクスを持つことはほとんどありません。

どうすればこの問題を解決できますか？簡単なテストで簡単にこの問題を引き起こすことは可能ですか？

出典

2012-03-14 arge

あなたのコードは無効です - あなたはあなたの 'init'メソッド内でフィールドを宣言することはできません...あなたがすることを意味しました簡単な割り当てですか？ –

変更可能なフィールドは、プライベートチャネル_channelが最終的でないことを意味します。フィールドを最後に設定してみてください。 – darijan

ヒントのためのThx Jon Skeet、私は編集しました、今は正しいはずです。 – arge

_channel = new Channel(...); 
synchronized (_channel)

私はそれがsync'edする必要があり、他のスレッドがチャネルを開けないとこの期間中に何かを送信する必要があります。

あなたのコードのように思われているように思えます。 xHandlerインスタンスごとに新しいChannelオブジェクトを作成し、この特定のChannelロックを取得するので、2つのスレッドが同時に実行される可能性があります。

インスタンス1は、チャネルインスタンス1を作成し、それが
xHandlerインスタンス2ロックです取得すると、それはどちらもあなたが本当に行う必要がある何を同時に

をされ実行

ロックのチャネル・インスタンス2を作成し、取得しxHandler

ムーを同期する特定のChannelオブジェクトを共有するltiple xHandlerインスタンス（例：にに割り当て、コンストラクタにChannelのインスタンスを受け取り、_channel = new Channel(...);行を削除して、同期コードをそのまま使用してください）。
Channel.classで同期します。これは、xHandlerの2つの異なるインスタンスが決して同時に使用できないことを意味します。

二つの異なるxHandlersための具体的なChannelを共有する例：

class xHandler 
{ 
    private final Channel _channel; 

    public xHandler(Channel channel) 
    { 
     this._channel = channel 
    } 

    // Methods... 

    void init() 
    {   
     synchronized (_channel) 
     { 
      // Do some stuff here...e.g. 
      _channel.send("..."); 
     } 
    } 

      public static void main(String[] args) 
      { 
       Channel myChannel = new Channel(...); 
       xHandler xHand1 = new xHandler(myChannel); 
       xHandler xHand2 = new xHandler(myChannel); 
       // Code to create/start your threads. 
       // xHand1 and xHand2 will not use myChannel simultaneously 
      } 

}

出典

2012-03-14 14:20:43