反双曲余弦 双曲余弦
我不时发现有人在野外使用双花括号反模式(也称为双花括号初始化 )。 这次是堆栈溢出 :
Map source = new HashMap(){{
put("firstName", "John");
put("lastName", "Smith");
put("organizations", new HashMap(){{
put("0", new HashMap(){{
put("id", "1234");
}});
put("abc", new HashMap(){{
put("id", "5678");
}});
}});
}};
如果您不了解语法,实际上很简单。 有两个元素:
- 我们正在创建通过编写以下内容扩展
HashMap
的匿名类 :new HashMap() { }
- 在该匿名类中,我们使用实例初始化程序通过编写以下内容来初始化新的匿名
HashMap
子类型实例:{ put("id", "1234"); }
本质上,这些初始化程序只是构造函数代码。
因此,为什么将其称为“双曲括号反模式”
确实有三个原因使之成为反模式:
1.可读性
这是最不重要的原因,它是可读性。 尽管编写起来可能会更容易一些,并且感觉更像JSON中的等效数据结构初始化:
{
"firstName" : "John"
, "lastName" : "Smith"
, "organizations" :
{
"0" : { "id", "1234" }
, "abc" : { "id", "5678" }
}
}
是的。 如果Java具有List
和Map
类型的集合文字,那将真的很棒。 从句法上讲,使用双花括号进行模拟很奇怪,而且感觉不太正确。
但是,让我们离开讨论口味和花括号的区域( 我们之前已经做过 ),因为:
2.每个实例一种
我们实际上是为每个实例创建一种类型! 每次以这种方式创建新地图时,我们也会隐式地为该HashMap
一个简单实例创建一个新的不可重用类。 如果您只执行一次,那可能很好。 如果将此类代码放在整个大型应用程序中,则会给ClassLoader
带来不必要的负担, ClassLoader
会将对所有这些类对象的引用保留在堆上。 不相信吗? 编译以上代码,并检出编译器输出。 它看起来像这样:
Test$1$1$1.class
Test$1$1$2.class
Test$1$1.class
Test$1.class
Test.class
如果Test.class
是此处唯一合理的类, Test.class
封闭类。
但这仍然不是最重要的问题。
3.内存泄漏!
真正最重要的问题是所有匿名类都有的问题。 它们包含对其封闭实例的引用,这确实是一个杀手.。 想象一下,您将聪明的HashMap
初始化放入了EJB或具有良好管理的生命周期的任何非常重的对象中,如下所示:
public class ReallyHeavyObject {
// Just to illustrate...
private int[] tonsOfValues;
private Resource[] tonsOfResources;
// This method almost does nothing
public void quickHarmlessMethod() {
Map source = new HashMap(){{
put("firstName", "John");
put("lastName", "Smith");
put("organizations", new HashMap(){{
put("0", new HashMap(){{
put("id", "1234");
}});
put("abc", new HashMap(){{
put("id", "5678");
}});
}});
}};
// Some more code here
}
}
因此,此ReallyHeavyObject
拥有大量的资源,这些资源一旦被垃圾回收或其他原因就需要正确清理。 但这对您来说并不重要,当您调用quickHarmlessMethod()
,它会立即执行。
精细。
让我们想象一下其他一些开发人员,他们重构该方法以返回您的地图甚至地图的一部分:
public Map quickHarmlessMethod() {
Map source = new HashMap(){{
put("firstName", "John");
put("lastName", "Smith");
put("organizations", new HashMap(){{
put("0", new HashMap(){{
put("id", "1234");
}});
put("abc", new HashMap(){{
put("id", "5678");
}});
}});
}};
return source;
}
现在您遇到了大麻烦! 现在,您已经无意中将ReallyHeavyObject
所有状态暴露给外部,因为这些内部类中的每个类都包含对封闭实例的引用,该实例是ReallyHeavyObject
实例。 不相信吗? 让我们运行这个程序:
public static void main(String[] args) throws Exception {
Map map = new ReallyHeavyObject().quickHarmlessMethod();
Field field = map.getClass().getDeclaredField("this$0");
field.setAccessible(true);
System.out.println(field.get(map).getClass());
}
该程序返回:
class ReallyHeavyObject
确实是的! 如果您仍然不相信它,可以使用调试器对返回的map
进行自省:
您将在匿名HashMap
子类型中看到封闭的实例引用。 而且所有嵌套的匿名HashMap
子类型也都具有这种引用。
因此,请不要使用这种反模式
您可能会说,规避问题3的所有麻烦的一种方法是使quickHarmlessMethod()
成为防止该封闭实例的静态方法,对此您是正确的。
但是,我们在上面的代码中见过的最糟糕的事情是,即使你知道你与你的地图,你可能在一个静态的背景下创建做什么的事实,未来开发商可能没有注意到这一点,重构/删除static
再次。 他们可能将Map
存储在其他单例实例中,从字面上看,实际上没有办法从代码中得知,可能只有对ReallyHeavyObject
一个悬空的,无用的引用。
内部阶级是野兽。 过去,它们造成了很多麻烦和认知失调。 匿名内部类甚至更糟,因为此类代码的读者可能真的完全忽略了他们封装了外部实例并且绕过这个封闭的外部实例这一事实。
结论是:
不要聪明,不要使用双括号初始化
翻译自: https://www.javacodegeeks.com/2014/12/dont-be-clever-the-double-curly-braces-anti-pattern.html
反双曲余弦 双曲余弦