Double Brace初始化类型混乱

Question

我认为这会相对容易，但唉，似乎它不是。

我目前正在使用Java EE 6在我的项目中为类似Facade的结构编写单元测试 对于测试，我使用Junit 4.11，Eclipse Kepler作为IDE。

从我所看到的，似乎有些东西＆＃34;错误＆＃34;使用双支撑初始化，但我不够了解为什么它不会像我认为的那样工作。

为了达到目的，我正在使用以下类来集中进行转换：

package com.example-company.util.converters;

import java.util.HashMap;
import java.util.Map;

import com.example-company.model.Location;
import com.example-company.model.Right;

public final class ModelConverters {

    private static final Map<Class<?>, ModelConverter<?, String>> modelConverterBacking = new HashMap<Class<?>, ModelConverter<?, String>>();
    static {
        modelConverterBacking.put(Right.class, new RightConverter());
        modelConverterBacking.put(Location.class, new LocationConverter());
    };

    public static <T> String convert(final T input)
            throws IllegalStateException {
        @SuppressWarnings("unchecked")
        ModelConverter<T, String> modelConverter = (ModelConverter<T, String>) modelConverterBacking
                .get(input.getClass());
        if (modelConverter == null) {
            throw new IllegalStateException("No mapping found for "
                    + input.getClass());
        }
        return modelConverter.convertToView(input);
    }
}

就此而言，它主要使用泛型和静态地图。现在我决定为此写一些单元测试。以下课程略有缩短，所有不能重现问题的测试用例都被删除了。

package com.example-company.test.unit.util.converters;

import static org.junit.Assert.assertEquals;
import com.example-company.model.Location;
import com.example-company.util.converters.ModelConverters;

import org.junit.Test;

public class ModelConvertersFacadeTests {

    @Test
    public void test_MappingForLocationExists() {
        final Location stub = new Location() {
            {
                setLocationName("");
            }
        };

        String actual = ModelConverters.convert(stub);
        assertEquals("", actual);
    }
}

到目前为止这么好，没有什么事情真的会发生，至少不是我现在得到的。那就是：一个花哨的IllegalStateException，带有以下堆栈跟踪：

java.lang.IllegalStateException: No mapping found for class com.example-company.test.unit.util.converters.ModelConvertersFacadeTests$1
    at com.example-company.util.converters.ModelConverters.convert(ModelConverters.java:23)
    at com.example-company.test.unit.util.converters.ModelConvertersFacadeTests.test_MappingForLocationExists(ModelConvertersFacadeTests.java:24)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
    at java.lang.reflect.Method.invoke(Method.java:597)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
    at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)
    at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:467)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:683)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:390)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:197)

我做的第一件事是再次运行，然后设置一个断点来检查ModelConverters#convert()

内发生的事情

我对此有些松懈：

似乎input.getClass()返回ModelConvertersFacadeTests。但为什么它不会返回com.example-company.model.Location？

Answer 1

  

看来input.getClass（）返回ModelConvertersFacadeTests

事实并非如此。您的stacktrace说这是班级：

  

com。示例-company.test.unit.util.converters.ModelConvertersFacadeTests $ 1

请注意最后的 $1 。这意味着您的类是匿名的（它没有自己的名称）内部类。

我们在屏幕截图中看到的this$0只是外部类的引用。

每次执行new SomeClass() { ... }时，您都会创建一个匿名内部类。

双括号初始化本身与此无关。每次使用双括号初始化时，您也会创建一个匿名内部类。

---

通过不同的地图查找解决

您的Map有Right.class和Location.class的映射，但它没有这两个类的子类的映射。

static {
    modelConverterBacking.put(Right.class, new RightConverter());
    modelConverterBacking.put(Location.class, new LocationConverter());
};

所做的事情（不是说这是最好的方法），就是遍历地图的按键并检查：

mapKey.isAssignableFrom(input.getClass())

当返回true时，你知道你有一个mapKey类，或者你有一个子类。

您可以循环遍历超类并实现传入的对象的接口，而不是遍历映射键，并为每个接口执行modelConverterBacking.get查找。效果会一样。

---

不使用匿名内部类

进行求解

您当前的代码是：

final Location stub = new Location() {
    {
        setLocationName("");
    }
};

如果你改为：

final Location stub = new Location();
stub.setLocationName("");

然后你不创建任何匿名的内部类，因此不会遇到这个问题。

但是，即使您只是这样做：

final Location stub = new Location() {};
stub.setLocationName("");

然后你有一个匿名的内部类，这会给你带来麻烦。

---

非常重要，不要混淆两个类ModelConvertersFacadeTests$1和ModelConvertersFacadeTests。