如何避免不同的方法循环相同的列表？

Question

我有一个从XML映射的类。为了简单起见，让我们想象这个类是这样的：

class Employee implements EmployeeIF {
    Map<AttributeIF,Object> attribute = new HashMap<>();
    @Override
    public Map<AttributeIF,Object> getAttributes() { return attribute; }
}

现在，现有的代码中充满了像这样的方法：

public int getSalary(EmployeeIF employee) {
    for(Entry<AttributeIF,Object> entry : employee.getAttributes()) {
        if(entry.getKey().getName().equals("salary")) return (Integer)entry.getValue();
    }
    return 0;
}

public int getAddress(EmployeeIF employee) {
    for(Entry<AttributeIF,Object> entry : employee.getAttributes()) {
        if(entry.getKey().getName().equals("address")) return (String)entry.getValue();
    }
    return "";
}

…等等。你肯定有这个想法。

我需要包含一个新的方法来从employee返回一个新的属性，但是由于我觉得这对mantain来说是可怕的，所以我拒绝在那里添加一个新的方法。我正在考虑使用action模式，以某种方式避免至少重复againg和for循环，但我不得不说，我找不到一个聪明的解决方案。

谢谢,Dani。

是的，我试过类似的事情

private Object getAttribute(EmployeeIF employee, String attribute)

Answer 1

这里有一个小例子，你可以如何获得，基于一个对象作为键，你没有，值。

public class TestObject {
    public String val;

    public TestObject(String val) {
        this.val = val;
    }

    public static TestObject createDummy(String val) {
        return new TestObject(val);
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) return true;
        if (!(obj instanceof TestObject)) return false;
        return ((TestObject)obj).val.equals(this.val);
    }

    @Override
    public int hashCode() {
        System.out.println("THIS ONE IS IMPORTANT");
        return val.hashCode();
    }
}

public class TestMap {
    public Map<TestObject, String> map = new HashMap<>();

    public String get(String keyVal) {
        return map.get(TestObject.createDummy(keyVal));
    }

    public static void main(String[] args) {
        TestMap map = new TestMap();
        TestObject o1 = new TestObject("A");
        map.map.put(o1,"B");
        TestObject o2 = new TestObject("B");
        map.map.put(o2,"C");
        TestObject o3 = new TestObject("C");
        map.map.put(o3,"D");
        System.out.println(map.get("B"));
    }
}

其关键是重写AttributeIf类中的equals和HashCode。因此，如果您传递的是您确实想要的AttributeIf的虚拟对象，则需要将该虚拟对象标识为与map中理论上“相等”的关键对象实例的实例相等。