删除/替换数组中的对象

Question

我有一个数组中的单词（对象）列表。每个单词都有一个数字、单词和提示。 （请不要让数字比数组中的索引多 1）我希望用户能够删除数组中的项目。我编写了一种读取用户输入（int）单词的方法，输入索引中的提示采用下一个数组中的提示和单词的值，然后该方法将采用后面的提示和单词我已经编写了该方法，但是每次我删除一个单词时，tat 之后的任何单词都会获取数组中最后一个对象的单词和提示

例如：首先就像

1   dog   bark

2 cat meow

3  cow moo

4  chicken cluck

5  pig  oink

用户删除 3 处的单词后

1 dog  bark

2  cat meow

3 pig oink

4 pig oink

谁能告诉我问题是什么？

代码：

public void deleteWord() throws IOException {

    if (wCount > 0) {
        int again = JOptionPane.YES_OPTION;
        while (again == JOptionPane.YES_OPTION) {
            int num = Integer.parseInt(JOptionPane.showInputDialog(null, "Enter the number of the word you wish to delete", "Enter word number", JOptionPane.PLAIN_MESSAGE))-1;
            int cnfrm = JOptionPane.showConfirmDialog(null, "Are you sure you wish to delete the word:" + "\n" + "\t" + wArr[num].getWrd(), "Are you sure?", JOptionPane.YES_NO_OPTION);
            if (cnfrm == JOptionPane.YES_OPTION) {
                for (int i = num; i < (wCount - 1); i++) {
                    for (int j = (i + 1); j < wCount; j++) {
                        wArr[i].setWrd(wArr[j].getWrd());
                        wArr[i].setHnt(wArr[j].getHnt());
                    }
                }
                wCount--;
                wArr[wCount] = null;
            }
            PrintWriter pw = new PrintWriter(new FileWriter("words.txt", false));
            for (int x = 0; x < wCount; x++) {
                pw.println(wArr[x].toString(1));
            }
            pw.close();
            displayWords();
            again = JOptionPane.showConfirmDialog(null, "Do you wish to delete another word?", "Delete another wod?", JOptionPane.YES_NO_OPTION);
        }
    } else {
        JOptionPane.showMessageDialog(null, "Thre are no words to delete", "ERROR", JOptionPane.ERROR_MESSAGE);
    }

}

编辑：

这是一项家庭作业，这显然意味着我仍然不了解编程，包括。

ArrayList

s。我会了解它们，但不幸的是这个项目（绞刑吏，如果你想知道的话）将于周一到期，所以我不会在这个程序中实现它。

Answer 1

你的问题是这部分：

for (int i = num; i < (wCount - 1); i++) {
    for (int j = (i + 1); j < wCount; j++) {
        wArr[i].setWrd(wArr[j].getWrd());
        wArr[i].setHnt(wArr[j].getHnt());
    }
}

最后总会将

i

位置的内容替换为

wCount - 1

位置的内容。 请改用以下内容：

for (int i = num; i < (wCount - 1); i++) {
    int j = i + 1;
    wArr[i].setWrd(wArr[j].getWrd());
    wArr[i].setHnt(wArr[j].getHnt());
}

但是正如对您的问题的评论中所建议的：为什么不使用 List （如 ArrayList）来代替？

Answer 2

当你应该有一个循环时，你却有两个循环。使用调试器很容易发现这种错误。

for (int i = num; i < (wCount - 1); i++) {
    for (int j = (i + 1); j < wCount; j++) {
        wArr[i].setWrd(wArr[j].getWrd());
        wArr[i].setHnt(wArr[j].getHnt());
    }
}

对于要删除的编号中的每个单词，您都将复制每个单词直到末尾，最后一个就是末尾。

for (int i = num; i < (wCount - 1); i++) {
        wArr[i].setWrd(wArr[i+1].getWrd());
        wArr[i].setHnt(wArr[i+1].getHnt());
}

这会将每个值复制一次。

Answer 3

使用此算法，它将获得内部循环体中 (i, j) 的所有组合，令人惊讶的是，将每个 (j) 分配给元素 (i)，最后分配给 (j) 的最后一个。

但是你不需要这样的算法来删除数组中的元素。你最好复制数组的剩余部分。

Word[] newWArr = new Word[wArr.length - 1];
System.arraycopy (wArr, 0, newWArr, 0, num);
System.arraycopy (wArr, num + 1, newWArr, num, wArr.length - num - 1);
wArr = newWArr;