简体   繁体   中英

Hashtable getting null for existing key

I have a very strange error in this code from Vaadin's ContainerHierarchicalWrapper :

for (Object object : children.keySet()) {
    LinkedList<Object> object2 = children.get(object);
}

The debugger shows this state: 调试器

How is that even possible? How can object2 be null ?

This is my actual code which causes the NPE (class EstablishRelationWindow ):

childrenContainer = new BeanItemContainer<>(PlaylistDTO.class);
childrenContainerHierarchy = new ContainerHierarchicalWrapper(childrenContainer);
buildChildrenTree(somePlaylistDTO);

private void buildChildrenTree(PlaylistDTO current) {
    childrenContainer.addBean(current);
    if(current.getChildren().isEmpty()) {
        childrenContainerHierarchy.setChildrenAllowed(current, false);
    } else {
        current.getChildren().forEach(child -> {
            buildChildrenTree(child);
            childrenContainerHierarchy.setParent(child, current); //line 104
        });
    }
}

Relevant output:

java.lang.NullPointerException: null
    at java.util.Collections.sort(Collections.java:175) ~[na:1.8.0_60]
    at com.vaadin.data.util.ContainerHierarchicalWrapper.updateHierarchicalWrapper(ContainerHierarchicalWrapper.java:191) ~[vaadin-server-7.6.8.jar:7.6.8]
    at com.vaadin.data.util.ContainerHierarchicalWrapper$PiggybackListener.containerItemSetChange(ContainerHierarchicalWrapper.java:838) ~[vaadin-server-7.6.8.jar:7.6.8]
    at com.vaadin.data.util.AbstractContainer.fireItemSetChange(AbstractContainer.java:242) ~[vaadin-server-7.6.8.jar:7.6.8]
    at com.vaadin.data.util.AbstractContainer.fireItemSetChange(AbstractContainer.java:228) ~[vaadin-server-7.6.8.jar:7.6.8]
    at com.vaadin.data.util.ContainerHierarchicalWrapper.fireItemSetChangeIfAbstractContainer(ContainerHierarchicalWrapper.java:506) ~[vaadin-server-7.6.8.jar:7.6.8]
    at com.vaadin.data.util.ContainerHierarchicalWrapper.setParent(ContainerHierarchicalWrapper.java:495) ~[vaadin-server-7.6.8.jar:7.6.8]
    at com.xinra.listaide.frontend.EstablishRelationWindow.lambda$3(EstablishRelationWindow.java:104) ~[classes/:na]
    at java.lang.Iterable.forEach(Iterable.java:75) ~[na:1.8.0_60]
    at com.xinra.listaide.frontend.EstablishRelationWindow.buildChildrenTree(EstablishRelationWindow.java:102) ~[classes/:na]

Edit: Code that creates PlaylistDTO objects:

public class DynamicProxyDTOFactory implements DTOFactory {

    @SuppressWarnings("unchecked")
    @Override
    public <T extends DTO> T createDTO(Class<T> type) {
        return (T) Proxy.newProxyInstance(
                type.getClassLoader(),
                new Class<?>[] { type },
                new DynamicProxy());
    }

    private static class DynamicProxy implements InvocationHandler {

        private Map<String, Object> properties = new HashMap<>();

        @Override
        public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
            if(method.getName().startsWith("set")) {
                properties.put(method.getName().substring(3), args[0]);
                return null;
            } else if(method.getName().startsWith("get")) {
                return properties.get(method.getName().substring(3));
            } else if(method.getName().startsWith("is")) {
                return properties.get(method.getName().substring(2));
            } else {
                return method.invoke(this, args); //for equals, hashCode etc.
            }
        }   
    }

}

I put together an SSCCE and investigated the issue further. Turns out, this actually doesn't work at all:

    } else {
        return method.invoke(this, args); //for equals, hashCode etc.
    }

Instead I extended the DynamicProxy as shown in this example :

    @Override
    public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
        if(method.getName().startsWith("set")) {
            properties.put(method.getName().substring(3), args[0]);
            return null;
        } else if (method.getName().startsWith("get")) {
            return properties.get(method.getName().substring(3));
        } else if (method.getName().startsWith("is")) {
            return properties.get(method.getName().substring(2));
        } else if (method.getName().equals("equals")) {
            return proxy == args[0];
        } else if (method.getName().equals("hashCode")) {
            return System.identityHashCode(proxy);
        } else if (method.getName().equals("toString")) {
            return proxy.getClass().getName() + "@" + Integer.toHexString(System.identityHashCode(proxy))
                    + " with InvocationHandler " + this;
        } else {
            throw new IllegalStateException(String.valueOf(method));
        }
    }   

This way hashCode and especially equals work as intended and the NPE does not occur.

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM