java.util.AbstractList.add中的UnsupportedOperationException

Question

我遇到一些代码块正常运行的问题.我不完全确定这个代码做了什么(我试图让一个过时的插件与我们的服务器一起正常工作),我只知道每运行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)

在此先感谢您的帮助.

Answer 1

您正在使用此处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.emptyList或者Collections.singletonList(返回不可修改的集合)可以用作优化,但不小心会被传递到尝试修改它们的方法中.因此,在修改集合之前制作防御性集合的方法是很好的做法(除非当然修改集合是一种方法的预期副作用) - 这样调用者可以自由地使用最合适的集合实现而不必担心它是否需要可以修改.

Answer 2

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

这意味着您无法向其添加更多元素.

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

这样您就可以创建一个可以添加的新列表.

Answer 3

问题是您使用Arrays.asList创建列表.根据提供的javadoc,返回的列表是固定大小,因此不支持添加.将返回的列表包装在arrayList的复制构造函数中,您应该进行设置.

Answer 4

就我而言，我使用过：

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

这使我的文件数组列表变得抽象。改为使用：

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

并且错误已修复。

Answer 5

列表是接口，您不能在其中添加值，直到它是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>>());

Answer 6

问题出在调用返回的列表对象的类中get。它没有add适当地重写这些方法，因此您的代码使用AbstractList.

如果不知道列表类是什么并且（如果是自定义代码）查看源代码，我们就无话可说。