0

我有一个用户列表。其中一些用户拥有第一名，而其他用户拥有第二名。所以我想要将此列表显示为首先它以排序顺序显示存在= 1st的用户，然后以排序顺序显示存在= 2的用户。这里的排序是根据用户名进行的。目前我能够做所有这些事情，但它需要很长时间才能完成，因为该列表有很多用户，大约 250 个。此外，用户的存在可以随时更改。我有一个套接字连接来收听，那时我还必须首先列出列表中的所有内容以显示更新的用户数据。我怎样才能以一种花费更少时间并且不让我的应用程序挂起的方式来做到这一点？

这是我目前正在做的事情：

    List<User> us = new ArrayList<User>();
    int num = model[j].getSize();
    String[] strArr = new String[num];
    for (int i = 0; i < num; i++) {
        strArr[i] = ((User)model[j].get(i)).getName();
        if(!isDuplicateSortedUser(strArr[i], us))
            us.add((User)model[j].get(i));
    }
    sortArray(Collator.getInstance(), strArr);
    User user;
    List<User> temp2 = new ArrayList<User>();
    List<User> temp1 = new ArrayList<User>();
    for (String string : strArr) {
        for (int i = 0; i < num; i++) {
            user = (User) us.get(i);
            if(user.getName().equals(string)){
                if(!isDuplicateSortedUser(user.getUserid(), temp2) && !temp2.contains(user) && !temp1.contains(user)){
                    if(user.getPresence().toLowerCase().equals("1st"))
                        temp2.add(user);
                    else
                        temp1.add(user);
                }
            }
        }
    }
    int l=0;
    for (User user2 : temp1) {
        model[j].setElementAt(user2, l);
        l++;
    }
    for (User user2 : temp2) {
        model[j].setElementAt(user2, l);
        l++;
    }

这model是DefaultListModel的JList。sortArray 方法是：

private void sortArray(Collator collator, String[] strArray) {
        String tmp;
        if (strArray.length == 1) return;
        for (int i = 0; i < strArray.length; i++) {
            for (int j = i + 1; j < strArray.length; j++) {
                if(collator.compare(strArray[i], strArray[j] ) > 0 ) {
                    tmp = strArray[i];
                    strArray[i] = strArray[j];
                    strArray[j] = tmp;
                }
            }
        } 
    }

如何优化上面的代码？

4

2 回答 2

1

将用户放入 ArrayList 并使用内置的 Java 排序功能：http: //download.oracle.com/javase/6/docs/api/java/util/Collections.html

这将调用一个合并排序，这将比您编写的更有效。

你到底想在第一个 for 循环中做什么？

据我所知，您只需要执行以下操作：

  1. 在所有用户上循环一次，将所有“第一个”用户扔到 ArrayList A 中，将所有“第二个”用户扔到 ArrayList B 中
  2. 排序 A，排序 B，
  3. 将 A 和 B 重新合并在一起。
于 2011-05-25T12:53:51.827 回答
0

正如@Jeroen 所指出的，实施Comparable是首选方法。您也可以Comparator按照RecordComparator或使用SortedComboBoxModel.

于 2011-05-25T18:58:43.043 回答