Java：如何使用clone（）和演员检查呢

Question

此代码：

class RawStringIterator {
        java.util.Stack<State> stateStack = new java.util.Stack<State>();
        RawStringIterator(RawStringIterator i) {
              stateStack = (java.util.Stack<State>) i.stateStack.clone();
        }
        /* ... */
}

给我这个警告：

Type safety: Unchecked cast from Object to Stack<Utils.OperatorTree.RawStringIterator.State>

我想我可以在这里忽略警告。但是我想知道如何使用 clone（）通常？我是否总是必须使用 @suppresswarnings（“ UNCANCED”）每次使用 clone（） ？还是我应该始终进行完全多余的额外检查？

Answer 1

如果您有选择的话，最好的是完全不是实现/使用 clone（），因为这是一个损坏的API 。只需实现 /使用复制构造函数即可。

如果出于某种紧迫的原因，您必须使用 clone（）但可以更改其实现，请考虑声明 stack＆lt; t＆gt; .clone（）才能返回 stack＆lt; t＆gt; 而不是对象 - 协变返回类型是合法的，因为Java5。

更新：如果 stack 是 java.util.stack ，请考虑它的javadoc ：

由 Deque 接口及其实现，应优先使用此类。

和例如 a>提供复制构造函数。

Answer 2

没有办法避免这里的演员阵容。 clone（）返回 object ，如果是 java.util.stack ，它不使用共同变化的返回类型。

如果这不是 java.util.stack ，则请勿实现 clone（） - 很难正确正确。改为复制构造符。

Answer 3

您别无选择，只能忽略它。

虽然不直接相关（因为您没有编写 clone（）方法），但此条目在 Java Generics常见问题可以很好地阅读（整个FAQ也是如此！）

Answer 4

是的，每次使用 clone（）时，您需要明确抑制警告。

这是您可能希望使用复制构造函数而不是 clone（）（如果有）的原因之一。

顺便说一句，当使用 RAWSTRINGITERATOR（RAWSTRINGITERATOR I）构造函数时，不必要的第一个初始化 stateStack 是不必要的：

class RawStringIterator {
    Stack<State> stateStack = new Stack<State>();
    RawStringIterator(RawStringIterator i) {
          stateStack = (Stack<State>) i.stateStack.clone();
    }
    /* ... */
}

您可能需要删除它。