Eclipse 警告“类型安全：未经检查的演员表”有递归解决方案

Question

在我的例子中，我有这个来操作给定 HashMap 的浅拷贝

public class SomeClass
{
    private HashMap<String, String> hashMap;
    public SomeClass( private HashMap<String, String> hashMap )
    {
        this.hashMap = (HashMap<String, String>) hashMap.clone();
    }
}

但是 eclipse 建议我提取到局部变量或方法或添加演员表，当我这样做时，它一直建议我相同的解决方案:)

我已经来到this post，从接受的答案中我觉得不太清楚

“新的 HashMap 创建调用正在浪费内存”

当我将其应用于我的代码时，我将拥有

if(songData.clone() instanceof HashMap)
{
    this.songData = (HashMap<String, String>) songData.clone();
}

我发现调用clone() 两次会消耗更多的过程。

有没有更好的方法来做到这一点？我的 sn-p 会不会对资源有害？

Answer 1

不要使用克隆。 clone() 是一种设计糟糕的方法，它需要对被克隆的对象有太多的了解，并且无论如何用户都经常错误地实现。 （有关这方面的更多信息，请参阅Effective Java）

复制地图的最佳方法是使用相同的数据创建一个新地图：

public SomeClass( HashMap<String, String> hashMap )
{
    this.hashMap = new HashMap<String,String>(hashMap);
}

另外作为仅供参考，public SomeClass( private HashMap<String, String> hashMap ) 不会编译。您不能使用带有 private 关键字的参数。从参数中删除“private”是可行的。

新的 HashMap 调用会浪费内存

这是您引用的另一篇文章的答案，因为他们的代码是：

private Map<String, String> someMap = new HashMap<String, String>();
someMap = (HashMap<String, String>)...

someMap 是通过调用 new Hashmap() 创建的，但随后该引用立即被替换为不同的 HashMap。

这与您的问题无关，因此您可以忽略它。