java.util.AbstractList.add上的UnsupportedOperationException


74

我遇到了使代码块无法正常运行的问题。我不完全确定这段代码的作用(我正在尝试获取一个过时的插件,无法与我们的服务器正常工作),我只知道它每20分钟运行一次并抛出错误。这是发生问题的代码部分:

public class DynamicThread extends Thread {
private LocalShops plugin = null;


public DynamicThread(ThreadGroup tgroup, String tname, LocalShops plugin) {
    super(tgroup, tname);
    this.plugin = plugin;
}

public void run() {
    Map<ItemInfo, List<Integer>> itemStockMap = Collections.synchronizedMap(new HashMap<ItemInfo, List<Integer>>());

    //Dump all the shop stock data into the map.
    for ( Shop shop : plugin.getShopManager().getAllShops() ) {
        for ( InventoryItem item : shop.getItems() ) {
            if (itemStockMap.containsKey(item.getInfo()))
                itemStockMap.get(item.getInfo()).add(item.getStock()); //Where error happens
            else
                itemStockMap.put(item.getInfo(), Arrays.asList(item.getStock()));     
        }
    }
    for(ItemInfo item : itemStockMap.keySet()) {
        List<Integer> stockList = GenericFunctions.limitOutliers(itemStockMap.get(item));
        //remove the map before re-adding it
        if (DynamicManager.getPriceAdjMap().containsKey(item)) 
            DynamicManager.getPriceAdjMap().remove(item);

        //Get the overall stock change for a given item and then calculate the adjustment given the volatility
        int deltaStock = GenericFunctions.getSum(stockList) - Config.getGlobalBaseStock();
        DynamicManager.getPriceAdjMap().put(item, GenericFunctions.getAdjustment(Config.getGlobalVolatility(), deltaStock)); 
    }

    Bukkit.getServer().getScheduler().callSyncMethod(plugin, plugin.getShopManager().updateSigns());
}

}

该错误发生在第42行,它是:

                itemStockMap.get(item.getInfo()).add(item.getStock());

它输出的错误每20分钟发生两次,中间间隔2秒。

2012-02-16 16:53:25 [INFO] Launch Dynamic Thread
2012-02-16 16:53:25 [SEVERE] Exception in thread "dynamic" 
2012-02-16 16:53:25 [SEVERE] java.lang.UnsupportedOperationException
2012-02-16 16:53:25 [SEVERE] at java.util.AbstractList.add(AbstractList.java:131)
2012-02-16 16:53:25 [SEVERE] at java.util.AbstractList.add(AbstractList.java:91)
2012-02-16 16:53:25 [SEVERE] at       com.milkbukkit.localshops.threads.DynamicThread.run(DynamicThread.java:42)

2012-02-16 16:53:27 [INFO] Launch Dynamic Thread
2012-02-16 16:53:27 [SEVERE] Exception in thread "dynamic" 
2012-02-16 16:53:27 [SEVERE] java.lang.UnsupportedOperationException
2012-02-16 16:53:27 [SEVERE] at java.util.AbstractList.add(AbstractList.java:131)
2012-02-16 16:53:27 [SEVERE] at java.util.AbstractList.add(AbstractList.java:91)
2012-02-16 16:53:27 [SEVERE] at     com.milkbukkit.localshops.threads.DynamicThread.run(DynamicThread.java:42)

在此先感谢您的帮助。


是否在ItemInfo对象上覆盖了equals和hashcode,因为您正在对hashmap进行containsKey调用。(itemStockMap.containsKey(item.getInfo()))
Sajan Chandran

Answers:


175

您正在Arrays.asList()用来在Map此处创建列表:

itemStockMap.put(item.getInfo(), Arrays.asList(item.getStock()));  

此方法返回List数组支持的不可调整大小的对象。从该方法的文档中:

返回由指定数组支持的固定大小的列表。(将返回的列表更改为“直写”到数组。)

为了使用可调整大小的List(并实际复制内容),请使用以下命令:

itemStockMap.put(
        item.getInfo(),
        new ArrayList<Integer>(Arrays.asList(item.getStock()))
); 

注意:通常,当看到UnsupportedOperationException被等抛出时add,通常表明某些代码正在尝试修改不可调整大小或不可修改的集合。

例如,Collections.emptyListCollections.singletonList(返回不可修改的集合)可以用作优化,但无意中将其传递给尝试对其进行修改的方法。出于这个原因,一种好的方法是在修改它们之前创建防御性副本的方法(除非修改集合是方法的预期副作用),这是一种好习惯-这样,调用方可以自由地使用最合适的collection实现,而不必担心它是否需要可以修改。


33

我想我已经解决了你的问题。根据传递给它的数组Arrays.asList(item.getStock())返回固定大小的列表

这意味着您不能向其中添加更多元素。

相反,您应该这样做new ArrayList(Arrays.asList(item.getStock()))

这样,您可以创建可以添加到的新列表。


完美的答案!
加拉夫

13

问题是您正在使用Arrays.asList创建列表。根据提供的javadoc,返回的列表是固定大小的,因此不支持添加。将返回的列表包装在arrayList的副本构造函数中,然后设置您。


10

列表是接口,您不能在其中添加值,直到它是ArrayList的实例为止(接口应由某个类实现)

例如:

    List<Integer> test = new ArrayList<>();
    test.add(new Integer(2));

    ArrayList<Integer> test2 = new ArrayList<>();
    test2.add(new Integer(2));

    List<Integer> test3 = Collections.EMPTY_LIST;
    test3.add(new Integer(2));

这里对象测试test2的是完美的,因为他们的目标ArrayList类这样除了可以
同时在TEST3它只是空单,所以你不能把它添加元素。

我也犯了同样的错误。

这是我的建议,当您必须执行添加或删除之类的操作时,请使用ArrayList,仅将List用作参考。

 Map<ItemInfo, ArrayList<Integer>> itemStockMap = Collections.synchronizedMap(new HashMap<ItemInfo, ArrayList<Integer>>());

7

就我而言,我曾经使用过:

List<File> removeFilesList= Collections.emptyList();

这使我的File arraylist抽象化。改为使用:

List<File> removeFilesList= new ArrayList<>();

并且错误已修复。


1

问题出在get调用返回的列表对象的类中。它不会add适当地覆盖方法,因此您的代码正在使用所提供的占位符方法AbstractList

在不知道列表类是什么,以及(如果是自定义代码的情况下)看到源代码的情况下,我们可以说的话不多。

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.