HashMap返回空HashMap的装饰器设计模式

Question

我正在尝试为HashMap实现装饰器设计模式。

我有两个装饰器-“UpperCaseHashMap”和“AddHashMap”。

实施1：-

然而，结果映射仅包含其中一个装饰器的效果，例如：-

>

仅AddHashMap效果，即结果为-{AA=AA}

map map=new AddHashMap<>（new UpperCaseHashMap<>（new HashMap<>()))；Map.Add（“AA”、“AA”）；system.out.println（地图）；

只有UpperCaseHashMap效果，即结果为-{AAADDED_BY_DECROATOR=AA}

map map=new UpperCaseHashMap<>（new AddHashMap<>（new HashMap<>()))；Map.Add（“AA”、“AA”）；system.out.println（地图）；

下面是装修师类的代码：-

    public class UpperCaseHashMap<K, V> extends HashMap<K, V> {

    /**
     * 
     */
    private static final long serialVersionUID = 1L;
    
    Map<K, V> map;
    
    public UpperCaseHashMap() {
        map = new HashMap<>();
    }

    UpperCaseHashMap(Map<K, V> map) {
        this.map = map;
    }

    public V put(K key, V value) {
        String temp = key.toString().toUpperCase();
        key = (K) temp;
        return super.put(key, value);
    }
    
}

    public class AddHashMap<K, V> extends HashMap<K, V> {

    /**
     * 
     */
    private static final long serialVersionUID = 1L;
    
    Map<K, V> map;
    AddHashMap(Map<K, V> map) {
        this.map = map;
    }

    public V put(K key, V value) {
        String temp = key.toString().concat("ADDED_BY_DECORATOR");
        key = (K) temp;
        return super.put(key, value);
        
    }
    
}

实现2：--得到的映射为空

公共类AddHashMapImpl2 扩展HashMapDecorator {

/**
 * 
 */
private static final long serialVersionUID = 1L;

Map<K, V> map;
AddHashMapImpl2(Map<K, V> map) {
    this.map = map;
}

public V put(K key, V value) {
    String temp = key.toString().concat("ADDED_BY_DECORATOR");
    key = (K) temp;
    return map.put(key, value);
    
}

}

public class UpperCaseHashMapImpl2<K, V> extends HashMapDecorator<K, V> {

/**
 * 
 */
private static final long serialVersionUID = 1L;

Map<K, V> map;

public UpperCaseHashMapImpl2() {
    map = new HashMap<>();
}

UpperCaseHashMapImpl2(Map<K, V> map) {
    this.map = map;
}

public V put(K key, V value) {
    String temp = key.toString().toUpperCase();
    key = (K) temp;
    return map.put(key, value);
}

}

Answer 1

您的行：

return super.put(key, value);

正在调用hashmap.put，而不考虑传递给构造函数的映射的实现。相反，您需要：

return map.put(key, value);

不过，我觉得你可能误会了装修工的工作方式。如果使用子类化，则需要将每个方法转发给委托。目前，您的类正在扩展hashmap，但随后专门委托put。这是行不通的。所有其他方法都将转到扩展的hashmap，不会找到放入委托中的数据。你需要重新考虑你的设计。

您可能需要考虑这样的设计：

@FunctionalInterface
interface MapDecorator<V> {
    V decorate(V value);
}

class DecoratedMap<K, V> extends HashMap<K, V> {
    private final List<MapDecorator<V>> decorators = new ArrayList<>();

    public void addDecorator(MapDecorator<V> decorator) {
        decorators.add(decorator);
    }

    @Override
    public V put(K key, V value) {
        for (MapDecorator<V> decorator: decorators)
            value = decorator.decorate(value);
        return super.put(key, value);
    }
}

我在这段代码中使用了'decorator'来匹配您最初的问题，但实际上，这并不是使用decorator设计模式，所以实际上，我会使用一个不同的名称，比如'transformer'。