DropDemo 中的 ListTransferHandler 有一个 bug

Question

这是从 OTN 讨论论坛 转发的，因为我在那里没有得到答案。

在 Java 中，我想在包含字符串的 JList 中启用拖放功能。为此，我使用了 Oracle DropDemo 中使用的 ListTransferHandler.java。不幸的是，这个ListTransferHandler.java有一个bug。

为了重现 ListTransferHandler.java 中的错误，我执行了以下操作：

• 从 http://download.oracle.com/javase/tutorial/uiswing/examples/zipfiles/dnd-DropDemoProject.zip
• 在 Netbeans 中运行它，
• 将列表放置模式更改为
• 插入拖放List Item 1 到上面的 List Item 0

而不是：

• List Item 1
• List Item 0

我得到这个：

• List Item 1
• List Item 1

这显然不是预期的结果。

Netbeans 的输出窗口没有显示任何错误消息。我试图自己找到这个错误，但没有成功。

那么我是否错过了在 JList 中启用 DnD 的要点？我不需要自己实现一个ListTransferHandler吗？

有人可以为我提供一个可用的 ListTransferHandler 或者知道如何修复演示中使用的 ListTransferHandler 吗？

谢谢并致以最诚挚的问候， 库尔特

Answer 1

这绝对是一个错误。看一下 ListTransferHandler#cleanup 方法。其目标是从列表中删除先前选择的项目。在删除之前执行索引校正：

        if (addCount > 0) {
            for (int i = 0; i < indices.length; i++) {
                if (indices[i] > addIndex) {
                    indices[i] += addCount;
                }
            }
        }

不知何故，初始化addCount的方法ListTransferHandler#importString从未被调用，因此永远不会进行校正。

作为解决方法，您可以自己初始化它：

        int addCount = indices.length;

这将修复 INSERT 放置模式。

UPD：

刚刚注意到其他放置模式也被破坏了。所以，最终的修复（似乎是这样）：

public class ListTransferHandler extends TransferHandler {
    private boolean insert;
    //........
    public boolean importData(TransferHandler.TransferSupport info) {
        //......
        insert = dl.isInsert();
        //......

    protected void exportDone(JComponent c, Transferable data, int action) {
        cleanup(c, insert && action == TransferHandler.MOVE);
    }

    protected void cleanup(JComponent c, boolean remove) {
        if (remove && indices != null) {
            int addCount = indices.length;
        //.....

}

此外，所有未使用的字段和方法都应该被删除。

Answer 2

@n0weak 感谢您的回答，我可以根据我的需要修复 ListTransferHandler （多个选择仍然被破坏，并且我没有测试除 INSERT 之外的其他模式）。

我发现除了上面的解决方案之外，还有必要记住 addIndex 。以下是我使用的代码更改：

public class ListTransferHandler extends TransferHandler {
    // ...
    private boolean insert;
    // ...

public boolean importData(TransferHandler.TransferSupport info) {
        // ...            
        // method local variables were mistakenly used
        insert = dl.isInsert();
        addIndex = dl.getIndex(); // also replace index with addIndex a few lines below
        addCount = indices.length;
        // ...

protected void exportDone(JComponent c, Transferable data, int action) {
        cleanup(c, insert && action == TransferHandler.MOVE);
    }

/* protected void importString(JComponent c, String str) {...} 
   This method is never called */