クラス内の不変性と状態の変化

Question

私は、不変クラスの概念についてはかなり新しいです。私は、このクラスは不変のものに変更することができる方法を理解しようとしてい

public class ConnectionMonitor implements MessageConsumer { 

    private final MonitorObject monitorObject; 
    private boolean isConnected = true; 

    private final static Logger logger = LogManager.getLogger(ConnectionMonitor.class); 

    public ConnectionMonitor(final MonitorObject monitorObject) { 
     this.monitorObject = monitorObject; 
    } 

    public boolean isConnected() { 
     return isConnected; 
    } 

    public void waitForReconnect() { 
     logger.info("Waiting for connection to be reestablished..."); 
     synchronized (monitorObject) { 
      enterWaitLoop(); 
     } 
    } 

    private void enterWaitLoop() { 
     while (!isConnected()) { 
      try { 
       monitorObject.wait(); 
      } catch (final InterruptedException e) { 
       logger.error("Exception occured while waiting for reconnect! Message: " + e.getMessage()); 
      } 
     } 
    } 

    private void notifyOnConnect() { 
     synchronized (monitorObject) { 
      monitorObject.notifyAll(); 
     } 
    } 

    @Override 
    public void onMessage(final IMessage message) { 
     if (message.getType() == IMessage.Type.CONNECTION_STATUS) { 
      final String content = message.getContent(); 
      logger.info("CONNECTION_STATUS message received. Content: " + content); 
      processConnectionMessageContent(content); 
     } 
    } 

    private void processConnectionMessageContent(final String messageContent) { 
     if (messageContent.contains("Disconnected")) { 
      logger.warn("Disconnected message received!"); 
      isConnected = false; 
     } else if (messageContent.contains("Connected")) { 
      logger.info("Connected message received."); 
      isConnected = true; 
      notifyOnConnect(); 
     } 
    } 
} 

： このクラスを考えてみましょう。

特に、ブール値フィールドisConnectedが接続状態を表すため、最終的にどのように表示されるかわかりません。

ConnectionMonitorのすべてのクライアントは、接続状態を取得するために isConnected()を照会するだけです。

私は、isConnectedへのロック変更が可能であること、またはアトミックブール値を使用していることを認識しています。

しかし、これを不変クラスに書き換える方法はわかりません。

Answer 1

を最小化することは、の変更を最小限に抑えることです。

不変オブジェクトには複数の利点があります。それらはシンプルでスレッドセーフで、自由に共有することができます。

ただし、時には変更が必要です。で

「有効ジャワ、」ジョシュア・ブロックは、これらのガイドラインを提案している：それらを可変させるために非常に良い理由がありますしない限り、

• クラスは不変でなければなりません。
• クラスを不変にすることができない場合は、可能な限りクラスの変更を制限します。

この例では、クラスのインスタンスが変更可能であるという非常に良い理由があります。しかし、プレイ中の第2のガイドラインも見ることができます。フィールドmonitorObjectにはfinalとマークされています。

Answer 2

教科書またはパラダイムの外では、実生活のオブジェクト（単目的クラスを考慮しない場合）はありません。zealot cellarは100％不変またはステートレスです。彼らは通常、彼らの声の音のようなチャンスを得るたびに公開時に不変性を説く人々。ある程度の変更がありますが、ここでの鍵は安全で非暴力的なやり方で行い、理由だけでオブジェクトを変異させないことです。オブジェクトの特定の部分が変更可能でなければならない理由を知り、手元にある可能性のある副作用をすべて知っている場合は、変更可能であることはまったく問題ありません。あなたが言ったように、ブール値isConnectedには、現在の接続状態を反映するためのセッターとゲッターまたはその他のメカニズムが必要です。基本的に2つの状態しか持たないバイナリスイッチであるため、変更可能な影響は最小限です。あなたは他の方法で同じ結果を達成することができますが、別のクラスの外部メソッドでそれを達成するには、フープとループを通過する必要があります。

TL; DR：変更可能な状態は悪いものではありませんが、悪用される可能性があります。

Answer 3

単純にその状態を他の場所に置きます。しかし、あなたの状況のた​​めに、それは論理的ですか？あなたはそれ自身に尋ねるべきです。

ConnectionMonitorを変更可能なままにすることをお勧めします。それは接続を「監視」する責任があるため、変更される可能性のある値を追跡する必要があります。それ以外の場合は、がであり、その状態を追跡できる別のオブジェクトが必要です。あなたは、

class MonitorManager { 
    Map<ConnectionMonitor, Boolean> connectionStatuses = ...; 
} 

それを簡単にする：

あなたはそれがのConnectionStateだとConnectionMonitorsをマップあなたのモニターのためのコンテナクラスを持つことができます：それは十分に説得力はない場合

は、その後、ここではいくつかの方法がありますこのマネージャーを各モニターに渡して、リスナーがマップにアクセスしてその接続のブール値を変更できるようにします。

class ConnectionMonitor { 
    private MonitorManager manager; 

    //.... 

    private void processConnectionMessageContent(final String messageContent) { 
     if (messageContent.contains("Disconnected")) { 
      logger.warn("Disconnected message received!"); 
      manager.connectionStatuses.put(this, false); 
     } else if (messageContent.contains("Connected")) { 
      logger.info("Connected message received."); 
      manager.connectionStatuses.put(this, true); 
      notifyOnConnect(); 
     } 
    } 
} 

しかし、開発者は、子オブジェクトがそのコンテナについて知ってはならないことを伝えるために並んでいます。データの責任が集まっthatsの接続を監視しながら

だから新しいオブジェクトを作成します。

class MonitorManager { 
    private Map<ConnectionMonitor, MonitoredData> data = ...; 

    public void createMonitor() { 
     MonitoredData data = new MonitoredData(); 
     this.data.put(new ConnectionMonitor(data), data); 
    } 
} 

class ConnectionMonitor inplements MessageConsumer { 
    private MonitoredData data; 

    public ConnectionMonitor(MonitoredData data) { 
     this.data = data; 
    } 

    //... 

    private void processConnectionMessageContent(final String messageContent) { 
     if (messageContent.contains("Disconnected")) { 
      logger.warn("Disconnected message received!"); 
     data.setConnected(false); 
    } else if (messageContent.contains("Connected")) { 
      logger.info("Connected message received."); 
      data.setConnected(true); 
      notifyOnConnect(); 
     } 
    } 
} 

class MonitoredData { 
    private boolean connected; 

    public void setConnected(boolean connected) { 
     this.connected = connected; 
    } 

    public boolean isConnected() { 
     return connected; 
    } 
} 

たぶん詳細をMonitoredDataに希望監視されているオブジェクトでより良いフィット感。より多くの文脈を与えたら助けが簡単になるでしょう。