我有这样的代码:

public static String SelectRandomFromTemplate(String template,int count) {
   String[] split = template.split("|");
   List<String> list=Arrays.asList(split);
   Random r = new Random();
   while( list.size() > count ) {
      list.remove(r.nextInt(list.size()));
   }
   return StringUtils.join(list, ", ");
}

我明白了:

06-03 15:05:29.614: ERROR/AndroidRuntime(7737): java.lang.UnsupportedOperationException
06-03 15:05:29.614: ERROR/AndroidRuntime(7737):     at java.util.AbstractList.remove(AbstractList.java:645)

这样做是正确的吗?Java.15


当前回答

Arrays.asList()返回一个不允许操作影响其大小的列表(注意,这与“不可修改”不同)。

你可以写new ArrayList<String>(Arrays.asList(split));来创建一个真正的副本,但看到你正在尝试做什么,这里有一个额外的建议(你有一个O(n²)算法在下面)。

你想从列表中删除list.size() - count(我们称之为k)随机元素。只是选择尽可能多的随机元素,并将它们交换到列表的最后k个位置,然后删除整个范围(例如使用subList()和clear())。这将把它变成一个精简和平均O(n)算法(O(k)更精确)。

更新:如下所述,这个算法只在元素是无序的情况下才有意义,例如,如果List表示一个Bag。另一方面,如果List具有有意义的顺序,则该算法不会保留它(polygeneluants的算法会保留它)。

更新2:所以回想起来,一个更好的(线性,保持顺序,但有O(n)个随机数)算法应该是这样的:

LinkedList<String> elements = ...; //to avoid the slow ArrayList.remove()
int k = elements.size() - count; //elements to select/delete
int remaining = elements.size(); //elements remaining to be iterated
for (Iterator i = elements.iterator(); k > 0 && i.hasNext(); remaining--) {
  i.next();
  if (random.nextInt(remaining) < k) {
     //or (random.nextDouble() < (double)k/remaining)
     i.remove();
     k--;
  }
}

其他回答

可能是因为您正在使用不可修改的包装器。

更改这一行:

List<String> list = Arrays.asList(split);

到这一行:

List<String> list = new LinkedList<>(Arrays.asList(split));

只需阅读JavaDoc中的asList方法:

返回对象的{@code List} 在指定数组中。的大小 {@code List}不能被修改, 即添加和删除是 不支持,但元素可以支持 集。元素的设置将修改 底层数组。

这是来自Java 6,但它看起来是一样的android Java。

EDIT

结果列表的类型是Arrays。数组列表,它是数组。class中的私有类。实际上,它只是你通过Arrays.asList传递的数组的List-view。结果是:如果你改变了数组,列表也会改变。由于数组不能重新调整大小,因此删除和添加操作必须不受支持。

以下是来自Arrays的代码片段

public static <T> List<T> asList(T... a) {
        return new ArrayList<>(a);
    }

    /**
     * @serial include
     */
    private static class ArrayList<E> extends AbstractList<E>
        implements RandomAccess, java.io.Serializable
    {
        private static final long serialVersionUID = -2764017481108945198L;
        private final E[] a;

所以当asList方法被调用时,它会返回它自己的私有静态类版本的列表,它不会覆盖AbstractList中的add函数来将元素存储在数组中。默认情况下,抽象列表中的add方法会抛出异常。

所以它不是正则数组列表。

问题是您正在使用数组. aslist()方法创建一个具有固定长度的列表 这意味着

由于返回的List是一个固定大小的List,所以不能添加/删除元素。

请看下面我正在使用的代码块

由于它是由asList()创建的迭代列表,因此不可能删除和添加,它是一个固定数组

List<String> words = Arrays.asList("pen", "pencil", "sky", "blue", "sky", "dog"); 
for (String word : words) {
    if ("sky".equals(word)) {
        words.remove(word);
    }
}   

这将很好地工作,因为我们正在获取一个新的数组列表,我们可以在迭代时进行修改

List<String> words1 = new ArrayList<String>(Arrays.asList("pen", "pencil", "sky", "blue", "sky", "dog"));
for (String word : words) {
    if ("sky".equals(word)) {
        words.remove(word);
    }
}

你的代码有很多问题:

在数组。返回一个固定大小的列表

来自API:

数组。asList:返回指定数组支持的固定大小的列表。

你不能再加了;你不能从中移走。您不能在结构上修改List。

Fix

创建一个LinkedList,支持更快的删除。

List<String> list = new LinkedList<String>(Arrays.asList(split));

在分割取正则表达式时

来自API:

字符串。split(String regex):根据给定正则表达式的匹配情况拆分字符串。

|是一个正则元字符;如果你想拆分一个字面值|,你必须将它转义到\|,它作为一个Java字符串字面值是“\\|”。

Fix:

template.split("\\|")

更好的算法

与其每次使用随机索引调用remove一个,不如在范围内生成足够多的随机数,然后使用listIterator()遍历List一次,在适当的索引处调用remove()。关于如何在给定范围内生成随机但不同的数字,有关于stackoverflow的问题。

这样,你的算法就是O(N)