在一个java语言群里面,有人抛了这么一段代码出来,问题是出现了下下图中的warning,问有什么好的方法消除
这种强转都是因为类型链条断掉了,写入的时候擦除了类型,读出来的时候也就只能强转了,那个instanceof 其实并没有帮到什么忙,无外乎把A异常变成了B异常。
最简单的解决方法也非常直观,就是加上 @SuppressWarnings("unchecked")。
这里先不谈用其他的方法相对优雅的除掉这个warning,而是看看这段代码本身的问题。
这是一个context,这种模式就是一个数据容器,啥都能装,通过编码的人来保证类型匹配,进去擦除类型,出来补上类型,能不能弄对,全看人。
这种模式类似于在其他的语言里面就拿个容器类型就开始编程,忽略一切的type信息。
1. context装进去的是有类型化的对象,出来就没有了,设计上讲究封装性,封装基本的就要保证对称,那么context抹除掉的东西,就应该由他来补上。
2. 由于他没有补上,所以所有使用的地方自己来补充,代码其实会产生很多冗余,考虑到他是一个context,那么实际上其他地方一定还有很多处有类似的代码
3. 这里模拟的其实是一个computeIfAbsent的逻辑,如果没有对象就补一个默认值,然后set进去,借用了isEmpty来婉转的表达容器里面没有这个元素。这里作者应当对java容器上的computeIfAbsent一类的方法不熟悉,因为兜兜转转做的就是这个事情。
4. 由于没有用computeIfAbsent,自己的实现中,无论对象是否存在于context中,都会new一个HashMap出来,这在多数情况下,都是一个浪费。
5. 更不要说,当他发现类型不匹配的时候,仅仅打了一行日志。 元素没有推进去,程序也不知道,只有日志知道了。
1. context做对称设计,出来的时候就把类型转换好,不要让调用者自己来做。 这样抑制告警就在一处即可。
2. 实现computeIfAbsent的逻辑封装,有就返回,没有就插入一个对象,然后返回对象。 对象的创建是按需的。
context的包装类样子如下
public static class SomeClass {
Map<String, Object> context = new HashMap<>();
@SuppressWarnings("unchecked")
public <T> T computeIfAbsent(String key, Supplier<T> defaultGen) {
return (T) context.computeIfAbsent(key, k -> defaultGen.get());
}
}
使用者样子如下
本文系作者在时代Java发表,未经许可,不得转载。
如有侵权,请联系nowjava@qq.com删除。