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（并实际复制内容），请使用以下内容：

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

---

注意：一般来说，当看到 add 等抛出 UnsupportedOperationException 时，这通常表明某些代码正在尝试修改不可调整大小或不可修改的集合。

例如，Collections.emptyList 或 Collections.singletonList（返回不可修改的集合）可能被用作优化，但意外地被传递到尝试修改它们的方法中。因此，方法在修改集合之前制作集合的防御性副本是一种很好的做法（当然，除非修改集合是方法的预期副作用），这样调用者就可以自由地使用最合适的集合实现，而不必担心它是否需要可修改。

Answer 2

我想我已经解决了你的问题。 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

List 是接口，在它是 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));

这里对象 test 和 test2 是完美的，因为它们是ArrayList类的对象，所以可以添加
而在test3中它只是空列表，所以你不能在其中添加元素。

我也犯了同样的错误。

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

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

Answer 6

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

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

Answer 7

就我而言，使用 java 17 它通过使用 toList() 从流中返回列表来引起；然后向其中添加元素，

private static List<String> getCancelTxs(List<BetTransaction> betTransactions) {
        return betTransactions.stream()
                .filter(betTx -> betTx.getType().equals(BetTransactionType.SOME_THING))
                .map(BetTransaction::getExternalTxId)
                .toList();
    }

然后通过以下方式添加：

List<String> cancelTxs = getCancelTxs(betTransactions);
cancelTxs.add(externalTxId);

我将 toList(); 替换为 .collect(Collectors.toList());
它解决了

Answer 8

出现这个问题是因为 List.of() 和 Collections.singletonList() 都创建不可变列表。这意味着您无法添加、删除或修改这些列表中的元素。以下是每个方法的详细说明：

1 : List.of(actionDetail):

List<ActionDetail> actionDetailList = List.of(actionDetail);

actionDetailList.add(new ActionDetail()); // 抛出 UnsupportedOperationException

2 : Collections.singletonList(actionDetail):

List<ActionDetail> actionDetailList = Collections.singletonList(actionDetail);

actionDetailList.set(0, new ActionDetail()); // 抛出 UnsupportedOperationException

因此，应该使用以下内容来操作元素

List<ActionDetail> actionDetailList = new ArrayList<>();
actionDetailList.add(actionDetail);

Answer 9

我使用 getter 来获取列表，但它返回列表的固定大小，并且您无法添加新元素，因此

您应该这样做

new ArrayList(Arrays.asList(item.getStock()))