このクラスは不変ですか？

Question

このクラスは不変ですか？

私はおおよそ "効果的なJava"、 "Item 15：Minimize mutability"のガイドラインに従おうとしているので、そのクラスは意図的に "最終的"ではありません。

public class Result<T extends Number> { 
    private final Map<String, T> result; 

    private Result(final Map<String, T> result) { 
     this.result = result; 
    } 

    public static final Result<? extends Number> of() { 
     return new Result<>(new LinkedHashMap<>()); 
    } 

    /** Add or replace value. */ 
    public Result<T> put(final String key, final T value) { 
     result.put(key, value); 
     return new Result<>(Collections.unmodifiableMap(result)); 
    } 

    public T get(final String key) { 
     return Collections.unmodifiableMap(result).get(key); 
    } 

    @Override 
    public String toString() { 
     return result.toString(); 
    } 
} 

Answer 1

号put方法は、そのコピーにputを呼び出し、Mapのコピーを作成し、コピーされたマップを構築し、新たなResultを返す必要があります。

/** Add or replace value. */ 
public Result<T> put(final String key, final T value) { 
    Map<T> copy = new LinkedHashMap<>(result); 
    copy.put(key, value); 
    return new Result<>(copy); 
} 

実装もNumberが不変であることのすべてのサブクラスに依存していることに注意してください。それ以外の場合は、getメソッドの実装にも問題があります。

public T get(final String key) { 
    //Relies on T being immutable 
    return result.get(key); 
}